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)
Thunderbird
Migration
Tracking
(blocking-thunderbird3.1 rc1+, thunderbird3.1 rc1-fixed)
RESOLVED
FIXED
Thunderbird 3.1rc1
People
(Reporter: bwinton, Assigned: bwinton)
References
Details
Attachments
(1 file)
4.07 KB,
patch
|
Bienvenu
:
review+
clarkbw
:
ui-review+
|
Details | Diff | Splinter Review |
We couldn't get this working for 3.1 beta 1, but it should be doable.
Assignee | ||
Comment 1•15 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: --- → ?
Comment 2•15 years ago
|
||
Ick, this hurts the migration assistant functionality a bunch, and needs to block, I think.
blocking-thunderbird3.1: ? → rc1+
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•15 years ago
|
Whiteboard: [ETA for review-ready patch: Friday, May 7]
Assignee | ||
Comment 4•15 years ago
|
||
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•15 years ago
|
Whiteboard: [ETA for review-ready patch: Friday, May 7] → [patch up, waiting for ui-r]
Comment 5•15 years ago
|
||
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•15 years ago
|
Attachment #443605 -
Flags: review?(bienvenu)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [patch up, waiting for ui-r] → [patch up, needs r]
Updated•15 years ago
|
Attachment #443605 -
Flags: review?(bienvenu) → review+
Comment 6•15 years ago
|
||
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•15 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•15 years ago
|
Whiteboard: [patch up, needs r] → [patch up, needs landing]
Assignee | ||
Comment 8•15 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
Closed: 15 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.
Description
•