Closed
Bug 526008
Opened 16 years ago
Closed 16 years ago
New homepage doesn't validate to its doctype
Categories
(support.mozilla.org :: General, defect)
support.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
1.5
People
(Reporter: stephend, Assigned: jsocol)
References
()
Details
(Whiteboard: sumo_only)
Attachments
(1 file)
|
8.72 KB,
patch
|
morgamic
:
review+
|
Details | Diff | Splinter Review |
Load http://validator.w3.org/check?uri=http%3A%2F%2Fsupport-stage.mozilla.org%2Fen-US%2Fkb%2F&charset=%28detect+automatically%29&doctype=Inline&group=0; we currently have 69 Errors, 61 warning(s).
| Assignee | ||
Comment 1•16 years ago
|
||
Looks like there are main issues that should clear up almost all of this:
* unescaped ampersands, and
* re-used IDs.
There are also role attributes from bug 519333, but those should go away with bug 526007.
| Assignee | ||
Updated•16 years ago
|
Assignee: nobody → james
| Assignee | ||
Comment 2•16 years ago
|
||
This makes the new home page (with the mozms2) theme validate as HTML5 (with the patch to bug 526007) and XHTML 1.0 Transitional (without it)--save one error from the ARIA roles.
Attachment #410368 -
Flags: review?(morgamic)
| Assignee | ||
Comment 3•16 years ago
|
||
I should add, bug 526638 will need to be fixed to fully clean up the validation errors. But without this patch, fixing that bug will break the layout. (ugh)
Comment 4•16 years ago
|
||
Comment on attachment 410368 [details] [diff] [review]
v1
This works for me, but there are still errors. Looks like you're aware of it.
Attachment #410368 -
Flags: review?(morgamic) → review+
| Assignee | ||
Comment 5•16 years ago
|
||
r55129
This fixes everything except:
The duplicate IDs from bug 526638.
One role attribute which is an error in XHTML 1.0 but will be fine in HTML5.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 6•16 years ago
|
||
Verified FIXED; we're down to 4 errors. A tad bit better than 69 ;-)
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Whiteboard: sumo_only
You need to log in
before you can comment on or make changes to this bug.
Description
•