1518632 - Show users who get a fresh profile instead of reusing the previous default a first run UI

Status: RESOLVED FIXED

(Toolkit :: Startup and Profile System, enhancement)

Comment 1

[Dave Townsend [:mossop]]

Note that I attempted to implement the modal dialog using html but it turns out to be very difficult to get a dialog sized correctly using html. Something we should investigate and fix elsewhere.

I also attempted to use fluent for the localization of the dialog and even with XUL hit sizing issues. Because fluent runs after layout the dialog would size assuming it had no content. It was possible to resize after fluent ran but then you see the size change visibly.

Comment 2

[Dave Townsend [:mossop]]

Attachment: Bug 1518632: Create a flag indicating when the old-style default profile was ignored and a new profile created. r=Gijs

Comment 3

[Dave Townsend [:mossop]]

Attachment: Bug 1518632: Show the first-run experience for users that get pushed to a new profile. r=Gijs

On startup of a fresh dedicated profile show a welcome page and a modal dialog
to explain what has happened.

Comment 4

[Dave Townsend [:mossop]]

Michael, can you answer this question?

https://phabricator.services.mozilla.com/D16120#inline-88009

Comment 5

[Verdi [:verdi] Best to slack me]

(In reply to Dave Townsend [:mossop] (he/him) from comment #4)

Michael, can you answer this question?

https://phabricator.services.mozilla.com/D16120#inline-88009

Yes, Flod is correct. It should read:
To sync information you’ve already saved to Firefox with this installation of &brandShortName;, sign in with your &syncBrand.fxAccount.label;.

Comment 6

[Dave Townsend [:mossop]]

Attachment: Bug 1518632: Fix conflicts from bug 1277060.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/05200c5388b4
Show the first-run experience for users that get pushed to a new profile. r=Gijs, r=flod, r=stomlinson

Comment 8

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b4d8ca47f938
Backed out 7 changesets (bug 1518632, bug 1463198, bug 1455707, bug 1522934, bug 1322797, bug 1474285) for build bustages at /builds/worker/workspace/build/src/toolkit/xre/nsAppRunner.cpp on a CLOSED TREE

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a5ab8fa035ce
Show the first-run experience for users that get pushed to a new profile. r=Gijs, r=flod, r=stomlinson

Comment 10

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/a5ab8fa035ce

Comment 11

[Dave Townsend [:mossop]]

The first patch here was rolled into the patch for bug 1474285.