Closed
Bug 1257348
Opened 8 years ago
Closed 8 years ago
GCLI close button disappears when hover.
Categories
(DevTools Graveyard :: Graphic Commandline and Toolbar, defect)
Tracking
(firefox46 unaffected, firefox47+ fixed, firefox48+ verified)
VERIFIED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox46 | --- | unaffected |
firefox47 | + | fixed |
firefox48 | + | verified |
People
(Reporter: johngraciliano, Assigned: philip.chee)
References
Details
(Keywords: regression, Whiteboard: [btpp-fix-now])
Attachments
(1 file)
3.12 KB,
patch
|
bgrins
:
review+
ritu
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:47.0) Gecko/20100101 Firefox/47.0 Build ID: 20160315004019 Steps to reproduce: Start Firefox (Developer Edition 47.0a2 or Nightly 48.0a1). Press Shift-[F2] to display the 'command line' Move the 'mouse pointer' on top of the close button ('x at the end') of the GCLI. Actual results: The close button becomes invisible. Expected results: The close button remain visible, perhaps just change its appearance, but not disappear. In previous releases, it simply fades a little.
Updated•8 years ago
|
Component: Untriaged → Developer Tools
Reporter | ||
Comment 1•8 years ago
|
||
The button now has the class 'close-icon' (it was 'devtools-closebutton') for which chrome://global/skin/global.css has: .close-icon { list-style-image: url("chrome://global/skin/icons/close.png"); -moz-image-region: rect(0, 20px, 20px, 0); } .close-icon:hover { -moz-image-region: rect(0, 40px, 20px, 20px); } .close-icon:hover:active { -moz-image-region: rect(0, 60px, 20px, 40px); } But in chrome://browser/skin/browser.css it states for #developer-toolbar-closebutton { list-style-image: url("chrome://devtools/skin/images/close.png"); ... opacity: 0.6; } ... #developer-toolbar-closebutton:hover { opacity: 0.8; } #developer-toolbar-closebutton:hover:active { opacity: 1; } But chrome://devtools/skin/images/close.png has only one 'sprite' hence the problem.* I suggest to include: -moz-image-region: auto; in the rule for #developer-toolbar-closebutton inside browser.css. That will solve issue even if min-resolution: 1.1dppx. ______ *The image size is also different 16×16 vs 20×20 with the class resource.
Reporter | ||
Updated•8 years ago
|
Component: Developer Tools → Developer Tools: Graphic Commandline and Toolbar
Reporter | ||
Comment 2•8 years ago
|
||
The button also appears smaller than it should be when not hover and the reason is in the asterisk of comment 1.
It's due to bug 1239464. Maybe bug 1225184 will fix it.
Blocks: 1239464
Status: UNCONFIRMED → NEW
status-firefox46:
--- → unaffected
status-firefox47:
--- → affected
status-firefox48:
--- → affected
tracking-firefox47:
--- → ?
tracking-firefox48:
--- → ?
Depends on: 1225184
Ever confirmed: true
Keywords: regression
OS: Unspecified → All
Hardware: Unspecified → All
Updated•8 years ago
|
Whiteboard: [multiviewport] [triage]
Comment 4•8 years ago
|
||
@Loic, I don't think bug 1239464 is related at all; could you please elaborate? It didn't touch the GCLI close icon or any stylesheet related to GCLI.
Flags: needinfo?(zer0)
Updated•8 years ago
|
Flags: needinfo?(epinal99-bugzilla2)
Comment 5•8 years ago
|
||
Regressed by bug 1252479.
Updated•8 years ago
|
Whiteboard: [multiviewport] [triage] → [btpp-fix-now]
Reporter | ||
Comment 6•8 years ago
|
||
Thank you Tim. The class change in https://hg.mozilla.org/mozilla-central/rev/bcc5851a4598 allows the button to be styled by rules in global.css. But these the rules for the id in browser.css have precedence and overide most of them. However, there is no overide for the -moz-image-region property. You may either add the line I suggest at the end of comment 1 or clean/simplify browser.css so the button can be styled with the code in common.css. There should be a good reason for the change in class regarding SeaMonkey, isn't there? Otherwise, it may be best to revert the class to what it was. But if the intention is to change the button appearance as that in tabs and elsewhere, cleaning/simplifying code in browser.css is the needed.
Assignee | ||
Comment 7•8 years ago
|
||
I originally did not remove the styles for #developer-toolbar-closebutton because I thought something else was using this id. But now I see that the toolbox.xul close button is using class="devtools-closebutton" so now we can remove all the #developer-toolbar-closebutton styles from: http://mxr.mozilla.org/comm-central/source/mozilla/browser/themes/osx/browser.css#3263 http://mxr.mozilla.org/comm-central/source/mozilla/devtools/client/themes/commandline.inc.css
Flags: needinfo?(philip.chee)
Assignee | ||
Comment 8•8 years ago
|
||
Try server build https://hg.mozilla.org/try/pushloghtml?changeset=6666fb3ea423
Assignee | ||
Comment 9•8 years ago
|
||
Oops: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8fc063aa0ce9
Assignee | ||
Comment 10•8 years ago
|
||
> +:root[devtoolstheme="dark"] #developer-toolbar > .close-icon:not(:hover) > image {
> + filter: invert(1);
In dark theme the toolkit close button image is very hard to notice - if not impossible so I've inverted the colours. But in light theme I avoid inverting the colours for the same reason.
Comment 11•8 years ago
|
||
Comment on attachment 8732648 [details] [diff] [review] Fix close button styles Review of attachment 8732648 [details] [diff] [review]: ----------------------------------------------------------------- I'm going to forward this review to Brian, who I hope can get to it more quickly that me. Sorry for the delay.
Attachment #8732648 -
Flags: review?(jwalker) → review?(bgrinstead)
Comment 12•8 years ago
|
||
Comment on attachment 8732648 [details] [diff] [review] Fix close button styles Review of attachment 8732648 [details] [diff] [review]: ----------------------------------------------------------------- Works for me, thanks ::: devtools/client/themes/commandline.inc.css @@ +59,5 @@ > width: 16px; > height: 16px; > } > > +:root[devtoolstheme="dark"] #developer-toolbar > .close-icon:not(:hover) > image { Please add a comment above this as in Comment 10. Something like: "The toolkit close button is low contrast in the dark theme so invert it"
Attachment #8732648 -
Flags: review?(bgrinstead) → review+
Comment 15•8 years ago
|
||
Comment on attachment 8732648 [details] [diff] [review] Fix close button styles Approval Request Comment [Feature/regressing bug #]: bug 1252479 [User impact if declined]: disappearing developer toolbar close icon on hover [Describe test coverage new/current, TreeHerder]: local testing, will soon be in Nightly (currently in mozilla-inbound) [Risks and why]: low, simple CSS change [String/UUID change made/needed]: none
Attachment #8732648 -
Flags: approval-mozilla-aurora?
Comment 16•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2ee670054a59
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 18•8 years ago
|
||
It regressed on linux, see bug 1260751.
Tracked 47+ as it's a recent regression.
Hello Johngraciliano, could you please verify this issue is fixed as expected on a latest Nightly build? Thanks!
Flags: needinfo?(johngraciliano)
Comment on attachment 8732648 [details] [diff] [review] Fix close button styles Recent regression, Aurora47+
Attachment #8732648 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 22•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/a9534031e9d2
Updated•8 years ago
|
Comment 23•8 years ago
|
||
I have successfully reproduce this bug on firefox nightly 48.0a1 (2016-03-16) with windows 7 (32 bit) Mozilla/5.0 (Windows NT 6.1; rv:48.0) Gecko/20100101 Firefox/48.0 I found this fix on latest aurora 47.0a2 (2016-04-19) Mozilla/5.0 (Windows NT 6.1; rv:47.0) Gecko/20100101 Firefox/47.0 Build ID : 20160419004026 I also found fix on latest nightly 48.0a1 (2016-04-19) Mozilla/5.0 (Windows NT 6.1; rv:48.0) Gecko/20100101 Firefox/48.0 Build ID :20160316030233 [bugday-20160420]
Updated•8 years ago
|
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Flags: needinfo?(johngraciliano)
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•