Closed
Bug 1375893
Opened 6 years ago
Closed 6 years ago
Update pinned-tab highlight/glow style for Photon
Categories
(Firefox :: Theme, defect, P1)
Firefox
Theme
Tracking
()
Tracking | Status | |
---|---|---|
firefox56 | --- | verified disabled |
firefox57 | --- | verified |
People
(Reporter: Dolske, Assigned: Dolske)
References
(Blocks 1 open bug)
Details
(Whiteboard: [photon-visual][p2])
Attachments
(2 files, 1 obsolete file)
3.88 KB,
patch
|
Details | Diff | Splinter Review | |
9.39 KB,
image/jpeg
|
Details |
The current highlight/glow on pinned tabs (used as a notification to indicate when the title has changed) doesn't work so well with the dark titlebar from bug 1367384 / bug 1367385. The Photon mocks show this as changing to a small blue dot.
Updated•6 years ago
|
Whiteboard: [photon-visual] → [photon-visual] [triage]
Updated•6 years ago
|
Flags: qe-verify+
Priority: -- → P2
Whiteboard: [photon-visual] [triage] → [photon-visual][p2]
Updated•6 years ago
|
QA Contact: brindusa.tot
Assignee | ||
Comment 1•6 years ago
|
||
I checked with Stephen -- this is the SVG from his demo mockup, and is fine to use. And the single image/color is fine for the default/light/dark themes.
Assignee: nobody → dolske
Attachment #8882094 -
Flags: review?(dao+bmo)
Updated•6 years ago
|
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment 2•6 years ago
|
||
Comment on attachment 8882094 [details] [diff] [review] Patch v.1 >--- a/browser/themes/shared/compacttheme.inc.css >+++ b/browser/themes/shared/compacttheme.inc.css >+%ifdef MOZ_PHOTON_THEME >+ --pinned-tab-glow-url: url(chrome://browser/skin/tabbrowser/indicator-tab-attention.svg); >@@ -84,7 +88,11 @@ toolbar:-moz-lwtheme-brighttext { > --tab-selection-color: #f5f7fa; > --tab-selection-background-color: #4c9ed9; > --tab-selection-box-shadow: none; >+%ifdef MOZ_PHOTON_THEME >+ --pinned-tab-glow-url: url(chrome://browser/skin/tabbrowser/indicator-tab-attention.svg); > .tabbrowser-tab:-moz-any([image], [pinned]) > .tab-stack > .tab-content[attention]:not([selected="true"]), > .tabbrowser-tab > .tab-stack > .tab-content[pinned][titlechanged]:not([selected="true"]) { >+%ifdef MOZ_PHOTON_THEME >+ background-image: var(--pinned-tab-glow-url); >+ background-position: center bottom -4px; Please remove background-image and the --pinned-tab-glow-url variable since you're not changing the value from what you already set in tabs.inc.css.
Attachment #8882094 -
Flags: review?(dao+bmo) → review+
Assignee | ||
Comment 3•6 years ago
|
||
Updated for landing. https://treeherder.mozilla.org/#/jobs?repo=try&revision=3af874360c71c0ed9d0dd01dd05de3cfa9171aed
Attachment #8882094 -
Attachment is obsolete: true
Pushed by jdolske@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b8e1f6c6ebd6 Update pinned-tab highlight/glow style for Photon. r=dao
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b8e1f6c6ebd6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Updated•6 years ago
|
Iteration: --- → 56.2 - Jul 10
Comment 6•6 years ago
|
||
Sweet screenshots: https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=a578ce873d805743283df9a3627372939d8c8b2c&newProject=mozilla-central&newRev=d536973fe668c6c6046fc3fda82e24f3379e3713&filter=fourPinned
Comment 7•6 years ago
|
||
(In reply to Justin Dolske [:Dolske] from comment #0) > The Photon mocks show this as changing to a small blue dot. This change introduced huge notification background for non-pinned tabs. Is it by purpose? See screenshot here: https://bug1383312.bmoattachments.org/attachment.cgi?id=8888959
Updated•6 years ago
|
status-firefox57:
affected → ---
Updated•6 years ago
|
QA Contact: brindusa.tot → ovidiu.boca
Comment 8•6 years ago
|
||
User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID: 20170907220212 I would like to verify this issue but I don't know exactly what to follow or what to look for. I've opened Facebook, Facebook Messenger, Skype in the browser (tabs unpinned) but there is no notification or glow. Only the number of notification is displayed in front of the tab title. Is this the expected result? I have used the Light theme with Compact density.
Flags: needinfo?(dolske)
Comment 9•6 years ago
|
||
For the pinned tabs the blue dot that represents the notification is displayed but not fully displayed. The bottom part is cut off. Is this also intended?
Comment 10•6 years ago
|
||
User Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID: 20170912220343 Verified on Windows 8.1 x64, on Windows 10 x64 and x32, Mac OS 10.12 and Ubuntu 14.04 with latest Nightly 57.0a1 and I cannot reproduce the issue.
Comment 11•6 years ago
|
||
Based on previous comment mark this bug verified.
Assignee | ||
Comment 12•6 years ago
|
||
(In reply to Vlad Bacia-Mociran [:VladB] from comment #9) > Created attachment 8905925 [details] > pin notification cut off > > For the pinned tabs the blue dot that represents the notification is > displayed but not fully displayed. The bottom part is cut off. Is this also > intended? Yes. (In reply to Vlad Bacia-Mociran [:VladB] from comment #8) > I would like to verify this issue but I don't know exactly what to follow or > what to look for. I've opened Facebook, Facebook Messenger, Skype in the > browser (tabs unpinned) but there is no notification or glow. Only the > number of notification is displayed in front of the tab title. Is this the > expected result? The tab's title text needs to change while it's in the background (since this highlight is meant to indicate unseen changes). IIRC when writing this I just used a little test page that did a setTimeout (to allow me to switch to another page) then then updated the title. I see this every day in normal usage, so agree it's verified.
Flags: needinfo?(dolske)
You need to log in
before you can comment on or make changes to this bug.
Description
•