-
Notifications
You must be signed in to change notification settings - Fork 39
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
Responsive Nav Menu Example issue #10 #52
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙌 Thanks @nelsonic. Corrected a couple of typos, ready to go.
relies on [Isabel Castillo](https://github.com/isabelc)'s | ||
brilliant _hidden_ `checkbox` idea: | ||
https://isabelcastillo.com/pure-css-mobile-toggle-menu | ||
(shared by @iteles in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha I didn't need the shout-out but thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you found the pearl. definitely worth recognising. recognition is core to our culture. ❤️
Co-Authored-By: Ines Teles Correia <[email protected]>
Co-Authored-By: Ines Teles Correia <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
menu tested and working well 👍
While building the
start|stop timer
feature dwyl/app#265,I tried to implement a mobile-first (responsive) navigation menu in Tachyons and failed miserably. 🤦♂
After some googling I discovered issue #10 (which I had opened a while back ...)
(It's always good to re-discover a @dwyl issue with some helpful content! the system works!!)
@Cleop left a comment #10 (comment) linking to the
dwyl-site
implementation of the CSS-only responsive navigation menu: https://github.com/dwyl/dwyl-site/pull/264/files#diff-b0638442d6af953ff33fca3d49113579L18The link and brief explanation is much better than nothing, so thanks for the comment! 🥇
And judging by how much ❤️ the comment has received from the community:
A lot of people using Tachyons have found this useful. 🎉
But ... rather than forcing people to read through a commit which isn't very beginner-friendly,
I've split out the code and simplified it into an example with in-line comments and explanation.
This PR includes
README.md
: https://github.com/dwyl/learn-tachyons/tree/responsive-nav-menu-issue%2310#responsive-navigation-menuexamples/nav-menu.html
file people can learn from and play with.@iteles / @SimonLab please take a look when you can. (not urgent, just nice-to-have)
Thanks! 🌻