Bug 1847276 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Treatment-a designs (expanded) performed better with imports in a recent [experiment](https://experimenter.services.mozilla.com/nimbus/embedded-import-wizard/summary). Treatment-b (non expanded) is currently the default experience as of 116 - we'd like to update to the one that performed better with our users.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: - Route to `about:welcome`
- Route to import screen (on easy setup flow select the "import" checkbox and continue)
- Notice the lack of expanded menu option (no `+` button, all available import options will be visible by default)
- Resize to <800px and confirm layout renders all available options without overflow
- Route to about:preferences and search "import" select `Import Data`, notice variant 2 designs (non-expanded, has `+` button, import options not visible by default)


(Steps can also be found in the test plan in the Phabricator patch)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Both layouts have been QA'd in Fx116 and in the experiment which included the layouts as different treatment branches. The layout change itself is minor (expanded vs non-expanded) and is covered by tests.
* **String changes made/needed**: N/A
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: Treatment-a designs (expanded) performed better with imports in a recent [experiment](https://experimenter.services.mozilla.com/nimbus/embedded-import-wizard/summary). Treatment-b (non expanded) is currently the default experience as of 116 - we'd like to update to the one that performed better with our users.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: 
- Route to `about:welcome`
- Route to import screen (on easy setup flow select the "import" checkbox and continue)
- Notice the lack of expanded menu option (no `+` button, all available import options will be visible by default)
- Resize to <800px and confirm layout renders all available options without overflow
- Route to about:preferences and search "import" select `Import Data`, notice variant 2 designs (non-expanded, has `+` button, import options not visible by default)


(Steps can also be found in the test plan in the Phabricator patch)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Both layouts have been QA'd in Fx116 and in the experiment which included the layouts as different treatment branches. The layout change itself is minor (expanded vs non-expanded) and is covered by tests.
* **String changes made/needed**: N/A
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: Treatment-a designs (expanded) performed better with imports in a recent [experiment](https://experimenter.services.mozilla.com/nimbus/embedded-import-wizard/summary). Treatment-b (non expanded) is currently the default experience as of 116 - we'd like to update to the layout that performed better with our users.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: 
- Route to `about:welcome`
- Route to import screen (on easy setup flow select the "import" checkbox and continue)
- Notice the lack of expanded menu option (no `+` button, all available import options will be visible by default)
- Resize to <800px and confirm layout renders all available options without overflow
- Route to about:preferences and search "import" select `Import Data`, notice variant 2 designs (non-expanded, has `+` button, import options not visible by default)


(Steps can also be found in the test plan in the Phabricator patch)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Both layouts have been QA'd in Fx116 and in the experiment which included the layouts as different treatment branches. The layout change itself is minor (expanded vs non-expanded) and is covered by tests.
* **String changes made/needed**: N/A
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: Treatment-a designs (expanded) performed better with imports in a recent [experiment](https://experimenter.services.mozilla.com/nimbus/embedded-import-wizard/summary). Treatment-b (non expanded) is currently the default experience as of 116 - we'd like to update to the layout that performed better with our users.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: 
- Route to `about:welcome`
- Route to import screen (on easy setup flow select the "import" checkbox and continue)
- Notice the lack of expanded menu option (no `+` button, all available import options will be visible by default)
- Resize to <800px and confirm layout renders all available options without overflow
- Route to about:preferences and search "import" select `Import Data`, notice variant 2 designs (non-expanded, has `+` button, import options not visible by default)


(Steps can also be found in the test plan in the Phabricator patch)
* **List of other uplifts needed**: [1847815](https://bugzilla.mozilla.org/show_bug.cgi?id=1847815) which is required for this patch as it decouples the pref used to change the layout for about:welcome 
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Both layouts have been QA'd in Fx116 and in the experiment which included the layouts as different treatment branches. The layout change itself is minor (expanded vs non-expanded) and is covered by tests.
* **String changes made/needed**: N/A
* **Is Android affected?**: No

Back to Bug 1847276 Comment 2