Merge pull request #809 from basecamp/fix-dialog-controller-error

Dialog target needs to be in the DOM on connect

app/views/cards/display/perma/_tags.html.erb

@@ -17,6 +17,8 @@
       <span class="for-screen-reader">Add a tag</span>
     </button>
 
-    <%= turbo_frame_tag card, :tagging, src: new_card_tagging_path(card) %>
+    <dialog class="popup panel flex-column align-start justify-start fill-white shadow txt-small" data-dialog-target="dialog">
+      <%= turbo_frame_tag card, :tagging, src: new_card_tagging_path(card) %>
+    </dialog>
   </div>
 </div>

app/views/cards/taggings/new.html.erb

@@ -1,7 +1,6 @@
 <%= turbo_frame_tag @card, :tagging do %>
-  <dialog class="popup panel flex-column align-start justify-start fill-white shadow txt-small"
+  <div class="flex-column"
     data-controller="filter navigable-list"
-    data-dialog-target="dialog filter navigable-list"
     data-navigable-list-focus-on-selection-value="false"
     data-navigable-list-actionable-items-value="true"
     data-action="keydown->navigable-list#navigate filter:changed->navigable-list#reset"
@@ -25,5 +24,5 @@
         </li>
       <% end %>
     </ul>
-  </dialog>
+  </div>
 <% end %>