From 227d5a8757cfe0cdd281ce29d272104db9dbf561 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@37signals.com>
Date: Fri, 11 Apr 2025 18:10:26 +0200
Subject: [PATCH] It is both reacting and reactions

So stick with the simpler dom id scope
---
 app/views/cards/comments/reactions/_reactions.html.erb | 2 +-
 app/views/cards/comments/reactions/new.html.erb        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/cards/comments/reactions/_reactions.html.erb b/app/views/cards/comments/reactions/_reactions.html.erb
index 396ed4e33..0e53e817b 100644
--- a/app/views/cards/comments/reactions/_reactions.html.erb
+++ b/app/views/cards/comments/reactions/_reactions.html.erb
@@ -1,4 +1,4 @@
-<%= turbo_frame_tag comment, :reacting do %>
+<%= turbo_frame_tag comment, :reactions do %>
   <div class="reactions flex flex-wrap align-center gap full-width">
     <div class="flex-inline flex-wrap gap">
       <%= render partial: "cards/comments/reactions/reaction", collection: comment.reactions.ordered, locals: { comment: comment } %>
diff --git a/app/views/cards/comments/reactions/new.html.erb b/app/views/cards/comments/reactions/new.html.erb
index 0a11d9c07..f5773c470 100644
--- a/app/views/cards/comments/reactions/new.html.erb
+++ b/app/views/cards/comments/reactions/new.html.erb
@@ -2,7 +2,7 @@
   <%= form_with url: card_comment_reactions_path(@comment.card, @comment),
         class: "reaction reaction__form flex-inline postion--relative max-width align-center fill-white gap expanded",
         html: { aria: { label: "New reaction" } },
-        data: { controller: "form", turbo_frame: dom_id(@comment, :reacting), action: "keydown.esc->form#cancel" } do |form| %>
+        data: { controller: "form", turbo_frame: dom_id(@comment, :reactions), action: "keydown.esc->form#cancel" } do |form| %>
     <label class="reaction__form-label flex gap" style="--column-gap: 0.7ch;">
       <figure class="reaction__avatar margin-none flex-item-no-shrink">
         <%= avatar_tag Current.user %>