Use classes instead of inline styles for hiding / showing

That's a very good point...I think in all actuality, this would still work here (if `display: none;` was used for the default hide) thanks to the cascading thing since this would be the last class added. Obviously that wouldn't be the case for inline styles, but we definitely shouldn't be doing that ourselves and discouraging others anyway.