236140 - profile manager "conclusion" sheet doesn't fit - horizontal scrollbar

Status: RESOLVED FIXED

(SeaMonkey :: Startup & Profiles, defect)

Description

[Asa Dotzler [:asa]]

On Mac OS X, the profile manager gets a horizontal scrollbar on the "conclusion"
sheet. I suspect the aqua buttons for "choose folder..." "use default folder"
and "region selection..." are the reason for the wide sheet.

Comment 1

[Asa Dotzler [:asa]]

Tested with 2004-03-01 SeaMonkey build on OS X 10.3 (Jaguar). 

In case it wasn't clear, I'm talking about the final page of the "create
profile" wizard, accessible from the profile manager. 

Comment 2

[Stefan [:stefanh]]

I still see this on branch and trunk. This is something we should fix before the
release...

Comment 3

[Stefan [:stefanh]]

OK, so the width of the sheet is currently set to 45em in
createProfileWizard.dtd. Now, there are three buttons: "Choose Folder...", "Use
Default Folder" and "Region Selection..."

If we change "Region Selection..." to "Select Region..." and increase the width
with 1em to 46em the sheet fits for me (Mac OS 10.3.9, Classic skin).

Kairo, what do you think?

Comment 4

[Stefan [:stefanh]]

 
> If we change "Region Selection..." to "Select Region..." and increase the width
> with 1em to 46em the sheet fits for me (Mac OS 10.3.9, Classic skin).

Actually, we don't gain much by changing the button label. The sheet fits
without changing the label if the width is set to 47em :)

Comment 5

[Stefan [:stefanh]]

Attachment: One-liner, increase width with 2em (Checked into branch and trunk)

OK, this patch will increase the width of the sheet with 2 em.

Comment 6

[Stefan [:stefanh]]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

btw, tested this with 640x480, and the sheet fits.

Comment 7

[[e-mail]]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

Until someone implements width units in XUL I guess this will have to do :-/

Comment 8

[Stefan [:stefanh]]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

Low-risk, SeaMonkey mac ui polish.

Comment 9

[Mike Schroepfer]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

Approved for 1.8b5 per bug meeting

Comment 10

[Ian Neal]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

Checking in (trunk)
createProfileWizard.dtd; new revision: 1.11; previous revision: 1.10
done

Comment 11

[Ian Neal]

Comment on attachment 195537 [details] [diff] [review]
One-liner, increase width with 2em (Checked into branch and trunk)

Checking in (branch)
createProfileWizard.dtd; new revision: 1.10.58.1; previous revision: 1.10
done

Comment 12

[Stefan [:stefanh]]

--> Fixed

Comment 13

[Axel Hecht [:Pike]]

As this shows up in my bonsai queries, let me sign this off as non-l10n.
This is just a size change, and seamonkey only, if a localizer starts reading the 
bug bottom up. (Like I did, sorry for the bug spam)