Make the Migration Assistant into a modal dialog on first launch.

RESOLVED FIXED in Thunderbird 3.1rc1


9 years ago
9 years ago


(Reporter: bwinton, Assigned: bwinton)


Thunderbird 3.1rc1
Dependency tree / graph

Firefox Tracking Flags

(blocking-thunderbird3.1 rc1+, thunderbird3.1 rc1-fixed)



(1 attachment)

We couldn't get this working for 3.1 beta 1, but it should be doable.
The migration assistant disappears behind the 3pane on first launch, because the system integration dialog pops up, pulling the 3pane to the front.  And on OS X, there's no indication that the migration assistant is even there.

(This would also fix bug 560745, because you couldn't close the 3pane if this modal window was on top of it.)

blocking-thunderbird3.1: --- → ?
Ick, this hurts the migration assistant functionality a bunch, and needs to block, I think.
blocking-thunderbird3.1: ? → rc1+


9 years ago
Blocks: 522631

Comment 3

9 years ago
That migration dialog would need to appear before any password requests are presented to the user (in fact, before any login takes place). See dependent
bug 522631 and bug 563290.
Whiteboard: [ETA for review-ready patch: Friday, May 7]
Hey Bryan,

I think this is probably a reasonable place to ask for a UI-review.  I'm setting an attribute on the buttons to indicate whether we're on the first run (and therefore modal) or not, and then switching on that in the CSS.  (We might want to rename the attribute to "isUpgrade", or something, as well.  Your call.)

One thing that's different is that we can't resize the modal dialog anymore, but I'm not sure how important you feel that is.

As always, lemme know if you want a try-server build to play with.

(Oh, and I've only tested it on Mac.)

Attachment #443605 - Flags: ui-review?(clarkbw)
Whiteboard: [ETA for review-ready patch: Friday, May 7] → [patch up, waiting for ui-r]
Comment on attachment 443605 [details] [diff] [review]
A patch to make the migration assistant modal on first launch.

I've only tested this on the Mac so far but it feels good.
Attachment #443605 - Flags: ui-review?(clarkbw) → ui-review+
Attachment #443605 - Flags: review?(bienvenu)
Whiteboard: [patch up, waiting for ui-r] → [patch up, needs r]


9 years ago
Attachment #443605 - Flags: review?(bienvenu) → review+
Did anyone verify what happens if a password dialog (e.g. master or normal) is displayed when starting up?

Especially on Mac as that can cause hangs...
I just tested that, and the password dialog doesn't get displayed until after the migration assistant closes.  (The order of dialogs I see in my tests on Mac is: Migration Assistant » Default Client » Mail server password.)

Whiteboard: [patch up, needs r] → [patch up, needs landing]
Checked in as:
Last Resolved: 9 years ago
Resolution: --- → FIXED
Whiteboard: [patch up, needs landing]
You need to log in before you can comment on or make changes to this bug.