Last Comment Bug 763437 - Do you want to send messages now [no] [yes] uses no and yes
: Do you want to send messages now [no] [yes] uses no and yes
Status: RESOLVED FIXED
: polish
Product: Thunderbird
Classification: Client Software
Component: General (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 16.0
Assigned To: :aceman
:
:
Mentors:
Depends on:
Blocks: 720356
  Show dependency treegraph
 
Reported: 2012-06-11 04:41 PDT by Andreas Nilsson (:andreasn)
Modified: 2012-07-07 07:29 PDT (History)
7 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
screenshot of dialog (13.36 KB, image/png)
2012-06-11 04:41 PDT, Andreas Nilsson (:andreasn)
no flags Details
patch (12.00 KB, patch)
2012-06-12 11:48 PDT, :aceman
mconley: review+
bwinton: ui‑review+
Details | Diff | Splinter Review

Description Andreas Nilsson (:andreasn) 2012-06-11 04:41:16 PDT
Created attachment 631855 [details]
screenshot of dialog

Would be nice to hook up this dialog with proper verbs to enforce the actions.
What about: [Later] [Send Now]

(I also hate that dialog keeps blocking me and want it to be a notification bar, but that is bug #763436)
Comment 1 :aceman 2012-06-12 00:32:09 PDT
Do you want to do this or is it free to take? :)
Comment 2 Andreas Nilsson (:andreasn) 2012-06-12 00:46:54 PDT
Up for grabs!
Comment 3 :aceman 2012-06-12 00:49:59 PDT
Thanks.
Does Seamonkey have this too?
Comment 4 :aceman 2012-06-12 11:48:09 PDT
Created attachment 632355 [details] [diff] [review]
patch

Double pack for Andreas, I did the other dialog (download msgs) too.
Comment 5 Ian Neal 2012-06-12 15:15:39 PDT
(In reply to :aceman from comment #3)
> Thanks.
> Does Seamonkey have this too?

SeaMonkey has:
Would you like to send your unsent messages now?
with buttons "Send", "Don't Send" and "Cancel"
Comment 6 :aceman 2012-06-12 15:22:35 PDT
Thanks, then you do not need this fix. Sometimes you are ahead :-P
Comment 7 Blake Winton (:bwinton) (:☕️) 2012-06-21 07:54:14 PDT
Comment on attachment 632355 [details] [diff] [review]
patch

This is definitely better.  ui-r=me, based on that.

I think "Send now" then "Later" would be better still, since that follows the order in time of the two buttons.

And even better would be to just send the messages, and not interrupt the user.  ;)  (It looks like we could change the default value of "offline.send.unsent_messages" to achieve that…)

Thanks,
Blake.
Comment 8 Wayne Mery (:wsmwk, NI for questions) 2012-06-21 08:07:00 PDT
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #7)
> And even better would be to just send the messages, and not interrupt the
> user.  ;)  (It looks like we could change the default value of
> "offline.send.unsent_messages" to achieve that…)
> 
> Thanks,
> Blake.

indeed, Bug 511079 would be better
Comment 9 :aceman 2012-06-21 13:13:03 PDT
I don't think that one is related...
Comment 10 :aceman 2012-06-21 13:18:11 PDT
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #7)
> I think "Send now" then "Later" would be better still, since that follows
> the order in time of the two buttons.

I am not sure what you mean here. Should I change the order of the buttons? Or what?
Comment 11 Blake Winton (:bwinton) (:☕️) 2012-06-21 13:26:47 PDT
Yes, switching the order of the buttons would make the dialog make more sense…
Comment 12 :aceman 2012-06-21 13:36:24 PDT
I think the order also depends on platform, so you want the "Send now" be the first one (primary and default) and "Later" be the second. I think this will make Send on the left and Later on the right on Windows.
Comment 13 :aceman 2012-06-21 14:09:28 PDT
But I think I have the buttons in that order. See BUTTON_POS_0 and BUTTON_POS_1 in the patch. I see "Send now" on the right on Linux (which often has buttons reversed). So what do you see?
Comment 14 Blake Winton (:bwinton) (:☕️) 2012-06-21 14:16:08 PDT
I'll give you three guesses…  ;)

(I see "[Later] [Send now]" on the Mac.)

I think there are other kinds of dialogs that have a more consistent cross-platform button order…
Comment 15 :aceman 2012-06-21 14:43:56 PDT
Kinds of dialogs? Culdn't find a better one in http://mxr.mozilla.org/comm-central/source/mozilla/embedding/components/windowwatcher/public/nsIPromptService.idl that also has the checkbox option. The confirmEx comments say the implementation (platform?) can decide the order. Why would we go against it. Also, the new order is the same as it was before, only labels have changed.
Comment 16 Blake Winton (:bwinton) (:☕️) 2012-06-21 16:43:15 PDT
I also didn't see a better one.  :(

The confirmEx comments also say that if you're replacing all the buttons with custom strings, you might be doing something wrong.  Perhaps we should just leave them as they are.  :P

And finally, I'm not sure I can explain why "[Send now] [Later]" is better than "[Later] [Send now]" better than I did above…  If you agree that following the order in time is better, then surely it's better on all the platforms, no?
Comment 17 :aceman 2012-06-21 23:47:44 PDT
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #16)
> And finally, I'm not sure I can explain why "[Send now] [Later]" is better
> than "[Later] [Send now]" better than I did above…  If you agree that
> following the order in time is better, then surely it's better on all the
> platforms, no?

But I can't order it in the same way on all platforms (as some platforms reverse the order automatically). Maybe by ifdef'ing the code by platform. Do we want that?
Comment 18 Magnus Melin 2012-06-24 23:09:12 PDT
I don't think it's actually so much a question of when it time you'll do it. It's really more of "send now" vs "i don't wanna deal with this/stupid dialog go away/irrelevant text". So i'd just go for the platform order for defaults.
Comment 19 Blake Winton (:bwinton) (:☕️) 2012-06-26 12:45:14 PDT
Okay, fine, whatever.  I think it looks strange to have "Later" "Now", but if there's no way to actually put them in the right order, then let's just check this in and be done with it.
Comment 20 Mike Conley (:mconley) - (needinfo me!) 2012-07-06 07:36:24 PDT
Comment on attachment 632355 [details] [diff] [review]
patch

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

This looks good to me. Thanks aceman!
Comment 21 Ryan VanderMeulen [:RyanVM] 2012-07-07 07:29:18 PDT
https://hg.mozilla.org/comm-central/rev/1457e1545968

Note You need to log in before you can comment on or make changes to this bug.