decidim-admin 0.28.4 → 0.29.0.rc1

data/app/packs/stylesheets/decidim/admin/_proposal_status.scss

@@ -0,0 +1,5 @@
+.proposal-status__color {
+  label {
+    @apply sr-only;
+  }
+}

data/app/packs/stylesheets/decidim/admin/application.scss

@@ -35,4 +35,6 @@
 @import "stylesheets/decidim/admin/_data_picker.scss";
 @import "stylesheets/decidim/admin/_legacy_foundation.scss";
 @import "stylesheets/decidim/admin/_sidebar-menu.scss";
+@import "stylesheets/decidim/admin/_datepicker.scss";
 @import "stylesheets/decidim/admin/_minimap.scss";
+@import "stylesheets/decidim/admin/_proposal_status.scss";

data/app/permissions/decidim/admin/permissions.rb

@@ -218,11 +218,11 @@ module Decidim
         @organization ||= context.fetch(:organization, nil) || context.fetch(:current_organization, nil)
       end
 
-      def user_can_enter_space_area?(**args)
+      def user_can_enter_space_area?(**)
         return unless permission_action.action == :enter &&
                       permission_action.subject == :space_area
 
-        space_allows_admin_access_to_current_action?(**args)
+        space_allows_admin_access_to_current_action?(**)
       end
 
       def space_allows_admin_access_to_current_action?(require_admin_terms_accepted: false)

data/app/queries/decidim/admin/user_filter.rb

@@ -2,10 +2,10 @@
 
 module Decidim
   module Admin
-    # A class used to filter users by whitelisted scope or searches on their
+    # A class used to filter users by allowed scope or searches on their
     # name
     class UserFilter < Decidim::Query
-      WHITELISTED_STATE_SCOPES = %w(
+      ALLOWED_STATE_SCOPES = %w(
         officialized
         not_officialized
         managed
@@ -32,7 +32,7 @@ module Decidim
         @state = state
       end
 
-      # List the User groups by the diferents filters.
+      # List the User groups by the different filters.
       def query
         users = scope
         users = filter_by_search(users)
@@ -50,7 +50,7 @@ module Decidim
       end
 
       def filter_by_state(users)
-        return users unless WHITELISTED_STATE_SCOPES.include?(state)
+        return users unless ALLOWED_STATE_SCOPES.include?(state)
 
         users.public_send(state)
       end

data/app/queries/decidim/admin/user_groups_evaluation.rb

@@ -24,7 +24,7 @@ module Decidim
         @state = state
       end
 
-      # List the User groups by the diferent filters.
+      # List the User groups by the different filters.
       def query
         @user_groups = filter_by_search(@user_groups)
         @user_groups = filter_by_state(@user_groups)

data/app/views/decidim/admin/attachment_collections/index.html.erb

@@ -14,7 +14,7 @@
         <thead>
           <tr>
             <th><%= t("activemodel.attributes.attachment_collection.name") %></th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody>

data/app/views/decidim/admin/attachments/_form.html.erb

@@ -17,8 +17,27 @@
         <%= form.select :attachment_collection_id, @form.attachment_collections.map { |c| [translated_attribute(c.name), c.id] }, include_blank: true %>
       </div>
 
-      <div class="row column">
-        <%= form.upload :file, button_class: "button button__sm button__transparent-secondary" %>
+      <div class="p-6" data-file-or-link-tabs-controller>
+        <%= cell "decidim/tab_panels", [
+          {
+            enabled: true,
+            id: "file",
+            text: "Upload file",
+            icon: "file-upload-line",
+            method: :cell,
+            selected: form.object.file.present?,
+            args: ["/decidim/attachments_file_tab", form]
+          },
+          {
+            enabled: true,
+            id: "link",
+            text: "Link",
+            icon: "link",
+            method: :cell,
+            selected: form.object.link.present?,
+            args: ["/decidim/attachments_link_tab", form]
+          }
+        ] %>
       </div>
     </div>
   </div>

data/app/views/decidim/admin/attachments/index.html.erb

@@ -18,7 +18,7 @@
             <th><%= t("models.attachment.fields.collection", scope: "decidim.admin") %></th>
             <th><%= t("models.attachment.fields.content_type", scope: "decidim.admin") %></th>
             <th><%= t("models.attachment.fields.file_size", scope: "decidim.admin") %></th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody>
@@ -34,7 +34,7 @@
                 <%= attachment.file_type %>
               </td>
               <td>
-                <%= number_to_human_size(attachment.file_size) %>
+                <%= attachment.file? ? number_to_human_size(attachment.file_size) : "-" %>
               </td>
               <td class="table-list__actions">
                 <% if allowed_to? :update, :attachment, attachment: attachment %>

data/app/views/decidim/admin/authorization_workflows/index.html.erb

@@ -14,24 +14,42 @@
     </h2>
   </div>
   <div class="card-section">
-    <table class="table-list table-list--lastleft">
+    <table class="table-list">
       <thead>
         <tr>
           <th><%= t("decidim.authorization_handlers.name") %></th>
-          <th><%= t("decidim.authorization_handlers.help") %></th>
+          <th class="!text-left"><%= t("decidim.authorization_handlers.help") %></th>
         </tr>
       </thead>
       <tbody>
         <% @workflows.each do |manifest| %>
           <tr>
             <% workflow = Decidim::Verifications::Adapter.new(manifest) %>
-            <td><%= link_to workflow.fullname, workflow.admin_root_path %></td>
             <td>
-              <ul>
-                <% t("decidim.authorization_handlers.admin.#{workflow.key}.help").each do |tip| %>
-                  <li><%= tip %></li>
-                <% end %>
-              </ul>
+              <% if workflow.has_admin_root_path? %>
+                <%= link_to workflow.fullname, workflow.admin_root_path %>
+              <% else %>
+                <%= workflow.fullname %>
+              <% end %>
+            </td>
+            <td class="!text-left">
+              <% i18n_key = "decidim.authorization_handlers.admin.#{workflow.key}.help" %>
+              <% if I18n.exists?(i18n_key, current_locale) %>
+                <ul class="list-decimal ml-4">
+                  <% t(i18n_key).each do |tip| %>
+                    <li><%= tip %></li>
+                  <% end %>
+                </ul>
+              <% else %>
+                <div class="flash warning">
+                  <ul>
+                    <% default_help = t("decidim.authorization_handlers.admin.default.help", authorization_handler: i18n_key) %>
+                    <% default_help.each do |tip| %>
+                      <li><%= tip %></li>
+                    <% end %>
+                  </ul>
+                </div>
+              <% end %>
             </td>
           </tr>
         <% end %>

data/app/views/decidim/admin/block_user/new.html.erb

@@ -19,7 +19,7 @@
               <%= cell("decidim/announcement", t(".already_reported_html"), callout_class: "alert" ) %>
             <% end %>
 
-            <% if Decidim.module_installed?(:templates) %>
+            <% if defined?(Decidim::Templates) %>
               <%= render "decidim/templates/admin/block_user_templates/template_chooser", form: f %>
             <% end %>
 

data/app/views/decidim/admin/categories/index.html.erb

@@ -14,7 +14,7 @@
         <thead>
           <tr>
             <th><%= t("activemodel.attributes.category.name") %></th>
-            <th class="actions"></th>
+            <th></th>
           </tr>
         </thead>
         <tbody>
@@ -38,7 +38,7 @@
               </td>
             </tr>
             <% category.subcategories.each do |subcategory| %>
-              <tr class="extra__table-list__subcategory">
+              <tr>
                 <td>
                   <%= link_to translated_attribute(subcategory.name), edit_category_path(current_participatory_space, subcategory) %><br>
                 </td>

data/app/views/decidim/admin/components/_form.html.erb

@@ -59,7 +59,7 @@
 
       <div id="panel-step_settings" class="card-section">
         <div class="row column">
-          <fieldset class="step-settings">
+          <fieldset>
             <div class="card-section">
               <% current_participatory_space.steps.each do |step| %>
                 <fieldset class="step-settings-<%= step.id %>">

data/app/views/decidim/admin/components/index.html.erb

@@ -14,12 +14,8 @@
         <div class="dropdown-pane" id="add-component-dropdown" data-dropdown data-auto-focus="true" data-close-on-click="true">
           <ul class="vertical menu add-components font-normal">
             <% @manifests.each do |manifest| %>
-             <%= link_to new_component_path(type: manifest.name), class: manifest.name do %>
-              <li>
-                <%= t("#{manifest.name}.name", scope: "decidim.components") %>
-              </li>
+              <li><%= link_to t("#{manifest.name}.name", scope: "decidim.components"), new_component_path(type: manifest.name), class: manifest.name %></li>
             <% end %>
-          <% end %>
           </ul>
         </div>
       </div>

data/app/views/decidim/admin/conflicts/index.html.erb

@@ -1,32 +1,10 @@
 <% add_decidim_page_title(t("title", scope: "decidim.admin.conflicts")) %>
-<div class="item_show__header">
-  <h1 class="item_show__header-title">
-    <%= t("title", scope: "decidim.admin.conflicts") %>
-  </h1>
-</div>
-
-<div class="filters__section">
-  <div class="fcell search">
-    <%= search_form_for(query, url: url_for) do |form| %>
-      <%= applied_filters_hidden_field_tags %>
-      <div class="input-group">
-        <%= form.search_field(
-              search_field_predicate,
-              class: "input-group-field",
-              label: false,
-              placeholder: t(".text")
-            ) %>
-        <div class="input-group-button">
-          <button type="submit" class="text-secondary" aria-label="<%= t("decidim.search.term_input_placeholder") %>">
-            <%= icon "search-line", class: "fill-secondary w-4 h-4" %>
-          </button>
-        </div>
-      </div>
-    <% end %>
-  </div>
-</div>
-
 <div class="card">
+  <div class="item_show__header">
+    <h1 class="item_show__header-title">
+      <%= t("title", scope: "decidim.admin.conflicts") %>
+    </h1>
+  </div>
   <div class="table-scroll">
     <table class="table-list">
       <thead>
@@ -39,7 +17,7 @@
         </tr>
       </thead>
       <tbody>
-        <% conflicts.each do |conflict| %>
+        <% @conflicts.each do |conflict| %>
           <tr>
             <td><%= conflict.current_user.name %></td>
             <td><%= conflict.managed_user.name %></td>
@@ -52,4 +30,3 @@
     </table>
   </div>
 </div>
-<%= decidim_paginate conflicts %>

data/app/views/decidim/admin/dashboard/_pending_moderations.html.erb

@@ -6,7 +6,7 @@
   </div>
 
   <div class="card-section">
-    <p><%= t(".anouncement", count: count_pending_moderations) %></p>
+    <p><%= t(".announcement", count: count_pending_moderations) %></p>
     <p><%= link_to t(".goto_moderation"), moderations_path, class: "button button__sm button__secondary mt-2" %></p>
   </div>
 </div>

data/app/views/decidim/admin/dashboard/show.html.erb

@@ -2,7 +2,7 @@
 
 <div class="item_show__header">
   <h1 class="item_show__header-title">
-  <%= t "decidim.admin.titles.dashboard" %> <%= current_organization.name %>
+  <%= t "decidim.admin.titles.dashboard" %> <%= current_organization_name %>
   </h1>
 </div>
 

data/app/views/decidim/admin/exports/_dropdown.html.erb

@@ -1,4 +1,4 @@
-<span class="exports button button__sm button__secondary button--simple" data-toggle="<%= dropdown_id(filters) %>">
+<span class="exports button button__sm button__secondary" data-toggle="<%= dropdown_id(filters) %>">
   <% if filters.present? %>
     <%= t("actions.export-selection", scope: "decidim.admin") %>
   <% else %>
@@ -10,11 +10,11 @@
   <ul class="vertical menu add-components">
     <% component.manifest.export_manifests.each do |manifest| %>
       <% manifest.formats.each do |format| %>
-        <%= link_to exports_path(component, id: manifest.name, format:, resource_id:, filters:), method: :post do %>
-          <li class="exports--format--<%= format.downcase %> exports--<%= manifest.name %>">
-            <%= t("decidim.admin.exports.export_as", name: t("decidim.#{component.manifest.name}.admin.exports.#{manifest.name}"), export_format: t("decidim.admin.exports.formats.#{format}")) %>
-          </li>
-        <% end %>
+        <li class="exports--format--<%= format.downcase %> exports--<%= manifest.name %>">
+          <%= link_to t("decidim.admin.exports.export_as", name: t("decidim.#{component.manifest.name}.admin.exports.#{manifest.name}"), export_format: t("decidim.admin.exports.formats.#{format}")),
+                      exports_path(component, id: manifest.name, format:, resource_id:, filters:),
+                      method: :post %>
+        </li>
       <% end %>
     <% end %>
   </ul>

data/app/views/decidim/admin/impersonatable_users/index.html.erb

@@ -1,6 +1,6 @@
 <% add_decidim_page_title(t("impersonatable_users", scope: "decidim.admin.titles")) %>
 <% if current_organization.available_authorizations.empty? %>
-  <div class="section">
+  <div>
     <%= cell("decidim/announcement", t(".needs_authorization_warning"), callout_class: "warning" ) %>
   </div>
 <% end %>
@@ -32,7 +32,7 @@
   </div>
   <div class="fcell search">
     <%= form_tag "", method: :get do %>
-      <div class="filters__search">
+      <div>
         <div class="input-group">
           <%= search_field_tag :q, @query,label: false, class: "input-group-field", placeholder: t(".search") %>
           <%= hidden_field_tag :state, @state %>
@@ -64,7 +64,7 @@
             <td><%= user.managed? ? t(".managed") : t(".not_managed") %></td>
             <td class="table-list__actions">
               <% if allowed_to? :impersonate, :impersonatable_user, user: user %>
-                <%= icon_link_to "login-box-line", new_impersonatable_user_impersonation_path(user), t(".impersonate"), class: "action-icon--impersonate" %>
+                <%= icon_link_to "user-line", new_impersonatable_user_impersonation_path(user), t(".impersonate"), class: "action-icon--impersonate" %>
               <% end %>
               <%= icon_link_to "time-line", impersonatable_user_impersonation_logs_path(user), t(".view_logs"), class: "action-icon--view-logs" %>
               <% if allowed_to? :promote, :impersonatable_user, user: user %>

data/app/views/decidim/admin/imports/_dropdown.html.erb

@@ -1,4 +1,4 @@
-<span class="imports button button__sm button__secondary button--simple" data-toggle="import-dropdown">
+<span class="imports button button__sm button__secondary" data-toggle="import-dropdown">
   <%= t "actions.import", scope: "decidim.admin" %>
   <%= icon "arrow-down-s-line", class: "dropdown-filter-icon" %>
 </span>
@@ -7,12 +7,10 @@
     <% if custom %>
       <%= yield %>
     <% else %>
-     <% component.manifest.import_manifests.each do |import_manifest| %>
-        <%= link_to admin_imports_path(component, resource_id:, name: import_manifest.name) do %>
-          <li class="imports--file imports--<%= import_manifest.name %>">
-            <%= import_manifest.message(:label, self) %>
-          </li>
-        <% end %>
+      <% component.manifest.import_manifests.each do |import_manifest| %>
+        <li class="imports--file imports--<%= import_manifest.name %>">
+          <%= link_to import_manifest.message(:label, self), admin_imports_path(component, resource_id:, name: import_manifest.name) %>
+        </li>
       <% end %>
     <% end %>
   </ul>

data/app/views/decidim/admin/imports/new.html.erb

@@ -5,24 +5,23 @@
 
     <div id="js-other-actions-wrapper">
       <% if import_manifest.has_example? %>
-        <span class="imports-example tiny button button__sm button__secondary button--simple mr-2" data-toggle="example-dropdown">
+        <span class="imports-example button button__sm button__secondary mr-2" data-toggle="example-dropdown">
           <%= t(".download_example") %>
           <%= icon "arrow-down-s-line", class: "dropdown-filter-icon" %>
         </span>
         <div class="dropdown-pane" id="example-dropdown" data-dropdown data-position=bottom data-alignment=right data-auto-focus="true" data-close-on-click="true">
           <ul class="vertical menu add-components">
             <% mime_types.each do |format, name| %>
-              <%= link_to admin_imports_example_path(@current_component, name: import_manifest.name, format:) do %>
-                <li class="examples--format--<%= format.downcase %> examples--<%= import_manifest.name %>">
-                  <%= t(".download_example_format", name:) %>
-                </li>
-              <% end %>
+              <li class="examples--format--<%= format.downcase %> examples--<%= import_manifest.name %>">
+                <%= link_to t(".download_example_format", name:),
+                            admin_imports_example_path(@current_component, name: import_manifest.name, format:) %>
+              </li>
             <% end %>
           </ul>
         </div>
       <% end %>
 
-      <%= link_to t(".actions.back"), manage_component_path(@current_component), class: "button button__sm button__secondary hollow tiny button--simple" %>
+      <%= link_to t(".actions.back"), manage_component_path(@current_component), class: "button button__sm button__secondary hollow button--simple" %>
     </div>
   </h1>
 </div>

data/app/views/decidim/admin/logs/_filters.html.erb

@@ -11,14 +11,14 @@
                     } %>
               </div>
               <div class="row column">
-                <%= form.datetime_field :created_at_dtgteq %>
+                <%= form.datetime_field :created_at_dtgteq, hide_help: true %>
               </div>
               <div class="row column">
-                <%= form.datetime_field :created_at_dtlteq %>
+                <%= form.datetime_field :created_at_dtlteq, hide_help: true, default_time: "23:59" %>
               </div>
               <div class="row column">
                 <div class="grid grid-cols-[70%_30%] items-end gap-x-4">
-                  <div class="cell auto">
+                  <div class="cell">
                     <%= form.search_field(search_field_predicate, class: "input-group-field", label: t(".user"), placeholder: t(".text")) %>
                   </div>
                   <div class="cell">

data/app/views/decidim/admin/managed_users/promotions/new.html.erb

@@ -5,7 +5,7 @@
   </h1>
 </div>
 
-<div class="section">
+<div>
   <%= cell("decidim/announcement", t(".explanation"), callout_class: "warning" ) %>
 </div>
 

data/app/views/decidim/admin/moderated_users/index.html.erb

@@ -21,7 +21,7 @@
       <th><%= t(".reason") %></th>
       <th><%= sort_link(query, :created_at, t("models.moderation.fields.created_at", scope: "decidim.moderations")) %></th>
       <th><%= sort_link(query, :report_count, t(".reports"), default_order: :desc) %></th>
-      <th class="actions"><%= t(".actions.title") %></th>
+      <th><%= t(".actions.title") %></th>
     </tr>
     </thead>
     <tbody>
@@ -45,7 +45,7 @@
           <%= moderation.report_count %>
         </td>
 
-        <td class="actions">
+        <td>
           <% if !moderation.user.blocked? && allowed_to?(:unreport, :moderate_users) %>
             <%= icon_link_to "arrow-go-back-line", ignore_moderated_user_path(id: moderation), t(".actions.unreport"), class: "action-icon--unreport", method: :put %>
           <% end %>

data/app/views/decidim/admin/moderations/index.html.erb

@@ -27,7 +27,7 @@
         <% if params[:hidden] %>
           <th><%= t("models.moderation.fields.hidden_at", scope: "decidim.moderations") %></th>
         <% end %>
-        <th class="actions"><%= t("actions.title", scope: "decidim.moderations") %></th>
+        <th><%= t("actions.title", scope: "decidim.moderations") %></th>
       </tr>
     </thead>
     <tbody>
@@ -69,22 +69,22 @@
               <%= l(moderation.hidden_at, format: :long) if moderation.hidden_at.present? %>
             </td>
           <% end %>
-          <td class="actions">
+          <td>
             <%= icon_link_to "fullscreen-line",
                              moderation_reports_path(moderation_id: moderation),
                              t("actions.expand", scope: "decidim.moderations"),
                              class: "action-icon--expand" %>
-            <% if !moderation.reportable.hidden? && allowed_to?(:unreport, permission_resource) %>
+            <% if !moderation.reportable.hidden? && allowed_to?(:unreport, authorization_scope) %>
               <%= icon_link_to "arrow-go-back-line",
                                unreport_moderation_path(id: moderation),
                                t("actions.unreport", scope: "decidim.moderations"),
                                class: "action-icon--unreport",
                                method: :put %>
             <% end %>
-            <% if !moderation.reportable.hidden? && allowed_to?(:hide, permission_resource) %>
+            <% if !moderation.reportable.hidden? && allowed_to?(:hide, authorization_scope) %>
               <%= icon_link_to "eye-line", hide_moderation_path(id: moderation), t("actions.hide", scope: "decidim.moderations"), class: "action-icon--hide", method: :put %>
             <% end %>
-            <% if moderation.reportable.hidden? && allowed_to?(:unhide, permission_resource) %>
+            <% if moderation.reportable.hidden? && allowed_to?(:unhide, authorization_scope) %>
               <%= icon_link_to "eye-line", unhide_moderation_path(id: moderation), t("actions.unhide", scope: "decidim.moderations"), method: :put %>
             <% end %>
           </td>

data/app/views/decidim/admin/moderations/reports/index.html.erb

@@ -38,7 +38,7 @@
       <% end %>
     </dt>
     <dd class="reported-content">
-      <div class="current">
+      <div>
         <%= reported_content_for moderation.reportable %>
       </div>
       <% if translatable_resource?(moderation.reportable) %>
@@ -70,7 +70,7 @@
       <th><%= t("models.report.fields.reason", scope: "decidim.moderations") %></th>
       <th><%= t("models.report.fields.locale", scope: "decidim.moderations") %></th>
       <th><%= t("models.report.fields.details", scope: "decidim.moderations") %></th>
-      <th class="actions"><%= t("actions.title", scope: "decidim.moderations") %></th>
+      <th><%= t("actions.title", scope: "decidim.moderations") %></th>
     </tr>
     </thead>
     <tbody>
@@ -79,7 +79,7 @@
         <td><%= t("decidim.admin.moderations.report.reasons.#{report.reason}") %></td>
         <td><%= report.locale.present? ? locale_name(report.locale) : "" %></td>
         <td><%= report.details&.truncate(250) %></td>
-        <td class="actions">
+        <td>
           <%= icon_link_to "fullscreen-line",
                            moderation_report_path(moderation_id: moderation, id: report.id),
                            t("actions.expand", scope: "decidim.moderations"),
@@ -92,13 +92,13 @@
 </div>
 <div class="item__edit-sticky">
   <div class="item__edit-sticky-container">
-    <% if !moderation.reportable.hidden? && allowed_to?(:unreport, permission_resource) %>
+    <% if !moderation.reportable.hidden? && allowed_to?(:unreport, authorization_scope) %>
       <%= link_to t("actions.unreport", scope: "decidim.moderations"), unreport_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary" %>
     <% end %>
-    <% if !moderation.reportable.hidden? && allowed_to?(:hide, permission_resource) %>
+    <% if !moderation.reportable.hidden? && allowed_to?(:hide, authorization_scope) %>
       <%= link_to t("actions.hide", scope: "decidim.moderations"), hide_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary alert" %>
     <% end %>
-    <% if moderation.reportable.hidden? && allowed_to?(:unhide, permission_resource) %>
+    <% if moderation.reportable.hidden? && allowed_to?(:unhide, authorization_scope) %>
       <%= link_to t("actions.unhide", scope: "decidim.moderations"), unhide_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary alert" %>
     <% end %>
   </div>

data/app/views/decidim/admin/moderations/reports/show.html.erb

@@ -31,7 +31,7 @@
       <% end %>
     </dt>
     <dd class="reported-content">
-      <div class="current">
+      <div>
         <%= reported_content_for moderation.reportable, locale: @report.locale %>
       </div>
       <% if translatable_resource?(moderation.reportable) %>
@@ -51,13 +51,13 @@
 
 <div class="item__edit-sticky">
   <div class="item__edit-sticky-container">
-    <% if !moderation.reportable.hidden? && allowed_to?(:unreport, permission_resource) %>
+    <% if !moderation.reportable.hidden? && allowed_to?(:unreport, authorization_scope) %>
       <%= link_to t("actions.unreport", scope: "decidim.moderations"), unreport_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary" %>
     <% end %>
-    <% if !moderation.reportable.hidden? && allowed_to?(:hide, permission_resource) %>
+    <% if !moderation.reportable.hidden? && allowed_to?(:hide, authorization_scope) %>
       <%= link_to t("actions.hide", scope: "decidim.moderations"), hide_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary alert" %>
     <% end %>
-    <% if moderation.reportable.hidden? && allowed_to?(:unhide, permission_resource) %>
+    <% if moderation.reportable.hidden? && allowed_to?(:unhide, authorization_scope) %>
       <%= link_to t("actions.unhide", scope: "decidim.moderations"), unhide_moderation_path(id: moderation), method: :put, class: "button button__sm button__secondary alert" %>
     <% end %>
   </div>

data/app/views/decidim/admin/newsletter_templates/index.html.erb

@@ -10,8 +10,8 @@
   <div class="card-section">
     <div class="grid md:grid-cols-2 gap-x-8 p-4">
       <% templates.each do |newsletter_template| %>
-        <div class="card card--mini pb-12 md:pb-0" id="<%= newsletter_template.name %>">
-          <iframe src="<%= preview_newsletter_template_path(newsletter_template.name) %>" class="email-preview w-full h-[60dvh]">
+        <div class="card pb-12 md:pb-0" id="<%= newsletter_template.name %>">
+          <iframe src="<%= preview_newsletter_template_path(newsletter_template.name) %>" class="w-full h-[60dvh]" data-email-preview>
           </iframe>
           <div class="card-footer">
             <h2 class="card-title flex items-center">

data/app/views/decidim/admin/newsletter_templates/show.html.erb

@@ -6,14 +6,14 @@
 </div>
 <div class="item__edit-form">
   <div class="card">
-    <div class="card-section p-4">
-      <iframe src="<%= preview_newsletter_template_path(template_manifest.name) %>" class="email-preview newsletter-template-preview w-full h-[60dvh]">
+    <div class="card-section">
+      <iframe src="<%= preview_newsletter_template_path(template_manifest.name) %>" class="w-full h-[60dvh]" data-email-preview>
       </iframe>
-      <div class="item__edit-sticky">
-        <div class="item__edit-sticky-container">
-          <%= link_to(t(".use_template"), new_newsletter_template_newsletter_path(template_manifest.name), class: "button button__sm button__secondary") %>
-        </div>
-      </div>
+    </div>
+  </div>
+  <div class="item__edit-sticky">
+    <div class="item__edit-sticky-container">
+      <%= link_to(t(".use_template"), new_newsletter_template_newsletter_path(template_manifest.name), class: "button button__sm button__secondary") %>
     </div>
   </div>
 </div>

data/app/views/decidim/admin/newsletters/index.html.erb

@@ -5,7 +5,7 @@
       <%= t ".title" %>
       <div class="flex ml-auto items-center gap-x-4">
         <strong class="font-bold text-md text-gray-2"><%= t("newsletters.index.subscribed_count", scope: "decidim.admin") %></strong>
-        <span class="subscribed_count font-semibold text-md text-gray-2"><%= recipients_count_query %></span>
+        <span data-subscribed-count class="font-semibold text-md text-gray-2"><%= recipients_count_query %></span>
         <% if allowed_to? :create, :newsletter %>
           <%= link_to t("actions.newsletter.new", scope: "decidim.admin"), [:newsletter_templates], class: "button button__sm button__secondary new" %>
         <% end %>