Closed Bug 560742 Opened 15 years ago Closed 15 years ago

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

Categories

(Thunderbird :: Migration, defect)

defect
Not set
normal

Tracking

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

RESOLVED FIXED
Thunderbird 3.1rc1
Tracking Status
blocking-thunderbird3.1 --- rc1+
thunderbird3.1 --- rc1-fixed

People

(Reporter: bwinton, Assigned: bwinton)

References

Details

Attachments

(1 file)

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.) Later, Blake.
blocking-thunderbird3.1: --- → ?
Ick, this hurts the migration assistant functionality a bunch, and needs to block, I think.
blocking-thunderbird3.1: ? → rc1+
Blocks: 563290
Blocks: 522631
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.) Thanks, Blake.
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]
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.) Later, Blake.
Whiteboard: [patch up, needs r] → [patch up, needs landing]
Status: ASSIGNED → RESOLVED
Closed: 15 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.

Attachment

General

Created:
Updated:
Size: