From 9d2297f65e72bdfeb3a23a9b37a56ede5e46110f Mon Sep 17 00:00:00 2001
From: Marcelo Paiva <mpaiva@gmail.com>
Date: Thu, 26 Feb 2026 12:18:31 -0500
Subject: [PATCH] Fix BetterHtml::DontInterpolateHere in _html_modal.html.erb

Move aria-labelledby conditional to a local variable above the tag
so the attribute uses a <%= %> expression instead of <% %> inside the tag.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 app/views/shared/_html_modal.html.erb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/views/shared/_html_modal.html.erb b/app/views/shared/_html_modal.html.erb
index 2cdd7b35..7a18733d 100644
--- a/app/views/shared/_html_modal.html.erb
+++ b/app/views/shared/_html_modal.html.erb
@@ -1,7 +1,8 @@
 <% uuid = local_assigns[:uuid] || SecureRandom.uuid %>
+<% modal_labelledby = local_assigns[:title] ? "modal-title-#{uuid}" : nil %>
 <input type="checkbox" id="<%= uuid %>" class="modal-toggle">
 <div id="<%= local_assigns[:id] %>" class="modal items-start !animate-none overflow-y-auto">
-  <div class="modal-box pt-4 pb-6 px-6 mt-20 max-h-none" role="dialog" aria-modal="true"<% if local_assigns[:title] %> aria-labelledby="modal-title-<%= uuid %>"<% end %>>
+  <div class="modal-box pt-4 pb-6 px-6 mt-20 max-h-none" role="dialog" aria-modal="true" aria-labelledby="<%= modal_labelledby %>">
     <% if local_assigns[:title] %>
       <div class="flex justify-between items-center border-b pb-2 mb-2 font-medium">
         <span id="modal-title-<%= uuid %>">