Closed Bug 1208685 Opened 6 years ago Closed 6 years ago

Dedupe alert.css and cleanup alert.js heading

Categories

(Toolkit :: Notifications and Alerts, defect)

defect
Not set
minor
Points:
2

Tracking

()

RESOLVED FIXED
mozilla44
Iteration:
44.1 - Oct 5
Tracking Status
firefox44 --- fixed

People

(Reporter: MattN, Assigned: MattN)

Details

Attachments

(1 file)

* Most of the CSS is shared so make a shared style sheet
* Use the common Components line
* Use Services.wm
* Remove JS modeline
Bug 1208685 - Dedupe alert.css and cleanup alert.js heading. r=bgrins
Attachment #8666289 - Flags: review?(bgrinstead)
Comment on attachment 8666289 [details]
MozReview Request: Bug 1208685 - Dedupe alert.css and cleanup alert.js heading. r=bgrins

https://reviewboard.mozilla.org/r/20511/#review18387

Looked over this earlier in a diff tool and it seems fine
Attachment #8666289 - Flags: review?(bgrinstead) → review+
https://hg.mozilla.org/mozilla-central/rev/743f99764667
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.