From 42aa213312310ad0ced78081dabd9bb35eab8fd9 Mon Sep 17 00:00:00 2001
From: Andy Smith <andy@37signals.com>
Date: Tue, 7 Oct 2025 10:48:05 -0500
Subject: [PATCH] Remove aside and toolbar role from perma actions

---
 app/views/cards/_container.html.erb                  | 4 ++--
 app/views/cards/container/footer/_published.html.erb | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/views/cards/_container.html.erb b/app/views/cards/_container.html.erb
index 057282d85..ab0d0d70c 100644
--- a/app/views/cards/_container.html.erb
+++ b/app/views/cards/_container.html.erb
@@ -3,10 +3,10 @@
     class="card-perma" style="--card-color: <%= card.color %>;">
     <%= render "cards/container/status", card: card %>
 
-    <aside class="card-perma__actions card-perma__actions--left" role="toolbar">
+    <div class="card-perma__actions card-perma__actions--left">
       <%= render "cards/container/gild", card: card if @card.published? && !@card.closed? %>
       <%= render "cards/container/image", card: card %>
-    </aside>
+    </div>
 
     <div class="card-perma__bg">
       <%= card_article_tag card, class: "card" do %>
diff --git a/app/views/cards/container/footer/_published.html.erb b/app/views/cards/container/footer/_published.html.erb
index c227f7f3c..ec2fe6527 100644
--- a/app/views/cards/container/footer/_published.html.erb
+++ b/app/views/cards/container/footer/_published.html.erb
@@ -1,7 +1,7 @@
 <%# FIXME: Let's move this aside outside of the card container section so these frames don't reload/flicker when card is replaced %>
-<aside class="card-perma__actions card-perma__actions--right" role="toolbar">
+<div class="card-perma__actions card-perma__actions--right">
   <%= turbo_frame_tag card, :watch, src: card_watch_path(card), target: "_top", refresh: :morph %>
   <%= turbo_frame_tag card, :pin, src: card_pin_path(card), refresh: :morph %>
-</aside>
+</div>
 
 <%= render "cards/container/closure", card: card %>