Remove useless h1 element when getting article title

VERIFIED FIXED in Firefox 16

Status

()

defect
VERIFIED FIXED
7 years ago
3 years ago

People

(Reporter: lucasr, Assigned: lucasr)

Tracking

unspecified
Firefox 17
All
Android
Points:
---

Firefox Tracking Flags

(firefox16 verified)

Details

Attachments

(1 attachment)

Assignee

Description

7 years ago
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+
https://hg.mozilla.org/mozilla-central/rev/e97800e74b63
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Assignee

Comment 3

7 years ago
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+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.