Issue metadata
Sign in to add a comment
|
Regression: Text Font on overlay is seen blur after click and hold action
Reported by
nutan.ga...@etouch.net,
Sep 8 2017
|
||||||||||||||||||||||
Issue descriptionChrome Version : 62.0.3202.13 8aba0a34f8ab922b12535ce172e7ae6434d8e603-refs/branch-heads/3202@{#75} OS : Mac(10.11.6, 10.12.3,10.12.5) Steps to reproduce: 1. Launch chrome, naviagte to chrome://settings/searchEngines 2. Click on 'ADD' button of 'Other search engines' (Add search engine overlay open). 3. Click and hold mouse outside the overlay continuously and observe 4. Observe Actual: Text Font on overlay is seen blur after click and hold action Expected : Text Font on overlay should not be seen blur after click and hold action Good build: 62.0.3201.0 Bad build: 62.0.3202.0
,
Sep 8 2017
,
Sep 11 2017
Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/7fa976fb9f2f1b59dea9034c793eea7c496b61d9..2a3de6795bf6ee9e3dae2313991c8b86f6a7f06f?pretty=fuller&n=10000 Suspect: r499066? @scottchen: Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Thanks.
,
Sep 12 2017
Removing Needs-Bisect as bisect is already provided in C#3.
,
Sep 12 2017
,
Sep 12 2017
Clicking outside and holding outside the dialog is not a supported user behavior. The dialog's pulsing animation is designed to be a temporary affect so I think temporary blurriness is fine - as long as it doesn't persist *after* the mouse is released. @dpapad thoughts?
,
Sep 12 2017
Is the blurriness caused by r499066? Or is it happening because of the pulsing animation itself regardless of rr499066? If the former is true, I think we should figure out if there is a better fix that does not introduce blurriness. Either way, we can afford some blurriness during the animation, but it should go away as soon as the animation has finished (regardless of whether the user is still holding a mouse button).
,
Sep 12 2017
Ah I see what happened - there was supposed to be a follow-up CL that improves some the blurring issue (https://chromium-review.googlesource.com/c/chromium/src/+/651576) but it got landed right after the branch. I'll merge it to M62.
,
Sep 12 2017
,
Sep 13 2017
Tested and confirmed in Canary. Approving this merge for M62 (branch:3202)
,
Sep 13 2017
Merged.
,
Sep 18 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 18 2017
Just to clarify per comment #12: I took comment #10 as the approval and merged it, so the fix is already in.
,
Sep 18 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by nutan.ga...@etouch.net
, Sep 8 2017