New issue
Advanced search Search tips

Issue 866694 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

scroll-behavior: smooth erroneously respected for find (ctrl+f) matches

Reported by sangeeth...@gmail.com, Jul 23

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36

Steps to reproduce the problem:
1. Go to any content-heavy URL and using DevTools, add `scroll-behavior: smooth` to `html` selector. Or go to https://nodejs.org/en/docs/guides/event-loop-timers-and-nexttick/ which currently does that.
2. Hit Ctrl+F, enter a common word
3. Search through the matches

What is the expected behavior?
To directly jump to the next or previous search matches upon navigating.

What went wrong?
Trying to go to next or previous match when `scroll-behavior: smooth` is present is animating the scroll to those matches. This takes very long when I want to say jump between first and last matches. Firefox doesn't respect this for Ctrl+F which I think is the correct thing to do.

Did this work before? No 

Chrome version: 67.0.3396.99  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 30.0 r0
 
chrome_scroll.gif
4.1 MB View Download
Labels: Needs-Triage-M67
Cc: phanindra.mandapaka@chromium.org
Labels: Triaged-ET Needs-Feedback
Thanks for filling the issue...

Tested this issue on reported chrome version 67.0.3396.99 using Windows 10. Attaching screen-cast for reference.
Steps: 
---------
1. Launched reported chrome and 
2. Navigated to given URL " https://nodejs.org/en/docs/guides/event-loop-timers-and-nexttick/ " and opened search by hitting ctrl+F
3. Opened Dev tools>> Console >> Enabled and Disabled "scroll-behavior: smooth"  
4. Entered text "queue" 
As we are observed that focus navigating normally. 

@Reporter: Could you please review the attached screen-cast and confirm if anything being missed here.
Could you please upgrade to latest chrome 68.0.3440.75, you can download latest chrome builds here: "https://www.chromium.org/getting-involved/dev-channel".let us know if issue still persists or not.

Thanks..!
866694.mp4
17.0 MB Download
Hi @phanindra,

Thanks for checking this. Yes, I have reviewed your screencast and it's exactly the behavior I originally reported. I updated Chrome to v68.0.3440.75 and unfortunately, the behavior still persists. I've attached a screencast below.
2018-07-27_21-10-01.mp4
3.9 MB View Download
Project Member

Comment 4 by sheriffbot@chromium.org, Jul 27

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Needs-Feedback

As per Comment #3 retested this issue and as we are unable to reproduce the issue on reported chrome version 68.0.3440.75 using Windows 10. 
Followed the steps in comment #2, As we are observed that focus navigating normally and not jumping like comment #3

@Reporter: Request you to retry this issue with fresh profile without any extensions/apps or reset all the flags and let us know if issue still persists.

Thanks..!
@phanindra

I have tested this once again on v68.0.3440.75. I went to chrome://flags and reset everything to default, relaunched the browser. Instead of creating a new profile, I opened in "Guest" mode since no extensions/app are interfering there. I went to the site I mentioned in Comment #1 where the CSS rule was present and did the exact same things mentioned in the comment. I'm observing the same scroll animation (instead of jumps) when using Ctrl+F.

I also have Canary installed at v70.0.3507.0. I opened a "Guest" window there and did the same steps and it's the same behavior there too. The procedure I've did for doing both of these is recorded and posted here already so I don't think any more clarification is necessary.

And yes, removing the CSS rule brings back the jump to search matches when using Ctrl+F on both stable/canary.
Project Member

Comment 7 by sheriffbot@chromium.org, Jul 31

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: swarnasree.mukkala@chromium.org
Components: Blink>Scroll
Labels: TE-NeedsTriageHelp
Unable to reproduce on chrome version #68.0.3440.106 and latest chrome version #71.0.3541.0 on Windows 10 by following steps in comment#0. As per comment#2 and comment#5, the issue is not reproducible on TE end, hence adding TE-NeedsTriageHelp label. Requesting anyone from Blink>Scroll team to look into the issue.

Attached screencast for reference.
Thanks.!
866694.mp4
15.4 MB Download

Comment 9 Deleted

I noticed this issue as well. Tested with version 68.0.3440.106 (Official Build) (64-bit) on macOS 10.13.6.

As OP mentioned, removing the CSS rule fixes this behavior. This is not an issue in the latest of Firefox.

Screen Recording 2018-09-04 at 11.01 AM.mov
8.5 MB View Download
Labels: Needs-Feedback
Owner: sunyunjia@chromium.org
`scroll-behavior: smooth` should make search jump animating. I think what sangeethmanayil@ wants is to not animate when the distance is too long. Right?
Status: Assigned (was: Unconfirmed)
@chaopeng, yes, you're right. Like I mentioned, Firefox doesn't respect this value for searches and I believe that'd be the right thing to do. Search should be immediate and the animating is just adding unnecessary delay in producing the desired results.

Sign in to add a comment