Closed Bug 567280 Opened 14 years ago Closed 13 years ago

In Windows 7 with large font and default theme Manage Identities button is not visible

Categories

(Thunderbird :: Theme, defect)

x86
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 9.0

People

(Reporter: ob, Assigned: Paenglab)

Details

Attachments

(1 file, 2 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2) Gecko/20100115 Firefox/3.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.1.9) Gecko/20100317 Thunderbird/3.0.4

In Windows 7 with large font and default theme Manage Identities button is not visible

Reproducible: Always

Steps to Reproduce:
1. Set Windows 7 to Large Font
2. Open Account Settings Dialog
3. No Manage Identities!!!
Andreas which one of the UX-jargon keyword should I use for this bug ?
I think you mean ux-keyword, ux-jargon is one of the keywords [1] ;)
I don't think it's any of them (the whole list is here[2]), this is just a regular accessibility theme bug and related to bug 416263 among others.

1. ux-jargon is basically the same as geek-talk.
2. http://spreadsheets.google.com/pub?key=tJxF8zTuLdEj9pUcxnLAemA&output=html
True which one would you pick ? And correct me when I don't add the rights one on other bugs.
Mass move to the new theme component.
Component: General → Theme
QA Contact: general → theme
So you have got a name for that bug? 
And where is help? HELP!!!!
I cannot use TB at all, as my new Netbook with Win7 won't show the button for "other identities" in TB and I only send mails from those identities. So I bought the Netbook for nothing and have to use the old computer with TB 2.0 for the mails?
Any hints as to if and when the new TB will have the button for the identities again?
The button might become visible if you make the dialog bigger.

Andreas could you look into this one ?
Attached patch Add scrollbars when needed (obsolete) — Splinter Review
I inserted a vbox with style="overflow: auto". If the pref window is smaller than the content, the scrollbar is shown. Now it is only implemented for "Account Settings", "Server Settings" and "Copies & Folders". The other settings are small enough and IMO don't need scrollbars. But I can add it when desired for consistency.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #552099 - Flags: ui-review?(bwinton)
Attachment #552099 - Flags: review?(mbanner)
Alternative approach:
1. Add flex="1" to the <groupbox>
2. Remove flex="1" from the <grid>
3. Add flex="1" to the <hbox> immediately following the <grid>
(In reply to neil@parkwaycc.co.uk from comment #8)
> Alternative approach:
> 1. Add flex="1" to the <groupbox>
> 2. Remove flex="1" from the <grid>
> 3. Add flex="1" to the <hbox> immediately following the <grid>

Yes this works for the "Account Settings". But I don't see a possibility like this for "Server Settings" and "Copies & Folders". I'll wait for the ui-r if the scrollbars are okay.
Comment on attachment 552099 [details] [diff] [review]
Add scrollbars when needed

Yeah, I'm not a huge fan of scrollbars, but they seem like the least bad way to fix this.  :)

I would also like to see this for Composition & Addressing, Junk Settings, Synchronization & Storage, Return Receipts, and Security, so ui-r=me with those added.
Attachment #552099 - Flags: ui-review?(bwinton) → ui-review+
Attached patch Add scrollbars when needed v2 (obsolete) — Splinter Review
Patch with the additional pages asked in comment 10. Giving ui-r+ after Blake's comment.
Attachment #552099 - Attachment is obsolete: true
Attachment #555763 - Flags: ui-review+
Attachment #555763 - Flags: review?(mbanner)
Attachment #552099 - Flags: review?(mbanner)
Comment on attachment 555763 [details] [diff] [review]
Add scrollbars when needed v2

Ok, I'm just not getting to this, in part because I need to test on windows 7 and I've not got that set up.

My only comment is that you should probably be indenting the code inside the vboxes that you are adding to follow the general style elsewhere.
Attachment #555763 - Flags: review?(mbanner) → review?(bwinton)
The patch affects only XUL files. So this could also be tested under Linux or Mac.

I hesitated to indent the code because this would change almost every line of the files.
Comment on attachment 555763 [details] [diff] [review]
Add scrollbars when needed v2

Review of attachment 555763 [details] [diff] [review]:
-----------------------------------------------------------------

Any reason you didn't add a scrollbar to the Outgoing Server Settings pane?
And yeah, I agree with Standard8 that the lines should be indented, even though it'll cause the diff to be large.

r=me with those two things fixed.

Thanks,
Blake.
Attachment #555763 - Flags: review?(bwinton) → review+
Patch addressing the review comments:
- Added the scrollbar to the Outgoing Server Settings pane (am-smtp.xul)
- Indented the files.
Attachment #555763 - Attachment is obsolete: true
Attachment #561799 - Flags: ui-review+
Attachment #561799 - Flags: review+
Keywords: checkin-needed
http://hg.mozilla.org/comm-central/rev/6e41730f8874
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 9.0
This is still an issue because the very existence of the button is unknown to new users. The window defaults to no-scroll-bars with the button removed so, unless you happen to know that it can be resized, the button is simply missing. In Linux the issue is worse because the resizing is made very difficult by the window extending below the 600-line bottom of the screen, so it cannot be resized with the mouse at all.

Would it not be better to replace the button with an "Identities" section in the tree on the left?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: