Closed
Bug 227990
Opened 21 years ago
Closed 20 years ago
UI Footer does not respect HTML 4 specifications
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: goobix, Assigned: Wurblzap)
References
Details
Attachments
(1 file)
882 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
We need to rephase "<nobr>[% q.name FILTER html %]</nobr>" in another way (<nobr> is not accepted as a valid HTML tag by the online http://validator.w3.org ). It's located in template/en/default/global/useful-links.html.tmpl . Due to the fact that it's in the footer, it makes most pages HTML-not valid.
Comment 1•21 years ago
|
||
Now, do we really need to? I agree validity is good, but there are several types of invalidity, of which extension attributes and elements are the least harmful. Browsers usually ignore unknown elements and attributes ("ignoring" elements means rendering its content without regarding the element around it). This behavior is supported and endorsed by the HTML specification (see <http://www.w3.org/TR/html401/appendix/notes.html#notes-invalid-docs>). Therefore, I'm in on using this sort of well-supported extension attributes to achieve an useful effect, as long as we're not counting on them for any critical behavior. Of course, CSS would be better. Our practical alternative here is |white-space: nowrap;|, but it has its share of problems, too. One, it's not particularly well supported especially on 5.x IEs. Secondly, CSS spec used to mark white-space as a valid rule for _block_ elements only, so most really good uses of it are invalid in the face of a CSS validator. Though, this was fixed in CSS2 errata, so it's probably a non-issue these days. I don't really care which one we use, but my gut feeling is that <nobr> still works better. Of course, if we're simply satisfied by replacing spaces with 's, that's another issue - but at least IE will then start wrapping on hyphens...
fixed by bug 245924?
Updated•20 years ago
|
Blocks: bz-validhtml
Assignee | ||
Comment 3•20 years ago
|
||
Assignee | ||
Updated•20 years ago
|
Assignee: myk → wurblzap
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•20 years ago
|
||
(In reply to comment #2) > fixed by bug 245924? No, unfortunately not.
Depends on: 245282
Assignee | ||
Updated•20 years ago
|
Attachment #170246 -
Flags: review?
Comment 5•20 years ago
|
||
Comment on attachment 170246 [details] [diff] [review] Use new no_break filter r=LpSolit
Attachment #170246 -
Flags: review? → review+
Updated•20 years ago
|
Flags: approval?
Assignee | ||
Updated•20 years ago
|
Whiteboard: patch awaiting approval
Updated•20 years ago
|
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.20
Assignee | ||
Updated•20 years ago
|
Whiteboard: patch awaiting approval → patch awaiting checkin
Comment 6•20 years ago
|
||
Checking in template/en/default/global/useful-links.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/useful- links.html.tmpl,v <-- useful-links.html.tmpl new revision: 1.33; previous revision: 1.32 done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: patch awaiting checkin
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•