1013897 - CSS coverage should allow generation of more than 1 report for a target

Status: RESOLVED FIXED

(DevTools :: Style Editor, defect)

Description

[Joe Walker [:jwalker]]

Previously is one box which contains the template, so the second time we create a report, the template was used already. We need 2 boxes the first containing the template which we clone before altering and moving it to the second box.

At the same time fix the indentation of the existing style editor components, and use a generator function to reduce indentation in the gcli converter.

Comment 1

[Joe Walker [:jwalker]]

Attachment: 0004-Bug-1013897-Allow-generation-of-more-than-one-report.patch

Previously there was one box which contained the template, so the second time we created a report, the template was used. Now there are 2 boxes the first containing the template which we clone before altering and moving it to the second box.

At the same time we fix the indentation of the existing style editor components, and use a generator function to reduce indentation.

Comment 2

[Joe Walker [:jwalker]]

Comment on attachment 8426379 [details] [diff] [review]
0004-Bug-1013897-Allow-generation-of-more-than-one-report.patch

r+ed and will land with bug 975522

Comment 3

[Joe Walker [:jwalker]]

Landed with bug 975522.