Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Store nav items in database #121

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Store nav items in database #121

wants to merge 10 commits into from

Conversation

GAsplund
Copy link
Member

@GAsplund GAsplund commented Sep 3, 2024

Key features

Known issues

  • There's really nothing stopping a nav category from having child items, even if the category (aka "parent") has a URL (which is considered an illegal state, but child items will just be ignored should that occur)
  • There is no "dynamic" CSS sizing on the navbar width, meaning that the content could look cramped before switching to hamburger menu

@GAsplund
Copy link
Member Author

GAsplund commented Sep 4, 2024

Okay, I'm not crazy. It seems like editing the navbar to be dynamic messes a lot with static routes. I think it propagates out to every other page but mainly is because (I think) that the root error page needs to be 100% static. I will look into this later when I have the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant