Closed Bug 1013897 Opened 6 years ago Closed 6 years ago

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

Categories

(DevTools :: Style Editor, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 32

People

(Reporter: jwalker, Assigned: jwalker)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.
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.
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #8426379 - Flags: review?(fayearthur)
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
Attachment #8426379 - Flags: review?(fayearthur)
Landed with bug 975522.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Depends on: 975522
Target Milestone: --- → Firefox 32
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.