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

fix: tab can't scroll to the right position #3649

Open
wants to merge 1 commit into
base: v2
Choose a base branch
from

Conversation

Sociosarbis
Copy link

Please makes sure the items are checked before submitting your PR, thank you!

  • Rebase before creating a PR to keep commit history clear.

  • Only One commit

  • No eslint errors

  • 原来的代码会出现不能选中的tab滚动到container的中间位置的问题。

  • 先假设在进行最后一次滚动前依然没有到达目标位置,距离设为delta,那么最后一次会只是滚动delta / 15,不能让delta归零。

@codecov-io
Copy link

Codecov Report

Merging #3649 into v2 will decrease coverage by 0.1%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##               v2    #3649      +/-   ##
==========================================
- Coverage   18.89%   18.79%   -0.11%     
==========================================
  Files         226      226              
  Lines        5598     5598              
  Branches      725      725              
==========================================
- Hits         1058     1052       -6     
- Misses       4422     4428       +6     
  Partials      118      118
Impacted Files Coverage Δ
src/components/tab/tab.vue 0% <0%> (ø) ⬆️
src/components/inline-calendar/util.js 69.41% <0%> (-3.53%) ⬇️
src/components/picker/index.vue 5.37% <0%> (-2.16%) ⬇️
src/components/inline-calendar/index.vue 28.76% <0%> (-0.69%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d06e45...15836f3. Read the comment docs.

@Sociosarbis
Copy link
Author

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

Successfully merging this pull request may close these issues.

2 participants