Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Don't autoscroll when using the mouse to add, delete or move selections #15546

Merged
merged 1 commit into from Sep 6, 2017

Conversation

as-cii
Copy link
Contributor

@as-cii as-cii commented Sep 5, 2017

Fixes #15489

Previously, we used to scroll the editor on behalf of the user when clicking near the edges. This behavior was being perceived as jarring, because it would move up/down the line the user was trying to click:

kapture 2017-09-05 at 15 23 22

With this pull-request we are restoring the behavior Atom 1.18 exhibited, which prevents the editor from auto-scrolling when using the mouse to add, delete or move selections:

kapture 2017-09-05 at 15 25 24

/cc: @nathansobo @ungb @Ben3eeE @50Wliu

@Ben3eeE
Copy link
Contributor

Ben3eeE commented Sep 5, 2017

/cc: @rsese

@rsese
Copy link
Contributor

rsese commented Sep 5, 2017

Confirmed the fix on macOS 10.12.6 with this branch ✌️

@ungb
Copy link
Contributor

ungb commented Sep 5, 2017

I was able to verify as well before and after! LGTM!

@as-cii as-cii merged commit 7bd2c67 into master Sep 6, 2017
@as-cii as-cii deleted the as-never-autoscroll-when-clicking-on-content branch September 6, 2017 09:12
as-cii pushed a commit that referenced this pull request Sep 6, 2017
…-on-content

Don't autoscroll when using the mouse to add, delete or move selections
as-cii pushed a commit that referenced this pull request Sep 6, 2017
…-on-content

Don't autoscroll when using the mouse to add, delete or move selections
@CptAdamica
Copy link

Hi! Where can I find thiese two files?

  • text-editor-component-spec.js
  • text-editor-component.js

:)

@CptAdamica
Copy link

1.19.6 is perfect, thanks. :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants