486955 - Identity Settings dialog has shrunk

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[u60234]

Attachment: screenshot

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4pre) Gecko/20090405 Shredder/3.0b3pre

The Identity Settings dialog has become too small for its content.

Steps to reproduce:
1. Open Account Settings and select an account.
2. Click on "Manage Identities..." button.
3. Select an identity and click "Edit" to open the Identity Settings dialog.
4. Select the "Composition & Addressing" tab.

Result:
The dropdown for where to place my signature is cut off.

This is a regression between BuildID=20090319044410 and BuildID=20090320034500. Backing out bug 390734 fixes this bug.

Comment 1

[Mark Banner (:standard8)]

I've just taken a look at this. The identity settings dialog currently has a hard-coded with of 40em:

http://mxr.mozilla.org/comm-central/source/mailnews/base/prefs/content/am-identities-list.xul#44

Probably the best way to fix this is in a similar manner to the preferences dialog and using localised style values per-platform:

http://mxr.mozilla.org/comm-central/source/mail/components/preferences/preferences.xul#69

then localisations can alter the width if necessary as well and we can tweak it per platform (i.e. it looks just fine on Mac with 40em).

Any takers? (TB 3b4 blocking).

Comment 2

[[e-mail]]

(In reply to comment #1)
> (i.e. it looks just fine on Mac with 40em).
I thought Mac used bold fonts by default, which tends to need a wider dialog.

It is possible to do without the width style if you add width="1" to all <description>&...;</description> tags. The dialog will then size to fit.
(Note: don't forget S/MIME!) This may result in a large dialog if the user has long account or smtp server names.

Comment 3

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

Interestingly, on linux is actually 40em, although it's set to be 35em - so it adjusts automatically for the would-be overflow.

Given sm is already string frozen, maybe we'll just make this hardcoded 40em too - i don't remember localizers complaining earlier. I'd also hope all platforms can use the same value here, if we need to i don't think it would hurt much to add a few ems in width.

I recalled the identity edit pops up at 0,0 on windows, so i fixed that too.

If we want to use ch instead, 40em was 67ch for me.

Comment 4

[Mark Banner (:standard8)]

Comment on attachment 397886 [details] [diff] [review]
proposed fix

Unforunately this has the side effect of making the identity dialog resizable on mac.

The SeaMonkey string freeze ends today and starts again on the 6th Oct. I've not seen evidence of locales wanting to change the width of the dialog but that doesn't mean they haven't realised they could ask to change it ;-)

Comment 5

[Vlado Valastiak [:wladow]]

Attachment: another screenshot

(In reply to comment #4)
The bug reporter is a Swedish localizer ;)
Adding another screenshot showing how badly we need to get this fixed.

Mozilla/5.0 (Windows; U; Windows NT 6.0; sk; rv:1.9.1.4pre) Gecko/20090903 Shredder/3.0b4pre

Comment 6

[[e-mail]]

Comment on attachment 397886 [details] [diff] [review]
proposed fix

I like 67ch :-)

>+  window.openDialog("am-identity-edit.xul", "", "chrome,modal,centerscreen", args);
Sadly at least in Suite the identity list dialog itself still opens at 0, 0 :-(

Comment 7

[Magnus Melin [:mkmelin]]

(In reply to comment #4)
> (From update of attachment 397886 [details] [diff] [review])
> Unforunately this has the side effect of making the identity dialog resizable
> on mac.

I'm confused. In bug 507392 comment 17 you said that on mac, titlebar makes dialogs resizable. Here i'm *removing* the (as i figured redundant) titlebar param, and that makes it resizable?

Comment 8

[[e-mail]]

(In reply to comment #7)
> (In reply to comment #4)
> > (From update of attachment 397886 [details] [diff] [review])
> > Unforunately this has the side effect of making the identity dialog resizable
> > on mac.
> I'm confused. In bug 507392 comment 17 you said that on mac, titlebar makes
> dialogs resizable. Here i'm *removing* the (as i figured redundant) titlebar
> param, and that makes it resizable?
He actually said "titlebar" and "resizable" makes dialogs NOT resizable. As per my bug 507392 comment #22, this isn't quite true; "resizable" and "resizable=yes" don't work, while "titlebar" and "resizable=no" do.

Comment 9

[Magnus Melin [:mkmelin]]

Ah yes. Just confused then;)

Comment 10

[Magnus Melin [:mkmelin]]

Attachment: proposed fix, v2

Carrying fwd sr=neil
(The identity-list it self should now also be centerscreened.)

Comment 11

[Mark Banner (:standard8)]

Comment on attachment 398680 [details] [diff] [review]
proposed fix, v2

That's better, thanks.

r=Standard8

Comment 12

[Magnus Melin [:mkmelin]]

changeset:   3546:37532e529fbe
http://hg.mozilla.org/comm-central/rev/37532e529fbe

->FIXED