Missing space between Finish your account and Extension requires new Permissions notifications
Categories
(Firefox :: Menus, defect, P2)
Tracking
()
People
(Reporter: rdoghi, Assigned: mhowell)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [proton-icons] [proton-appmenu-notifications] [priority:2b] [proton-uplift])
Attachments
(3 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
(deleted),
image/png
|
Details |
[Affected platforms]:
Platforms: All
[Steps to reproduce]
-
Launch the Firefox browser and install a few older addons like :
Install version 1.35.2 of Bitwarden – Free Password manager - https://addons.mozilla.org/en-US/firefox/addon/bitwarden-password-manager/versions/?utm_content=search&utm_medium=referral&utm_source=addons.mozilla.org
install version 7.0.0 of New Tab Override - https://addons.mozilla.org/en-US/firefox/addon/new-tab-override/versions/?utm_content=recommended_fallback&utm_medium=referral&utm_source=addons.mozilla.org -
Set the extensions.update.interval - 10
-
Restart the Browser and wait for the Menu notifications to be displayed
-
Create a new account in order for the Finish Account setup notification to be displayed.
[Expected result]
There should a 4px space between the Extensions require new Permissions and Finish your account setup notification banners
[Actual result]
The Finish you account setup/ Youve Been Disconnected notifications are stretching to the Extensions require new permissions notifications.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Turns out that some changes to how the margin on the accounts menu item works
have allowed us to fix this bug and also simplify how the bottom margin on
menu notifications works at the same time; now, just having margin below every
banner all the time means that we get the correct spacing between multiple
banners and also between the last banner and the accounts item.
Reporter | ||
Comment 2•4 years ago
|
||
There seems to be some space missing between the banners and the Sign in button as well.
Comment 5•4 years ago
|
||
bugherder |
Reporter | ||
Comment 6•4 years ago
|
||
This issue is Verified as fixed in our latest Nightly 90.0a1 (2021-05-06) on Mac, Windows and Ubuntu.
Comment 7•4 years ago
|
||
The patch landed in nightly and beta is affected.
:mhowell, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 8•4 years ago
|
||
Comment on attachment 9220485 [details]
Bug 1707203 - Always unconditionally leave margin below app menu notifications. r=#desktop-theme-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: MR1 regression causing a cosmetic defect in app menu notification banners, where there's no spacing between them and the accounts menu item.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This is just a CSS fix to the margin on the menu notifications.
- String changes made/needed:
Comment 9•4 years ago
|
||
The uplift needs PM approval for MR1
Updated•4 years ago
|
Comment 11•4 years ago
|
||
bugherder uplift |
Comment 12•4 years ago
|
||
Comment on attachment 9220485 [details]
Bug 1707203 - Always unconditionally leave margin below app menu notifications. r=#desktop-theme-reviewers
Approved for 89 beta 11, thanks.
Reporter | ||
Comment 13•4 years ago
|
||
This issue is Verified as Fixed in our latest Beta 89.0b11 on Windows Mac and Ubuntu.
Updated•3 years ago
|
Description
•