[openstreetmap/openstreetmap-website] More CSS cleanups (#2709)

Andy Allan notifications at github.com
Wed Jul 15 12:40:55 UTC 2020


This PR contains several CSS cleanups. 

The main theme is continuing the work of disentangling the small screen stuff (which should be done with media breakpoints or builtin bootstrap functionality) from the navigation bar changes (which is based on the length of the text in the menus, not on the screen size).

Various other changes were found during this work too.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/2709

-- Commit Summary --

  * Remove dead CSS code - nothing has the user_map class
  * Rework the about header size change to be based on the screen width, not the menu state
  * Remove webkit-appearance: none
  * Move the login form rules to be based on screen size
  * Rename nav-related body classes
  * Use bootstrap backgrounds and text utilities for read issue reports
  * Remove unused css
  * Remove unnecessary css

-- File Changes --

    M app/assets/javascripts/application.js (10)
    M app/assets/stylesheets/common.scss (27)
    M app/assets/stylesheets/small.scss (64)
    M app/views/issues/show.html.erb (4)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/2709.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2709.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2709
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20200715/5e8c486c/attachment.htm>


More information about the rails-dev mailing list