[Firefox Alpenglow] Separators in the Bookmarks toolbar are almost invisible
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox83 | --- | unaffected |
firefox84 | --- | verified |
firefox85 | --- | verified |
People
(Reporter: obotisan, Assigned: Gijs)
References
(Regression)
Details
(Keywords: regression)
Attachments
(5 files)
Affected versions
- Firefox 84.0b4
- Firefox 85.0a1
Affected platforms
- Ubuntu 18.04 x64
- Windows 10 x64
- macOS 10.15
Preconditions
- Make sure that the Firefox Alpenglow is enabled.
Steps to reproduce
- Open Bookmarks Toolbar
- Add a few separators via context menu ( right-click on Bookmarks Toolbar and click on "New Separator").
Expected result
- The separator is visible.
Actual result
- Separator is almost invisible.
Regression range
- If it's a regression, I will try to find it asap.
Suggested severity
- S3
Reporter | ||
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Comment 1•3 years ago
|
||
- Last good: 2020-10-20 (20201020145547)
- First bad: 2020-10-21 (20201021095128)
- Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5b5ffbe4add97c98003026e0d9a730302161b646&tochange=d8861d51b01e9489672f998648d67662a60a8b3a
- Potential regressor: Bug 1665442
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
I'm on a data-limited mobile connection until next Tuesday, and I don't have a build handy that has both alpenglow and the pre-patch appearance. Can you take a screenshot of what the separators used to look like, ie what's the expected look of the separators in the light/dark alpenglow theme? And what does the hamburger menu separator look like in those cases?
Reporter | ||
Comment 3•3 years ago
|
||
Reporter | ||
Comment 4•3 years ago
|
||
Reporter | ||
Comment 5•3 years ago
•
|
||
The hamburger menu separators or any other menu on the url bar don't seem to be changed. They are the same on the latest version as they were on the older unaffected version.
Assignee | ||
Comment 6•3 years ago
|
||
(In reply to Oana Botisan, Desktop Release QA from comment #5)
The hamburger menu separators or any other menu on the url bar don't seem to be changed. They are the same on the latest version as they were on the older unaffected version.
Sure, but the new separators now look the same as the one next to the hamburger, which was always very very faint / invisible (also in the "unaffected" version).
I'll see if I can find any designs / ideas around what these should look like.
Reporter | ||
Comment 7•3 years ago
|
||
The separators from the hamburger menu are a bit faded, but they are distinguishable. The bookmark separator are invisible.
Assignee | ||
Comment 8•3 years ago
|
||
(In reply to Oana Botisan, Desktop Release QA from comment #7)
The separators from the hamburger menu are a bit faded, but they are distinguishable. The bookmark separator are invisible.
I don't mean the ones inside the menu, I mean the vertical line to the left of the hamburger button itself, which is just as invisible as the bookmarks ones...
Reporter | ||
Comment 9•3 years ago
|
||
I don't mean the ones inside the menu, I mean the vertical line to the left of the hamburger button itself, which is just as invisible as the bookmarks ones...
In that case, that one is affected too. I looked inside the menu bar. Sorry for the confusion.
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 11•3 years ago
|
||
Assignee | ||
Comment 12•3 years ago
|
||
Comment on attachment 9190894 [details]
Bug 1679136 - fix separator colour in alpenglow light and dark, r?jaws
Beta/Release Uplift Approval Request
- User impact if declined: Separators are invisible
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: See comment #0
- List of other uplifts needed: n/a
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Super low risk. Just changing some CSS variables for the alpenglow theme only, and incrementing the version number so this change takes effect.
- String changes made/needed: nope
Assignee | ||
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/568324188993 fix separator colour in alpenglow light and dark, r=jaws
Comment 14•3 years ago
|
||
bugherder |
Comment 15•3 years ago
|
||
Comment on attachment 9190894 [details]
Bug 1679136 - fix separator colour in alpenglow light and dark, r?jaws
Approved for 84.0b8.
Updated•3 years ago
|
Comment 16•3 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Reporter | ||
Comment 17•3 years ago
|
||
I verified the fix using Latest Nightly 85.0a1 and Firefox 84.0b8 on Windows 10 x64 (both with dark and light theme), Ubuntu 18.04 x64 and macOS 11.0.1. The issue is not reproducing anymore.
On Ubuntu and macOS even though the line is visible is still lighter than the ones that are on Windows. Is that intended? Or it's the difference on the settings from my monitors?
Assignee | ||
Comment 18•3 years ago
|
||
(In reply to Oana Botisan, Desktop Release QA from comment #17)
I verified the fix using Latest Nightly 85.0a1 and Firefox 84.0b8 on Windows 10 x64 (both with dark and light theme), Ubuntu 18.04 x64 and macOS 11.0.1. The issue is not reproducing anymore.
Thanks for the quick verification.
On Ubuntu and macOS even though the line is visible is still lighter than the ones that are on Windows. Is that intended? Or it's the difference on the settings from my monitors?
The exact colour will depend on the background - it's semi-transparent. The semi-transparent colour specified is the same cross-platform, so for separators in exactly the same place there shouldn't be a difference across platforms. Happy to take a look at a specific screenshot if you think there's still something off compared to Windows?
Reporter | ||
Comment 19•3 years ago
|
||
I was working on the image and just realized they are exactly the same colour. I think the difference I saw was because of my monitor settings (my main monitor has really low brightness). Sorry for the confusion.
I will mark this issue as verified fixed.
Updated•3 years ago
|
Description
•