Skip to content

Double decker nav #235

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

Merged
merged 9 commits into from
Oct 4, 2024
Merged

Double decker nav #235

merged 9 commits into from
Oct 4, 2024

Conversation

Rich-Harris
Copy link
Member

WIP. The basic idea is that instead of having two navs, stacked awkwardly, we have a double decker nav on the routes that need it — for now just the playground, but I can see this maybe working for the tutorial as well.

Has a few other style tweaks intended to get rid of some visual noise and make everything a bit more consistent.

image

Copy link

vercel bot commented Oct 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
omnisite ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 4, 2024 0:34am

@dummdidumm
Copy link
Member

On dark mode it looks particularly nice because of the different background colors of the nav and the file names etc bar. On desktop I can't decide whether or not it looks clean or too spacey/lost because there's no separation at all between the nav bar, the REPL control, and the file names etc

@Rich-Harris
Copy link
Member Author

tweaked it a little bit, the lower nav now has the same shadow that the single decker nav would normally have, providing that separation

@Rich-Harris
Copy link
Member Author

gonna go ahead and merge to unblock other work

@Rich-Harris Rich-Harris merged commit 50037a8 into main Oct 4, 2024
3 checks passed
@Rich-Harris Rich-Harris deleted the double-decker-nav branch October 4, 2024 18:49
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.

2 participants