Skip to content

Feat: handle scroll offset on anchor links#641

Merged
tblivet merged 1 commit intoPrestaShop:developfrom
tblivet:feat/scroll_padding
Jun 26, 2024
Merged

Feat: handle scroll offset on anchor links#641
tblivet merged 1 commit intoPrestaShop:developfrom
tblivet:feat/scroll_padding

Conversation

@tblivet
Copy link
Contributor

@tblivet tblivet commented Jun 17, 2024

Questions Answers
Description? Add functionality to handle the offset of a sticky header when clicking on anchor links and prevent item overlapping.
Type? new feature
BC breaks? no
Deprecations?
Fixed ticket?
Sponsor company @PrestaShopCorp
How to test? This feature is related to an issue find in this PR #640 (review) (BUG 1)

Before:

scroll_before.mov

After:

scroll_after.mov

@tblivet tblivet requested review from Hlavtox, boherm and nicosomb June 17, 2024 12:35
Copy link

@florine2623 florine2623 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, it is QA ✅

Screen.Recording.2024-06-26.at.11.43.26.mov

@tblivet tblivet merged commit 3491039 into PrestaShop:develop Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants