Skip to content

fix: reset header visibility when switching notes#83

Merged
j4rviscmd merged 1 commit intomainfrom
fix/header-visibility
Mar 27, 2026
Merged

fix: reset header visibility when switching notes#83
j4rviscmd merged 1 commit intomainfrom
fix/header-visibility

Conversation

@j4rviscmd
Copy link
Copy Markdown
Owner

Summary

  • When switching from a scrolled note to a short note, the header remained hidden because the scroll direction state was not reset
  • Expose resetHeader from useScrollDirection hook to clear internal state (suppress timer, accumulated wheel delta) and force the header visible
  • Call resetHeader() in selectNote when switching notes

Test plan

  • Open a long note, scroll down to hide the header
  • Switch to a short note (content that doesn't require scrolling)
  • Verify the header is visible
  • Verify the header still hides/shows correctly when scrolling within a note
  • Verify header state is restored correctly on app restart

When switching from a scrolled note to a short note, the header
remained hidden because the scroll direction state was not reset.
Expose resetHeader from useScrollDirection and call it in selectNote
to clear internal state and force the header visible.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@j4rviscmd j4rviscmd added the bug Something isn't working label Mar 27, 2026
@j4rviscmd j4rviscmd merged commit d85222f into main Mar 27, 2026
2 checks passed
@j4rviscmd j4rviscmd deleted the fix/header-visibility branch March 27, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant