Closed
Bug 1366573
Opened 7 years ago
Closed 7 years ago
Page action button looks almost vanished on mousehover
Categories
(Firefox :: Theme, defect, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox53 | --- | unaffected |
| firefox54 | --- | unaffected |
| firefox55 | --- | verified |
People
(Reporter: euthanasia_waltz, Assigned: Gijs)
References
Details
(Whiteboard: [photon-structure])
Attachments
(2 files)
screenshot taken on LinuxMint 18.1
Page action button becomes almost invisible when mouse hover on the button in Compact Dark theme.
It's fine on Win 7.
status-firefox53:
--- → unaffected
status-firefox54:
--- → unaffected
status-firefox55:
--- → affected
tracking-firefox55:
--- → ?
|
Assignee | |
Comment 2•7 years ago
|
||
We won't ship the Photon changes for 55 so this doesn't need to track. Dão, you run Linux, can you reproduce this? Any idea what's going on? I'm a bit lost - aren't we using an svg with context-fill, which should still be white in the dark theme? Or did that change somehow?
tracking-firefox55:
? → ---
Flags: needinfo?(dao+bmo)
|
||
Comment 3•7 years ago
|
||
(In reply to :Gijs from comment #2) > We won't ship the Photon changes for 55 so this doesn't need to track. > > Dão, you run Linux, can you reproduce this? Yep. > Any idea what's going on? I'm a > bit lost - aren't we using an svg with context-fill, which should still be > white in the dark theme? Or did that change somehow? Looks like this is changing the color on hover: http://searchfox.org/mozilla-central/rev/2933592c4a01b634ab53315ce2d0e43fccb82181/toolkit/themes/linux/global/toolbarbutton.css#40-42 You'll probably need to explicitly set the color to inherit.
Flags: needinfo?(dao+bmo)
|
|
||
Updated•7 years ago
|
Whiteboard: [photon-structure][triage]
|
||
Updated•7 years ago
|
Flags: qe-verify?
Priority: -- → P2
Whiteboard: [photon-structure][triage] → [photon-structure]
| Comment hidden (mozreview-request) |
|
|
||
Updated•7 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: UNCONFIRMED → ASSIGNED
Iteration: --- → 55.7 - Jun 12
Ever confirmed: true
Priority: P2 → P1
|
|
||
Comment 5•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8875727 [details] Bug 1366573 - force foreground color inheritance on page action button, https://reviewboard.mozilla.org/r/147150/#review151322
Attachment #8875727 -
Flags: review?(dao+bmo) → review+
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/3952f89f9b1d force foreground color inheritance on page action button, r=dao
|
||
Comment 7•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/3952f89f9b1d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
|
||
Comment 8•7 years ago
|
||
I have reproduced this bug with Nightly 55.0a1 (2017-05-21) in Deepin OS (64-bit). This bug's fix is verified with latest Nightly 55.0a1 (64-bit). Build ID : 20170611100318 User Agent : Mozilla/5.0 (X11; Linux x86_64; rv:55.0) Gecko/20100101 Firefox/55.0 [bugday-20170607]
|
|
Assignee | |
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
|
|
||
Updated•7 years ago
|
Flags: qe-verify?
You need to log in
before you can comment on or make changes to this bug.
Description
•