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: fixed Anchor scroll bug #1688

Merged
merged 1 commit into from
Jul 10, 2023
Merged

fix: fixed Anchor scroll bug #1688

merged 1 commit into from
Jul 10, 2023

Conversation

shijiatongxue
Copy link
Collaborator

中文模板 / Chinese Template

What kind of change does this PR introduce? (check at least one)

  • Bugfix

PR description

如果滚动 actions 存在多个,且第一个纵向不能滚动时,点击时没有反应问题。

我们应该找出来有纵向滚动的元素,忽略掉仅横向滚动元素

Changelog

🇨🇳 Chinese

  • Fix: 修复 Anchor 在部分场景下,点击无法跳转问题

🇺🇸 English

  • Fix: Fix the problem that Anchor cannot jump when clicked in some scenarios

Checklist

  • Test or no need
  • Document or no need
  • Changelog or no need

Other

  • Skip Changelog

Additional information

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 7, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit f0d4ec2:

Sandbox Source
pr-story Configuration
Semi Design: Simple Story Configuration

@cypress
Copy link

cypress bot commented Jul 7, 2023

Passing run #1748 ↗︎

0 212 10 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Merge f0d4ec2 into 4cdb478...
Project: semi-design Commit: 9b81e06df5 ℹ️
Status: Passed Duration: 11:12 💡
Started: Jul 7, 2023 11:09 AM Ended: Jul 7, 2023 11:20 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@YannLynn YannLynn merged commit 608c83e into main Jul 10, 2023
8 of 9 checks passed
@YannLynn YannLynn deleted the fix/anchor-scroll-2 branch July 10, 2023 11:15
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.

3 participants