Skip to content

Conversation

AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Sep 18, 2025

Purpose

When navigating to a new document, the scroll position was preserved.
This PR changes this behavior to scroll back to the top of the page when navigating to a new document.

Fixe: #1393

@AntoLC AntoLC self-assigned this Sep 18, 2025
@AntoLC AntoLC added bug Something isn't working frontend labels Sep 18, 2025
@AntoLC AntoLC linked an issue Sep 18, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Sep 18, 2025

Size Change: +80 B (0%)

Total Size: 3.65 MB

Filename Size Change
apps/impress/out/_next/static/24365ee5/_buildManifest.js 0 B -862 B (removed) 🏆
apps/impress/out/_next/static/54355b99/_buildManifest.js 865 B +865 B (new file) 🆕

compressed-size-action

@AntoLC AntoLC requested a review from Ovgodd September 18, 2025 15:42
Copy link
Collaborator

@Ovgodd Ovgodd left a comment

Choose a reason for hiding this comment

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

nice !

When navigating to a new document, the scroll
position was preserved. This commit changes this
behavior to scroll back to the top of
the page when navigating to a new document.
@AntoLC AntoLC merged commit c23ff54 into main Sep 22, 2025
19 of 22 checks passed
@AntoLC AntoLC deleted the fix/scroll-to-top branch September 22, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scroll location is not reset

2 participants