The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in Thunderbird 3.1rc1

Status

Thunderbird
Migration
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: bwinton, Assigned: bwinton)

Tracking

Trunk
Thunderbird 3.1rc1
Dependency tree / graph

Firefox Tracking Flags

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

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
We couldn't get this working for 3.1 beta 1, but it should be doable.
(Assignee)

Comment 1

7 years ago
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

Updated

7 years ago
Blocks: 522631

Comment 3

7 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.
(Assignee)

Updated

7 years ago
Whiteboard: [ETA for review-ready patch: Friday, May 7]
(Assignee)

Comment 4

7 years ago
Created attachment 443605 [details] [diff] [review]
A patch to make the migration assistant modal on first launch.

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)
(Assignee)

Updated

7 years ago
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+
(Assignee)

Updated

7 years ago
Attachment #443605 - Flags: review?(bienvenu)
(Assignee)

Updated

7 years ago
Whiteboard: [patch up, waiting for ui-r] → [patch up, needs r]

Updated

7 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...
(Assignee)

Comment 7

7 years ago
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.
(Assignee)

Updated

7 years ago
Whiteboard: [patch up, needs r] → [patch up, needs landing]
(Assignee)

Comment 8

7 years ago
Checked in as:
http://hg.mozilla.org/comm-central/rev/c7d38819e838
and
http://hg.mozilla.org/releases/comm-1.9.2/rev/c6f8c6e66fb8
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
status-thunderbird3.1: --- → rc1-fixed
Resolution: --- → FIXED
Whiteboard: [patch up, needs landing]
You need to log in before you can comment on or make changes to this bug.