Closed Bug 92635 Opened 23 years ago Closed 23 years ago

save password text is disappear for the password dialog for ONLY FIRST time Launch new profile.

Categories

(SeaMonkey :: MailNews: Message Display, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED WORKSFORME
mozilla0.9.6

People

(Reporter: huang, Assigned: jag+mozilla)

References

Details

(Keywords: regression)

Attachments

(3 files)

used 07-27-11-0.9.2 build

save password text is disappear for the password dialog

1) Login to an Mail account
2) Actual Results: Note the save password text is disappear for the password
dialog

Expected results: Should display "Use Password Manager to remember this
password" for the password dialog.
This is ONLY occurring for the FIRST time launch for the new profile....
Summary: save password text is disappear for the password dialog → save password text is disappear for the password dialog for the FIRST time Launch for new profile.
Summary: save password text is disappear for the password dialog for the FIRST time Launch for new profile. → save password text is disappear for the password dialog for ONLY FIRST time Launch new profile.
Adding pp for the keywords since problem is not occurring for the window 
platform
Keywords: pp
Removing pp since problem is also occurring on Windows.
And please note that this problem will occur after user remove mozilla directory
for the brand new profile.....
Keywords: pp
Nominating nsBranch since users won't feel good with the blank text for the 
login dialog as following attach screen shot.
Keywords: nsBranch
Keywords: regression
OS: Linux → All
Blocks: 99230
this seems to be working for me now. Karen can you retry? If it works now, let's
mark it works for me and get if off the nsBranch radar. Otherwise I think I need
to nsBranch+ this bug. 
I still seeing this problem on the current branch 09-12-05-0.9.4 build.
But it is occurring intermittently....sometimes time happen, sometimes not....
And there was bugscape 6020 
logged http://bugscape.netscape.com/show_bug.cgi?id=6020
No longer blocks: 99230
minusing for now. I just don't see this at all anymore. If you see it with high
regularity then we need to revist this. 
Keywords: nsbranchnsbranch-
I am still seeing this problem on Linux 09-17-05-0.9.4 build when creating new 
profile......
Branch build 2001-10-02-05: WinMe, still a problem in a new profile
Branch buid 2001-10-02-03: Mac 9.1, still a  problem in a new profile
Based on my testing, above Ninoschka comments & above Scott's comments:

>minusing for now. I just don't see this at all anymore. If you see it with high
>regularity then we need to revist this. 

I am wondering to know whether there is possibility for fixing this bug for 
nsbranch since users definitely will see this after they remove mozilla 
directory for the brand new profile.....
QA Contact: esther → olgam
Renominating nsBranch since this is so obviously on UI and without this 
fix, users definitely will see this.... 
Keywords: nsbranch-nsbranch
loks like a good one to take. what are the chances we can get this in before friday?
Whiteboard: [Need ETA] [PDT]
This morning I saw this problem for AOL account in existing profile with 
multiple accounts.
Actually jag's been working very hard trying to come up with a fix to common
dialogs.xul for us. I'm going to re-assign this to him since it isn't a mail
specific fix anyway. We'll keep an eye out and if he can cook something good up
for us, I'll take it to PDT.
Assignee: sspitzer → jaggernaut
Keywords: nsbranchnsbranch+
So after some tinkering I've found out that the XBL binding for the checkbox 
doesn't get attached until after onload is done, meaning that any code that 
depends on checkbox's methods and properties (e.g. .label, .checked) will fail. 
I think this is also the cause of the dialog not being tall enough since the 
check image (or the empty box, initially) won't be taken into account while the 
dialog is intrinsically sized.

I have an ugly work around. mscott, could you lobby for r=, sr= and a= while I 
dig deeper into why the xbl binding attach is delayed? I suspect the right fix 
to be more involved that what we'd take for 0.9.5.
Comment on attachment 52094 [details] [diff] [review]
Work around missing label and incorrect height.

sr=mscott

I'll find someone to put an r= in this work around. Thanks jag!
Attachment #52094 - Flags: superreview+
adding to tracker bug.
Blocks: 99508
Comment on attachment 52094 [details] [diff] [review]
Work around missing label and incorrect height.

r=sspitzer

make sure to test on all platforms, I remember bugs before where sizeToContent() did unexpected things (on linux, I think).
Attachment #52094 - Flags: review+
pls check this into the branch - PDT+, pending linux testing out ok.
Whiteboard: [Need ETA] [PDT] → [Need ETA] [PDT+] fix for 094 branch
Where "unexpected things" == "crash the product".

I've tested on Linux, it works fine there. Can someone test this on Mac for me?

The only disadvantage to this patch is that the resize is visible (one can see
it grow to the larger state) in this one case. Normally the dialog pops up at
the right size so that a resize can't be noticed.
well i guess I'd rather have the penalty of seeing the resize that first time
than having the dialog look the way it does now.

Jag, can you check this into the branch tonight? If you can't, I can check it in
for ya.
*** Bug 81442 has been marked as a duplicate of this bug. ***
When will this fix be in the trunk or the branch?
hey jag, I don't know if you saw my earlier question but can you check this into
the branch ASAP? If not, I can do it for ya! Thanks.
I checked it in last night (well, early this morning), forgot to mention that
here.
Checked it in on the branch 0.9.4, that is.
Whiteboard: [Need ETA] [PDT+] fix for 094 branch → [Need ETA] [PDT+] FIXED on 094 branch
Keywords: vbranch
nothing to see here. adding vbranch keyword since this is in the 094 branch.
Thanks for checking it in this morning jag.
Branch build 2001-10-05-05: WinMe, fixed

Branch build 2001-10-05-03: Mac 9.1, not fixed.
Branch build 2001-10-05-04: Linux RH 7.1, not fixed.
Did the fix make it into the Mac and Linux builds?
QA Contact: olgam → nbaca
Yes. After I remove mozilla directory for creating some new profiles for testing 
this bug:
I have the same results with Ninoschka:
This fixed on Windows, but not fix on Linux and Mac platforms yet....
(WinNT 10-05-05-0.9.4, Linux 10-05-04-0.9.4 & Mac 10-05-03-0.9.4 builds)
Hmmm... This is in cross platform code, I don't know why linux and mac didn't
pick it up. I tested it on all three platforms, saw the bug on all three, and
saw my fix work on all three.
Whiteboard: [Need ETA] [PDT+] FIXED on 094 branch → [PDT+] FIXED on 094 branch
Ninoschka: does this work on Mac and Linux now?
Branch build 2001-10-08-03: Mac 9.l, Mac X
Branch build 2001-10-08-04: Linux RH 7.1
It appears ok on these branch builds (in a new profile I created an IMAP
account, and in another new profile created a POP account).
I had to back out (on the 0.9.4 branch) the fix for the vertical sizing problem
since that caused other problems. The underlying problem is bug 103928, which is
hard to fix, and any work-around would have to be done in commonDialog.js, which
makes it easy to cause other highly visible problems (as shown by my previous
work-around). In short, I suggest we look at bug 103928 for the trunk, and keep
0.9.4 as it is.

The good thing about all this is that at least the checkbox label shows up now
and is correctly checked/unchecked.
Depends on: 103928
thanks for trying jag. Taking off the branch+ radar and the PDT radar since we
won't be taking a fix for 103928 on the branch.
Keywords: nsbranch+, vbranch
Whiteboard: [PDT+] FIXED on 094 branch
Target Milestone: --- → mozilla0.9.6
removing obsolete dependency.
No longer blocks: 99508
Marking nsbranch because the Password dialog still appears cutoff.

Branch build 2001-10-10-05: WinMe
In a new profile when Mail starts if the first dialog that appears asking if you
want Netscape6 to be the default, then the password dialog appears ok. If
Netscape6 is already the default then the first dialog to appear is the password
dialog, and the dialog appears cutoff but the text is present. 
Workaround: I can't resize the dialog but if I close and then Get Messages again
then the password dialog appears ok. 

Branch build 2001o-10-10: Mac 9.1, Linux RH 7.1
The password dialog is also cutoff on the mac and linux as well.

In all cases if the Webmail is activated in a new profile then the password
dialog is ok.
Keywords: nsbranch
hey ninoshcka, we had had our chance to fix this bug last week and it ended up
making things worse. While it looks ugly this is no longer in the stop ship
category and that's why I took it off the radar. Putting back the minus. Sorry =(
Keywords: nsbranchnsbranch-
i think we want to leave the nomination for MailNews team to + or -. Putting
back nsbranch, and marking [PDT-] for 094. Sorry for the spam, but I want to
preserve the work the teams have already done in their triage.
Keywords: nsbranch-nsbranch
Whiteboard: [PDT-]
doh! Sorry, missed marked a bug. Returning this to the pristine mscott marking
of nsbranch-.
Keywords: nsbranchnsbranch-
Whiteboard: [PDT-]
Blocks: 107067
Keywords: nsbranch-
It looks like this has been fixed with some related bug that hyatt fixed
recently.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WORKSFORME
Trunk build 2002-03-01: WinMe, Linux RH 7.1, Mac 9.1
Verified Fixed.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: