As requested by @Firefishy at #osmf-operations. Also coloured the 
"suspend" action in danger red.

<img width="736" height="87" alt="Links to Confirm 
or Suspend a user. The second one in red to express danger" 
src="https://github.com/user-attachments/assets/661f5764-ad4f-40e5-b733-a6661e205b99"
 />

Something that annoys me of these action links: the confirmation modal is 
common to both (and other links that can appear here) and simply reads 
"Confirm" in English. I would have preferred a descriptive 
"Suspend user %{user_name}" (etc) or similar, to further help prevent 
an accident. Thoughts? It's a bit of a faff, with all the i18n strings, 
wouldn't do it as part of this PR anyway.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Swap actions around for consistency with users/lists
  * Colour 'suspend' action as dangerous

-- File Changes --

    M app/views/users/show.html.erb (16)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to