Closed
Bug 1343495
Opened 7 years ago
Closed 7 years ago
CSS Grid Highlighter icon is blurry
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(firefox52+ wontfix, firefox53+ verified, firefox54 verified)
VERIFIED
FIXED
Firefox 54
People
(Reporter: zer0, Assigned: zer0)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
59 bytes,
text/x-review-board-request
|
gl
:
review+
lizzard
:
approval-mozilla-aurora+
jcristau
:
approval-mozilla-beta-
|
Details |
25.99 KB,
image/png
|
Details | |
11.66 KB,
image/png
|
Details | |
28.51 KB,
image/png
|
Details |
The current CSS Grid Highlighter icon is still blurry, that it's more evident on 1ddpx than retina displays. It should be crisp no matter the dpi.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
I forgot to mention that where the icon itself now is not blurry anymore, it's still not crisp in rule view; so Gabriel review the icon but this bug is not over yet. The reason is because the current icon in the rule view has size `1em x 1em`, so to speak 11x11 pixels. The icons are made to be 16x16, so since they're SVG they will work with anything can scale well (8x8, 32x32…). We mentioned that in the past when we where talking about this issue. Possible solutions: 1. Make the grid icon specifically for the rule view, so 11x11, and force the css style to be 11x11 instead of 1em x 1em. 2. Increase the size of the icons in rule-view, at least the grid icon, so that they will be 16x16 pixels. Gabriel, what do you think? I'm going to attach a screenshot of the icon in the rule-view at 16x16, just to have a comparison.
Flags: needinfo?(gl)
Assignee | ||
Comment 3•7 years ago
|
||
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8842376 [details] Bug 1343495 - CSS Grid Highlighter icon is blurry; https://reviewboard.mozilla.org/r/116256/#review117826
Attachment #8842376 -
Flags: review?(gl) → review+
Updated•7 years ago
|
Flags: needinfo?(gl)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•7 years ago
|
||
Assignee | ||
Comment 7•7 years ago
|
||
Comment hidden (mozreview-request) |
Assignee | ||
Comment 9•7 years ago
|
||
The last commit is for having an optimized path as the first commit I made – I forgot in the second.
Comment 10•7 years ago
|
||
Pushed by mferretti@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/17a520550438 CSS Grid Highlighter icon is blurry; r=gl
Assignee | ||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Flags: qe-verify+
Assignee | ||
Comment 11•7 years ago
|
||
Comment on attachment 8842376 [details] Bug 1343495 - CSS Grid Highlighter icon is blurry; Approval Request Comment [Feature/Bug causing the regression]: CSS Grid Inspector (Bug 1181227) [User impact if declined]: Users will have blurry icon for CSS Grid Inspector, especially in 1dppx display (regular ones, no retina for example). [Is this code covered by automated tests?]: No (it's an icon improvement). [Has the fix been verified in Nightly?]: Not yet, just landing. [Needs manual test from QE? If yes, steps to reproduce]: 1. http://labs.jensimmons.com/2016/examples/grid-content-1.html 2. Toggle Devtools (Tools > Web Developer > Toggle Tools) 3. Select the <main> node in the inspector 4. Select the Rules panel 5. See that the grid icon shown after the `display` property of the element in the Rules panel is blurry (that's more evident in 1dppx monitor). [List of other uplifts needed for the feature/fix]: [Is the change risky?]: No. [Why is the change risky/not risky?]: Doesn't involve code, it's an icon improvement of what we've landed already. [String changes made/needed]: None.
Attachment #8842376 -
Flags: approval-mozilla-beta?
Attachment #8842376 -
Flags: approval-mozilla-aurora?
Comment 12•7 years ago
|
||
Comment on attachment 8842376 [details] Bug 1343495 - CSS Grid Highlighter icon is blurry; Too late for 52, sorry.
Attachment #8842376 -
Flags: approval-mozilla-beta? → approval-mozilla-beta-
Comment 13•7 years ago
|
||
(In reply to Julien Cristau [:jcristau] from comment #12) > Comment on attachment 8842376 [details] > Bug 1343495 - CSS Grid Highlighter icon is blurry; > > Too late for 52, sorry. Wanted to know if it is absolutely impossible to uplift this to beta since you commented on Bug 1342051 about uplifting that bug. We do want this uplifted since one of the biggest pushes we are doing on the devtools, devrel and marketing for FF52 is to market and ship this grid inspector.
Updated•7 years ago
|
Flags: needinfo?(jcristau)
Comment 14•7 years ago
|
||
Bug 1342051 was called out as a blocker for shipping, so it gets a last minute exception, but I didn't think a less-than-24-hour-old bug about a tiny icon being slightly blurry qualified.
Flags: needinfo?(jcristau)
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/17a520550438
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Updated•7 years ago
|
status-firefox52:
--- → affected
status-firefox53:
--- → affected
Comment 16•7 years ago
|
||
We can take this for 53 aurora. Uplifting this now to 52 would delay the entire release. We could keep it open and tracked in case of a dot release, though.
tracking-firefox52:
--- → +
tracking-firefox53:
--- → +
Comment 17•7 years ago
|
||
Comment on attachment 8842376 [details] Bug 1343495 - CSS Grid Highlighter icon is blurry; Minor icon resolution fix, let's take it for 53 aurora.
Attachment #8842376 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 18•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/21bb36dab77e
Comment 19•7 years ago
|
||
Verified fixed using the latest Nightly 54.0a1 (2017-03-06) on Ubuntu 16.04, Mac OS X 10.11 and Windows 10 x64
Status: RESOLVED → VERIFIED
Comment 20•7 years ago
|
||
Verified fixed Fx 53b1 on Win 10, Ubuntu 14.04, OS X 10.11.
Flags: qe-verify+
Updated•7 years ago
|
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•