Closed Bug 428886 Opened 17 years ago Closed 13 years ago

enhancement: don't ignore newlines in HTML tooltips

Categories

(Firefox :: General, enhancement, P4)

enhancement

Tracking

()

RESOLVED DUPLICATE of bug 358452
Firefox 3.6a1

People

(Reporter: amnonaar, Assigned: amnonaar)

Details

(Whiteboard: [has patch] [needs review gavin])

Attachments

(1 file)

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
Attachment #315465 - Flags: review?(gavin.sharp)
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).
Version: unspecified → Trunk
We can't ask to W3C to change this behaviour? It obviously doesn't make any sense.
Assignee: nobody → amnonaar
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Whiteboard: [has patch] [needs review gavin]
Just to say, I've found this to be an issue as well.
Priority: -- → P4
Target Milestone: --- → Firefox 3.2a1
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)
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.

Attachment

General

Created:
Updated:
Size: