Closed
Bug 428886
Opened 17 years ago
Closed 13 years ago
enhancement: don't ignore newlines in HTML tooltips
Categories
(Firefox :: General, enhancement, P4)
Firefox
General
Tracking
()
RESOLVED
DUPLICATE
of bug 358452
Firefox 3.6a1
People
(Reporter: amnonaar, Assigned: amnonaar)
Details
(Whiteboard: [has patch] [needs review gavin])
Attachments
(1 file)
1.02 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.12) Gecko/20060601 Firefox/2.0.0.12 (Ubuntu-edgy)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008011304 Minefield/3.0b3pre
According to the HTML standard, in the TITLE attribute, CRs should be converted to spaces while LFs should be discarded. This has been implemented in trunk (bug 218223). However, I propose to ignore the standard this time because:
(1) this causes documents that are identical except for newline conventions (LF or CR/LF) to appear differently -- doesn't make sense
(2) it causes words that are separated by whitespace in the source to be not separated in the rendered document -- also doesn't make sense
So I'm asking that LF characters be converted to spaces along with the other whitespace chars.
Example:
<a title="simple
test">test</a>
shows as "simpletest" with UNIX newlines and "simple test" with DOS newlines.
Reproducible: Always
Assignee | ||
Comment 1•17 years ago
|
||
Attachment #315465 -
Flags: review?(gavin.sharp)
Comment 2•17 years ago
|
||
The HTML5 draft is very clear that LF "represents a line break":
http://www.w3.org/html/wg/html5/#the-title
> (LF or CR/LF) to appear differently -- doesn't make sense
FWIW, I agree, but I think we should make a line break for any combination,
for compatibility with other popular browsers (IE and Safari).
Updated•17 years ago
|
Version: unspecified → Trunk
Comment 3•17 years ago
|
||
We can't ask to W3C to change this behaviour? It obviously doesn't make any sense.
Updated•16 years ago
|
Assignee: nobody → amnonaar
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Whiteboard: [has patch] [needs review gavin]
Comment 4•16 years ago
|
||
Just to say, I've found this to be an issue as well.
Updated•16 years ago
|
Priority: -- → P4
Target Milestone: --- → Firefox 3.2a1
Comment 5•16 years ago
|
||
Comment on attachment 315465 [details] [diff] [review]
convert LF chars to space instead of ignoring them
Apologies for the long delay in reviewing here...
Making this change seems reasonable, but I'm reluctant to do it without comparing our behavior to the other mainstream browsers (Safari, Opera, IE, etc.), ideally on as wide a range of testcases as possible. Bug 218223 includes some testcases that might be useful.
Attachment #315465 -
Flags: review?(gavin.sharp)
Updated•13 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•