Skip to content

Nav state#348

Merged
outofambit merged 5 commits intomainfrom
nav-state
May 9, 2024
Merged

Nav state#348
outofambit merged 5 commits intomainfrom
nav-state

Conversation

@outofambit
Copy link
Copy Markdown
Collaborator

@outofambit outofambit commented May 8, 2024

  • refactors the state management of the main nav links and jump to links panels to:
    • on mobile: make it such that both panels cannot be open at the same time
    • on desktop: at least one panel must be open at all times
    • watches for viewport changes require switching from one behavior to another

this introduces a bit more javascript to the app, but i think its unavoidable to get the behavior right here

@outofambit outofambit enabled auto-merge May 9, 2024 00:08
@outofambit outofambit merged commit d0519bf into main May 9, 2024
@outofambit outofambit deleted the nav-state branch May 9, 2024 00:12
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.

1 participant