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

Incomplete Header in Account Settings - Security

RESOLVED FIXED in Thunderbird 3.0b2

Status

Thunderbird
Account Manager
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Willie Reid, Assigned: philor)

Tracking

({regression})

Trunk
Thunderbird 3.0b2
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090130 SeaMonkey/2.0a3pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090130 Shredder/3.1a1pre

Open Account Settings and select Security. The header is shortened, preceded with the code orient="vertical">
Happens in all accounts and with fresh new profile containing one account.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
(Reporter)

Comment 1

9 years ago
[URL=http://img218.imageshack.us/my.php?image=accountsettingses6.jpg][IMG]http://img218.imageshack.us/img218/4676/accountsettingses6.th.jpg[/IMG][/URL]
Created attachment 359933 [details]
Screenshot ffrom the reporter

This is not a Forum and Forum-Codes doesn't work here. Either just post the URL as comment without addong additional Codes or better add it in the URL field of the bug or the best thing is to attach it as attachment
Component: General → Account Manager
QA Contact: general → account-manager
(Reporter)

Comment 3

9 years ago
Oops! Sorry. Thanks for doing it for me.
(Assignee)

Comment 4

9 years ago
The fun part is that it's a regression from bug 278549, which landed last June, but apparently that stray bit of content in http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/mailnews/extensions/smime/resources/content/am-smime.xul&rev=1.18&mark=51#51 doesn't actually show in 1.9.1, only in 1.9.2 (which, Willie, did you really intend to be using 3.1a1pre rather than 3.0b3pre, or did you get caught by the accidental update Thursday and you haven't yet downloaded a 3.0 nightly to get back?).
Assignee: nobody → philringnalda
Blocks: 278549
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: regression
OS: Windows XP → All
Hardware: x86 → All
Version: unspecified → Trunk
(Reporter)

Comment 5

9 years ago
Well, I've actually been running both and yes, it didn't show in 1.9.1
(Assignee)

Comment 6

9 years ago
Created attachment 359970 [details] [diff] [review]
Remove 'em

I'm not quite sure just what a <page orient="vertical"> is, but since we've done without this one being vertical for seven months now, and since most of the am-*.xuls had their orient="vertical" removed by Blake in some 2001 XUL syntax change megapatch, I'm pretty sure we'll get by without these two.
Attachment #359970 - Flags: superreview?(neil)
Attachment #359970 - Flags: review?(neil)

Updated

9 years ago
Attachment #359970 - Flags: superreview?(neil)
Attachment #359970 - Flags: superreview+
Attachment #359970 - Flags: review?(neil)
Attachment #359970 - Flags: review+

Comment 7

9 years ago
Comment on attachment 359970 [details] [diff] [review]
Remove 'em

<page orient="vertical"> is like <vbox orient="vertical">.

The reason the spurious text appears on trunk is that the layout team have been simplifying the code incidentally removing many of the special cases that apply to XUL; in this particular case the case was that XUL used to ignore random text nodes.
(Assignee)

Comment 8

9 years ago
http://hg.mozilla.org/comm-central/rev/c550e59d094e
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b2
You need to log in before you can comment on or make changes to this bug.