pixel perfection for avatar menu icon |
|||
Issue descriptionWe display the avatar menu icon at 16x16. Currently we use the attached SVG and scale down, but for all other SVG icons of this size in primary UI we actually use a separate asset optimized for display at very small dimensions. (foo.1x.icon vs. foo.icon) Do we not want the same for this,one?
,
Jul 10 2017
ping Alan
,
Jul 14 2017
What assets are exported at 1x and 2x that you're referring to? How do we ensure quality across fractional displays then?
,
Jul 14 2017
Literally all the icons in primary ui have two definitions: navigation icons, close buttons, the bookmark star, the app menu icon, bookmark folders, etc. Currently we don't ensure pixel perfection at fractional scale factors, or even 3x for that matter, but pixel perfection is less important/noticeable the greater the dsf.
,
Jul 15 2017
Do you remember where the cutover is? My memory is that we use 1x for all DSFs < 2, and 2x for all DSFs >= 2. Things aren't too bad since these are vector-based. It's just that for low pixel densities we wanted some more-optimized versions.
,
Jul 15 2017
It's 1x for 1x and 2x (scaled) for all else.
,
Jul 17 2017
So there's 1x and 2x svgs for all primary UI and then we programmatically color?
,
Jul 17 2017
Yes. Some icons that are larger and/or less prominent only have one variety. Also note that 1x and 2x never differ only in scale (that would defeat the purpose of the specific 1x asset).
,
Jul 17 2017
I looked into it...supplying 1 variety vs. 2 makes little difference from the UX side of things. Does it make a difference for eng? Arguably, the scaled version is better than the rest (#3). See attached 1 - major modifications to original scaled artwork 2 - 2px modification to original scaled artwork 3 - original scaled artwork
,
Jul 17 2017
> Does it make a difference for eng? Nope. If we're happy with the way the scaled version looks, we're all good here. Usually pixel perfection consists of aligning straight lines to pixel boundaries but there are no straight lines in this icon, so it makes some sense if we just stick to plain scaling.
,
Jul 18 2017
Sticking with plain scaling works for me then |
|||
►
Sign in to add a comment |
|||
Comment 1 by sheriffbot@chromium.org
, Jun 26 2017