Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Select] onScroll triggered and change options, scrollbar go back to top #2168

Closed
Zhaocl1997 opened this issue Jan 13, 2022 · 5 comments
Closed
Labels
feature request New feature or request

Comments

@Zhaocl1997
Copy link
Contributor

TuSimple/naive-ui version (版本)

2.24.1

Vue version (Vue 版本)

3.2.26

Browser and its version (浏览器及其版本)

chrome 97

System and its version (系统及其版本)

win10 Enterprise

Node version (Node 版本)

16.13

Reappearance link (重现链接)

https://codesandbox.io/s/elated-brown-dzhl9?file=/src/Demo.vue

Reappearance steps (重现步骤)

Focus on select, scroll down, and when bottom reached triggered, scrollbar go back to top

Expected results (期望的结果)

do not scroll to top when options changed

Actual results (实际的结果)

scrollbar go back to top when options changed

Remarks (补充说明)

@github-actions github-actions bot added the untriaged need to sort label Jan 13, 2022
@07akioni 07akioni added consideration-needed and removed untriaged need to sort labels Jan 14, 2022
@07akioni
Copy link
Collaborator

We need to consider how the behavior should work since sometimes user may want to scroll to top & sometimes user may want not to scroll to top.

@Zhaocl1997
Copy link
Contributor Author

Ok, it's werid though when i want to load more options from backend, it just scroll back to top. Or maybe provide a flag(?) to decide whether scroll to top when options changed? Is that possible?

@07akioni
Copy link
Collaborator

Ok, it's werid though when i want to load more options from backend, it just scroll back to top. Or maybe provide a flag(?) to decide whether scroll to top when options changed? Is that possible?

Will try.

@07akioni 07akioni added feature request New feature or request and removed consideration-needed labels Jan 16, 2022
@07akioni
Copy link
Collaborator

closed in 09d362d

@Zhaocl1997
Copy link
Contributor Author

ok, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants