No need to enclose the title in a h1 element as we only use the text anyway.
Attachment #654216 - Flags: review?(mark.finkle)
Attachment #654216 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Comment on attachment 654216 [details] [diff] [review] Remove useless h1 node around title [Approval Request Comment] User impact if declined: Useless memory usage, unnecessary code. Testing completed (on m-c, etc.): Landed in m-c, no regressions. Risk to taking this patch (and alternatives if risky): Very low, just removes unnecessary code. String or UUID changes made by this patch: None.
Attachment #654216 - Flags: approval-mozilla-aurora?
Attachment #654216 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Pushed to mozilla-aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/bd8d5ac3a368
You need to log in before you can comment on or make changes to this bug.