1211861 - Dock Options subdialog too narrow

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Comment 1

[Javi Rueda :javirid]

STR:

1. Set mail.preferences.inContent to true
2. Navigate to General
3. Click App Icon Options... button

What happens:

The subdialog shows as in the screenshoot attached.

What was expected:

Subdialog shoud show completelly. Even buttons are cut off on the right side.

Comment 2

[Javi Rueda :javirid]

Attachment: Subdialog

Comment 3

[Richard Marti (:Paenglab)]

Attachment: 1211861.patch

The problem is, this dialog uses for the info text a caption to make it bold. But captions don't wrap.

My patch is using now a description with class bold to make the info bold.

Comment 4

[Javi Rueda :javirid]

Comment on attachment 8670910 [details] [diff] [review]
1211861.patch

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

Your patch fixed the problem, Richard, although I think "bold" makes the caption stand out too much.

Comment 5

[Javi Rueda :javirid]

Forget it. Original caption was also bold. So it is fine right now with your patch. Thank you Richard.

Comment 6

[aleth [:aleth]]

Comment on attachment 8670910 [details] [diff] [review]
1211861.patch

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

Fixes this particular bug, so r+.

There are still some issues with this dialog, if one wanted to be perfectionist:

* There is a blue box around the close button in the top right. This goes away when you press TAB.

* The spacing between the lines of text looks bad. E.g. the space between the "App icon badge" caption and the next line is less than the intra-line spacing in "You can disable the badge...". Some extra whitespace around the caption (above and below) should fix this.

Comment 7

[Richard Marti (:Paenglab)]

Attachment: 1211861.patch v2

(In reply to aleth [:aleth] from comment #6)
> Comment on attachment 8670910 [details] [diff] [review]
> 1211861.patch
> 
> Review of attachment 8670910 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Fixes this particular bug, so r+.
> 
> There are still some issues with this dialog, if one wanted to be
> perfectionist:

No need to be perfectionist, it should only look good. ;)

> * There is a blue box around the close button in the top right. This goes
> away when you press TAB.

I moved the script before the prefpane and here the focus in no more shown. Also for you?

> * The spacing between the lines of text looks bad. E.g. the space between
> the "App icon badge" caption and the next line is less than the intra-line
> spacing in "You can disable the badge...". Some extra whitespace around the
> caption (above and below) should fix this.

I added some spacing, do you like it more?

Comment 8

[aleth [:aleth]]

Comment on attachment 8670962 [details] [diff] [review]
1211861.patch v2

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

(In reply to Richard Marti (:Paenglab) from comment #7)
> > * There is a blue box around the close button in the top right. This goes
> > away when you press TAB.
> 
> I moved the script before the prefpane and here the focus in no more shown.
> Also for you?

Yes, it's gone now!

However, the real bug here is that the options in the window are not keyboard-accessible (i.e. if you keep pressing TAB you never get to the checkbox etc). That's for a separate bug though (and maybe m-c?).

> > * The spacing between the lines of text looks bad. E.g. the space between
> > the "App icon badge" caption and the next line is less than the intra-line
> > spacing in "You can disable the badge...". Some extra whitespace around the
> > caption (above and below) should fix this.
> 
> I added some spacing, do you like it more?

That's better, thanks!

Comment 9

[Richard Marti (:Paenglab)]

(In reply to aleth [:aleth] from comment #8)
> 
> However, the real bug here is that the options in the window are not
> keyboard-accessible (i.e. if you keep pressing TAB you never get to the
> checkbox etc). That's for a separate bug though (and maybe m-c?).

On Windows this works (okay, not this dialog as it exists only on OS X). If you want access the options with the access-keys you need to press on Windows additionally SHIFT to the ALT- combinations.

Comment 10

[aleth [:aleth]]

(In reply to Richard Marti (:Paenglab) from comment #9)
> (In reply to aleth [:aleth] from comment #8)
> > 
> > However, the real bug here is that the options in the window are not
> > keyboard-accessible (i.e. if you keep pressing TAB you never get to the
> > checkbox etc). That's for a separate bug though (and maybe m-c?).
> 
> On Windows this works (okay, not this dialog as it exists only on OS X). If
> you want access the options with the access-keys you need to press on
> Windows additionally SHIFT to the ALT- combinations.

What access keys?

My expectation would be that (as with any other dialog) TAB cycles through the dialog elements. Instead, I can only select the close button or ?? (the background?). Shift-TAB cycles in the opposite direction as expected.

Ctrl-TAB switches between tabs.
Option-TAB and Option-Shift-TAB doesn't appear to do anything.

Comment 11

[aleth [:aleth]]

(In reply to aleth [:aleth] from comment #10)
> My expectation would be that (as with any other dialog) TAB cycles through
> the dialog elements. Instead, I can only select the close button or ?? (the
> background?). Shift-TAB cycles in the opposite direction as expected.

This doesn't work properly for the main in-content prefs either - only the textbox elements are reachable.
I've just checked and on Firefox Aurora I see the same behaviour.
That's pretty broken accessibility, isn't it? :-/

Comment 12

[Richard Marti (:Paenglab)]

Yeah, OS X, or TB and FX on OS X, looks not so well accessible. I also can't access the menulists in Write window with TAB. Or I haven't found where to enable such things in OS X. Any hints?

Comment 13

[aleth [:aleth]]

(In reply to Richard Marti (:Paenglab) from comment #12)
> My expectation would be that (as with any other dialog) TAB cycles through
> the dialog elements. Instead, I can only select the close button or ?? (the
> background?). Shift-TAB cycles in the opposite direction as expected.

After some investigating, it turns out that this behaviour is normal on OSX unless you turn on Full Keyboard Access in Preferences->Keyboard->Shortcuts. Sorry for the false alarm!

Comment 14

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/d357afc6e70bec7e9e5ef80c81f6f63be983c2f4
Bug 1211861 - Let the info text wrap in Dock option. r=aleth

Comment 15

[Richard Marti (:Paenglab)]

(In reply to aleth [:aleth] from comment #13)
> After some investigating, it turns out that this behaviour is normal on OSX
> unless you turn on Full Keyboard Access in Preferences->Keyboard->Shortcuts.
> Sorry for the false alarm!

Ah yes, that's it. Thank you for the tip.