[openstreetmap/openstreetmap-website] Wrap table header rows in thead (#2460)

Andy Allan notifications at github.com
Wed Dec 11 16:26:43 UTC 2019


While working on replacing our stripy table logic with an out-of-the-box bootstrap solution, I ran into a problem with table header rows getting the stripes applied to them. This is because we're missing `<thead>` wrappers on some tables.

This PR adds those wrappers, which currently do nothing much, but will make the future bootstrap work easier and those future commits much smaller.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Wrap table header rows in thead

-- File Changes --

    M app/views/diary_entries/comments.html.erb (12)
    M app/views/notes/mine.html.erb (18)
    M app/views/oauth_clients/index.html.erb (12)
    M app/views/user_blocks/_blocks.html.erb (34)
    M app/views/users/index.html.erb (34)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/2460.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2460.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/2460
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20191211/b74cb1fa/attachment.html>


More information about the rails-dev mailing list