701 bytes, patch
|Details | Diff | Splinter Review|
When doing a W3C HTML validation, mozilla.org/contribute does not validate correctly to its doctype. Results: http://validator.w3.org/check?uri=http%3A%2F%2Fwww.mozilla.org%2Fcontribute&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.654
Happy to fix this if there's a patch available.
Assignee: nobody → tobbi.bugs
Status: NEW → ASSIGNED
Created attachment 449567 [details] [diff] [review] HTML5 compatibility patch I think this patch should correct the problem!
Urgh, now that I see it: Does it make sense to remove the two ending </div>s in footer.inc.php or would it be better if I'd remove them in contribute/index.html?
Re question in comment #3, I haven't looked at the patch closely yet but if the div tags in the footer are a problem across the site I imagine it makes sense to update footer.inc.php. If this is just a problem on this page, probably remove them in contribute/index.html. Will wait to deal with patch until I hear back.
I've seen that my patch is wrong. Other HTML files use the template as well and they pass the test. I'll submit a new one asap.
Created attachment 449658 [details] [diff] [review] v.2: Removing stray <div> ending tags in the index file itself
I just checked in a change that seems to be working. http://viewvc.svn.mozilla.org/vc?view=revision&revision=68427 Would be good to have the page and form verified to make sure nothing got busted.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.