Closed Bug 227990 Opened 21 years ago Closed 20 years ago

UI Footer does not respect HTML 4 specifications

Categories

(Bugzilla :: User Interface, defect)

2.17.6
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: goobix, Assigned: Wurblzap)

References

Details

Attachments

(1 file)

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.
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
&nbsp;'s, that's another issue - but at least IE will then start wrapping on
hyphens...
fixed by bug 245924?
Assignee: myk → wurblzap
Status: NEW → ASSIGNED
(In reply to comment #2)
> fixed by bug 245924?

No, unfortunately not.
Depends on: 245282
Attachment #170246 - Flags: review?
Comment on attachment 170246 [details] [diff] [review]
Use new no_break filter

r=LpSolit
Attachment #170246 - Flags: review? → review+
Flags: approval?
Whiteboard: patch awaiting approval
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.20
Whiteboard: patch awaiting approval → patch awaiting checkin
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
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: