-
Notifications
You must be signed in to change notification settings - Fork 64
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
Toggle Boxes Example? #57
Comments
You can argue that the “show/hide” interaction missing with For your CodePen demo, mind the Firefox visual styling: The CSS override being used with |
Sometimes it takes a really long time for new features to land (look at the Ambient Light API, colorpicker, datepicker) so that's why I think it is important to show what can be done even if it isn't supported well enough for production. The more people that know about it the more up votes for those features.
I can't find one either :( |
Fair enough. I think you can definitely make arguments for showing examples without 100% support (depending on the goals of this project). And it is just like the color picker example which doesn’t yet have support in a couple of major browsers, so it is probably a good fit. Interestingly |
@jpdevries Here’s a Firefox fix: http://codepen.io/opattison/pen/ORoJoE?editors=1100 (set |
@opattison that is awesome!!! Thank you for the Firefox fix. Ya, I think |
Thoughtbot has a post with some examples and one of them is Toggle Boxes.
You Don't Need JavaScript for That! https://robots.thoughtbot.com/you-don-t-need-javascript-for-that
I've forked their Codepen and updated it to use
<details>
Codepen
http://codepen.io/jpdevries/pen/kkjpBJ?editors=1100
I'm not sure if it is ok for
<dd>
and<dt>
to not be in the same immediate parent but the example is also marked up FAQ style.The text was updated successfully, but these errors were encountered: