I was looking at app/assets/javascripts/views/restrooms/index.js and found that it was indented by tabs and had some custom js animations that could have been done with jQuery, since we're already using it. So I changed some things.
Note that I changed the animation for toggling the map to a fade in/out, since I thought it was cleaner, but we can change it back. jQuery has functions for sliding up and down as well.
I recommend looking at individual commit diffs instead of the combined one, since I changed the indentation so every line was replaced.
Checklist
[x] Tested Mobile Responsiveness
[ ] Added Unit Tests
[ ] CI Passes
[ ] Deploys to Heroku on test Correctly (Maintainers will handle)
[ ] Added Documentation (Service and Code when required)
Context
app/assets/javascripts/views/restrooms/index.jsand found that it was indented by tabs and had some custom js animations that could have been done with jQuery, since we're already using it. So I changed some things.Note that I changed the animation for toggling the map to a fade in/out, since I thought it was cleaner, but we can change it back. jQuery has functions for sliding up and down as well.
I recommend looking at individual commit diffs instead of the combined one, since I changed the indentation so every line was replaced.
Checklist