Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Autoconfig wizard: Fix styling and order of buttons on Windows

RESOLVED FIXED in Thunderbird 8.0

Status

Thunderbird
Account Manager
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: sid0, Assigned: sid0)

Tracking

unspecified
Thunderbird 8.0
x86
Windows 7

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

Created attachment 541969 [details] [diff] [review]
patch v1

The Windows platform standard is to have cancel as the rightmost button. The patch uses CSS box ordinals to fix this. (I've also cleaned up a bit of unused CSS code.)

Also, different sizes for button text is pretty non-standard. As a replacement, bwinton suggested bolding the important buttons, so I've done that.

One thing I'm concerned about is third-party themes -- all of them would also have to include the same CSS. I'm not sure if we have a way to not do this, or if this is worth worrying about at all.

Blake, is it fine if you do both the code and the UI reviews?

(note: I spent a whole day trying to use a XUL dialog to get the correct order, but it's just broken for anything non-trivial)
Attachment #541969 - Flags: ui-review?(bwinton)
Attachment #541969 - Flags: review?(bwinton)
(Assignee)

Comment 1

6 years ago
Created attachment 541971 [details]
Screenshot (start state)
(Assignee)

Comment 2

6 years ago
Created attachment 541972 [details]
Screenshot (result state)
(Assignee)

Comment 3

6 years ago
Created attachment 541973 [details]
Screenshot (manual edit)

Comment 4

6 years ago
This dialog is imitating a wizard, with Next and Back buttons. That's why the Next button is on the very right. This is intentional and approved like that.
Also, the Cancel button should be next to the Back button.

Suggest WONTFIX.

Updated

6 years ago
Attachment #541969 - Flags: review-

Comment 5

6 years ago
I like the change to remove the different button sizes and instead make it bold.
(Assignee)

Comment 6

6 years ago
The dialog is clearly not a wizard -- wizards have separate pages and a (mostly) linear ordering, while our dialog shows all the information at once and has branching state transitions. While it's great (and probably better) that on Mac and Linux the two coincide, on Windows that simply isn't the case.

Your point about the order of the Create button (I assume you meant that instead of cancel) is well taken -- I myself wasn't sure. But in that case, I think the order should 

Anyway, I'm going to make Blake make the call here.
(Assignee)

Comment 7

6 years ago
> But in that case, I think the order should

... be stop|re-test, continue|create, cancel. That's a very simple fix though.
(Assignee)

Comment 8

6 years ago
Heh, turns out that on Windows, wizards have their cancel buttons to the right too, e.g. http://dotnet.tech.ubc.ca/CourseWiki/images/f/f6/Windows_7_ClearType_Tuner.gif
Comment on attachment 541969 [details] [diff] [review]
patch v1

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

"stop|re-test, continue|create, cancel" seems like the obvious analogue to "back, forward, cancel", so r=me and ui-r=me, with the nit below fixed, and the question below answered.

::: mail/themes/pinstripe/mail/accountCreation.css
@@ -10,5 @@
> -  text-decoration: underline;
> -  background-color: inherit;
> -  cursor: pointer;
> -}
> -

I would like to see a commented out definition for ".important-button" here, so that we can compare the style files across platforms.

::: mail/themes/qute/mail/accountCreation.css
@@ +5,3 @@
>  }
>  
> +/* Set the order of the buttons to: (continue|create account), (stop|re-test), cancel */

Does that do the right thing in RTL languages?
Attachment #541969 - Flags: ui-review?(bwinton)
Attachment #541969 - Flags: ui-review+
Attachment #541969 - Flags: review?(bwinton)
Attachment #541969 - Flags: review+
(In reply to comment #9)
> Does that do the right thing in RTL languages?

Yes, the order of the buttons correctly gets reversed to cancel, create account, re-test on the left and manual/advanced config on the right. I tested by adding a CSS rule for window { direction: rtl; } .
https://hg.mozilla.org/comm-central/rev/45a849aee616
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Target Milestone: --- → Thunderbird 8.0
Comment on attachment 541969 [details] [diff] [review]
patch v1

This is a polish fix -- wonder if it'd make sense to land it on Aurora.
Attachment #541969 - Flags: approval-comm-aurora?
... though I guess there is a non-zero l10n impact in that we're changing the size of some text.
Comment on attachment 541969 [details] [diff] [review]
patch v1

I think we should leave it to 8 and l10n can have a chance to check it there when they do their normal work.
Attachment #541969 - Flags: approval-comm-aurora? → approval-comm-aurora-
You need to log in before you can comment on or make changes to this bug.