Change 'close warning' link to button

mozilla / popcorn.webmaker.org

RETIRED

https://blog.mozilla.org/foundation-archive/mozilla-learning/product-update-for-appmaker-and-popcorn-maker/

MIT License

146 stars 116 forks source link

Change 'close warning' link to button #600

Closed negomi closed 9 years ago

negomi commented 9 years ago

This PR:

changes the link to a button element
restyles it following @k88hudson's design (https://bugzilla.mozilla.org/show_bug.cgi?id=903611#c11)
reorders the Localized.get() calls so the text flows properly around the new (floated) button on smaller screens

negomi commented 9 years ago

The button is now static in the layout file. While tweaking the styling to work with the new layout, I thought a transform might be a nicer alternative to the transition on margin-top, as it seemed like a cleaner way of animating to an unspecified height. Ironically, most of the browsers this warning will appear in won't have support for either, but I've included it anyway just in case :)