search

ucla / logon

self-service identity management prototype. not always up to date viewable build:
http://ucla.github.io/logon/
BSD 3-Clause "New" or "Revised" License
1 stars 8 forks source link

Override `display: none;` with `.show` #16

Closed bezhermoso closed 7 years ago

bezhermoso commented 7 years ago

Hi @erutan,

In order to implement the Get Help / More Information toggle on mobile, I needed a way to override the display: none; CSS rule on .mfa .supporting-info on mobile. I could use inline styles, but I thought adding a class is a cleaner approach.

Let me know what you think and if there is a more elegant solution.

erutan commented 7 years ago

Adding a show class makes sense to me - I put in the display: none; more as a proof of concept for prototyping, but it's still a bit simpler than adding even more classes with js. :)

bezhermoso commented 7 years ago

Awesome, thanks.

When toggling the Help content and mobile, I'm just hiding .content and adding .show to .suppported-info. Thanks for merging.