UI Footer does not respect HTML 4 specifications

RESOLVED FIXED in Bugzilla 2.20

Status

()

defect
RESOLVED FIXED
16 years ago
6 years ago

People

(Reporter: goobix, Assigned: Wurblzap)

Tracking

2.17.6
Bugzilla 2.20
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
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

16 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
&nbsp;'s, that's another issue - but at least IE will then start wrapping on
hyphens...
(Assignee)

Updated

15 years ago
Assignee: myk → wurblzap
Status: NEW → ASSIGNED
(Assignee)

Comment 4

15 years ago
(In reply to comment #2)
> fixed by bug 245924?

No, unfortunately not.
Depends on: 245282
(Assignee)

Updated

15 years ago
Attachment #170246 - Flags: review?

Comment 5

15 years ago
Comment on attachment 170246 [details] [diff] [review]
Use new no_break filter

r=LpSolit
Attachment #170246 - Flags: review? → review+

Updated

15 years ago
Flags: approval?
(Assignee)

Updated

14 years ago
Whiteboard: patch awaiting approval
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.20
(Assignee)

Updated

14 years ago
Whiteboard: patch awaiting approval → patch awaiting checkin

Comment 6

14 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
Last Resolved: 14 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.