Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

added scroll restoration on forward/backward navigation#1878

Open
KitsuneKode wants to merge 1 commit intocode100x:main from
KitsuneKode:feature/remember-last-scroll-position
Open

added scroll restoration on forward/backward navigation #1878
KitsuneKode wants to merge 1 commit intocode100x:main from
KitsuneKode:feature/remember-last-scroll-position

Conversation

@KitsuneKode
Copy link
Contributor

@KitsuneKode KitsuneKode commented Aug 16, 2025

PR Fixes:

  • 1 Added Scroll restoration to contents

The course content page scrolls to the top on navigation(backward/forward).
Scrolling through the same weeks of content again and again to find the video/week is just straight up annoying.
This PR fixes it, restores back to the point before navigation.

Checklist before requesting a review

  • [x ] I have performed a self-review of my code
  • [x ] I assure there is no similar/duplicate pull request regarding same issue

Copy link

@KitsuneKode

  • isn't ref logic redundant?
  • also is there any use of { scroll: false } as we're using window.scrollTo logic in useEffect anyway?
    it seemed to work fine w/o them too..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /