Add option to hide navlinks on mobile #204
Conversation
🦋 Changeset detectedLatest commit: 6158cea The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Thank you for submitting this @ThomasSingleton I really appreciate it! This is a good first draft but my concern with this implementation is that it doesn't really give the developer an option for if they DO want to have the nav links for their mobile users. Perhaps instead of not rendering it at all we could instead nest it in the mobile navigation menu above the "sidebar" links (screenshot attached with some annotations to give an example). It could be a separate option completely for how the developer wants it rendered (if they want to keep it along the top for mobile, don't render anything at all, or instead render it inline like in the screenshot). What do you think of that? Would that still achieve your use case and would that be something you'd be interested in implementing? |
This is a very crude, band-aid "fix" for #72.
Adds optional
navLinksMobileDisplayconfig option.Allows control over whether main navlinks are shown
flexor hiddennoneon mobile breakpoints.Defaults to
flex, retaining the previous behavior where nav links were visible (sometimes, sort-of) on mobile.