Location icon bubble arrow has glitch at 1.5x |
||||||||
Issue descriptionSee attached when opening lock on Windows 53.0.2782.0 (Official Build) canary (32-bit). There's some funny bits between the arrow and the main part of the window. xref to https://bugs.chromium.org/p/chromium/issues/detail?id=615153 which seems similar.
,
Jun 29 2016
It also looks to me like there's an issue with how the arrow is stroked, where the left side isn't stroked correctly. I saw something like this at 2x scale also. If that isn't fixed in this bug we should file a followup for it.
,
Jun 29 2016
Yeah I fixed something like that with the previous bug, where ostensibly hairline strokes were losing precision and drawing weirdly. I'll take a look at it too. I opened the cert bubble at 1.25 and I saw a couple places where it's doing that.
,
Jun 29 2016
I suspect any issues here are common to all bubbles and not location-icon-specific, so fixing this would improve a number of places.
,
Jun 30 2016
This actually happens regardless of whether MD is on or not. I'll still try to fix it now because the other bug is fresh in my mind.
,
Jul 13 2016
The main issue is that bubbles are still painted with images, and those seem to have a lot of issues at fractional dsfs. I hacked the bubble to draw programmatically and I think it looks decent (see attached screenshots). There are two issues before I actually submit this though: 1. It's tricky to get the bubble borders to draw transparent, like they are in the image version. I don't know how much we care about keeping it that way. 2. There are a bunch of variations on the bubbles that as far as I can tell are used almost nowhere. For example there's a "BIG_SHADOW" type which is used by app_list_view and tray_bubble_view, neither of which I can figure out how to trigger. It'd be great to pare these down to what we care about, which if it were up to me would be only "small shadow" and maybe "no shadow."
,
Jul 18 2016
The specific special cases you mention both sound like CrOS-specific UI to me. Can you say more about what you mean regarding the border transparency? Maybe a comparison shot along with a description of the issue and e.g. why just drawing with a non-255 alpha value doesn't work.
,
Jul 18 2016
If you think it'd be prudent I can enable my new UI only for Windows, and then specialists for other platforms can do the work to enable it elsewhere. Then I can focus on small/no shadow. I'll write up more about the border transparency in a bit.
,
Jul 18 2016
I don't think that would be a good idea. Such a change would require us to have both programmatic and image-based paths in the codebase. We should try to address all of views at once.
,
Aug 3 2016
I think bug 590285 comment 27 problem 1 is probably covered under this bug. Do you agree?
,
Aug 3 2016
#10: I don't think so actually. I see the same kind of artifact at 1x and 2x, so it doesn't have to do with hidpi.
,
Aug 4 2016
Hmm. Does it repro in non-material as well or is it at least MD-specific? I assume you're on the hook for "draw bubble borders in a non-crappy way" and thus this would still be on your plate even if it's a separate issue, but if you're not going to file a bug for it tell me.
,
Aug 9 2016
#12: I filed bug 635996 for that.
,
Nov 23 2016
,
Nov 23 2016
,
Nov 23 2016
,
Sep 18
Archiving old bugs that have only received trivial updates for some time. If you feel this issue should still be addressed, feel free to reopen it or to file a new issue. Thanks!
,
Sep 19
We no longer draw these arrows. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by pkasting@chromium.org
, Jun 29 2016Labels: M-53 Proj-MaterialDesign-NativeUI OS-Chrome OS-Linux
Owner: bsep@chromium.org