If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

use of wbr tag by YahooGroups corrupts text in simple HTML view

RESOLVED FIXED in Thunderbird 3

Status

Thunderbird
Preferences
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Stuart Halliday, Assigned: alta88)

Tracking

unspecified
Thunderbird 3

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.8.1.14) Gecko/20080404 Firefox/2.0.0.14
Build Identifier: 2.0.0.14 (20080421)

YahooGroups when sent a plain text email returns it to its members with a HTML component added. This component contains numerous <wbr> tags which causes Thunderbird to display a space character when viewed in Simple HTML mode.

This causes quoting and copying of the text or code displayed in emails to be corrupted.

Reproducible: Always

Steps to Reproduce:
1. Send a plain text message to YahooGroups consisting of long phrases (urls for example or code)
2. Receive email from YahooGroups and view in Simple HTML

Actual Results:  
Note that <wbr> have been inserted into the HTML part of the email.

Long urls or code sections have 'spaces' inserted into it which renders the use of the url or code corrupt.

ie
VisualBasic code section from a html email from YahooGroups

Set objIE = CreateObject(<wbr>&quot;InternetExplore<wbr>r.Application&quot;<wbr>)<br>
<br>
With objIE<br>
       .visible=1<br>
       .left=1000<br>
       .top=00<br>
       .height=123<br>
       .width=260<br>
       .menubar=0<br>
       .toolbar=0<br>
       .statusBar=0<br>
       .FullScreen=<wbr>0<br>
       .AddressBar=<wbr>0<br>
       .Resizable=0<br>
       .navigate &quot;c:\spell_it_<wbr>form.htm&quot;<br>
End With<br>

this is shown as:

Set objIE = CreateObject( "InternetExplore r.Application" )

With objIE
       .visible=1
       .left=1000
       .top=00
       .height=123
       .width=260
       .menubar=0
       .toolbar=0
       .statusBar=0
       .FullScreen= 0
       .AddressBar= 0
       .Resizable=0
       .navigate "c:\spell_it_ form.htm"
End With



Expected Results:  
I expect that <wbr> should be displayed as no character.

Adding the wbr tag to pref "mailnews.display.html_sanitizer.allowed_tags" gets rid of the problem.

As there are thousands of Thunderbird users using YahooGroups, this is a problem which potentially affects a lot of users.
(Reporter)

Comment 1

9 years ago
Created attachment 325385 [details]
Typical received email from a YahooGroup
(Reporter)

Comment 2

9 years ago
Created attachment 325386 [details]
Typical email from YahooGroups showing url corruption
(Reporter)

Comment 3

9 years ago
Easy to fix, add wbr to the list of tags already in "mailnews.display.html_sanitizer.allowed_tags" in the default build so that all Thunderbirds users can benefit.

Though should <wbr> be shown as a space in the first place?
In Original HTML view the tag is displayed as 'no character'. Only in Simple HTML is it shown as a single space character. So this may be a bug.

I've tried to tell fellow Thunderbird users on my YahooGroups, but they're ordinary folk and run a mile if I give instructions on how to launch the advanced config editor. :-)

 
(Reporter)

Comment 4

9 years ago
Also if I view a email in Simple View with these <wbr> tag showing as spaces, if I copy the text displayed then the copy is made to the clipboard with these spaces in.

If I click on the reply icon in Thunberbird then the email is quoted with the <wbr> replaced by spaces.

So if other people on YahooGroups then see these quoted emails, they can't click on the urls quoted as they're corrupted with spaces...

(Assignee)

Updated

9 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 5

9 years ago
Created attachment 325496 [details] [diff] [review]
patch for review
Attachment #325496 - Flags: review?(neil)

Updated

9 years ago
Attachment #325496 - Flags: review?(neil) → review+
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
This patch is in mailnews/ the review rules (http://www.mozilla.org/mailnews/review.html) require sr for patches in mailnews/. Given the size of the patch, you could have got it from Neil at the same time as the review (and you actually still can).
Keywords: checkin-needed
Assignee: nobody → alta88
(Assignee)

Comment 7

9 years ago
Comment on attachment 325496 [details] [diff] [review]
patch for review

sr? neil, please checkin while you're at it..
Attachment #325496 - Flags: superreview?(neil)

Updated

9 years ago
Attachment #325496 - Flags: superreview?(neil) → superreview+
Sorry for the spam; I just saw that this has been checked in a while ago. Any reason not to close the bug?
(Reporter)

Comment 9

9 years ago
Has the patch been approved to be added to the next Build release?
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Comment 10

9 years ago
Yes (for thunderbird3).
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → Thunderbird 3
You need to log in before you can comment on or make changes to this bug.