[openstreetmap/openstreetmap-website] Add diary discussion heading and move subscribe button next to it (PR #4518)

Anton Khorev notifications at github.com
Sun Feb 11 23:49:20 UTC 2024


Places (un)subscribe button on diary entry pages like it's on changeset pages. That requires adding a heading above the comments to make it clear to what you're subscribing.

![image](https://github.com/openstreetmap/openstreetmap-website/assets/4158490/df945d38-f810-47b2-81f3-696f52bbfdcc)

Previously the button was right below the new comment box which is somewhat confusing, see #4491. You don't want to accidentally press it after typing in a comment.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Add diary discussion heading and move subscribe button next to it

-- File Changes --

    M app/views/diary_entries/show.html.erb (21)
    M config/locales/en.yml (1)

-- Patch Links --

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

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4518
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4518 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240211/de7154bf/attachment.htm>


More information about the rails-dev mailing list