decidim-core 0.29.0.rc3 → 0.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d58ac45e047a62b04a7625b6da1c4ee92c2a62d9bf4e1969cdfb324257e5ef9
-  data.tar.gz: 94a06bf8519d6234945a39dc86cc9794149fa3dc09c89273dcd475a64a3ad78b
+  metadata.gz: 37f39b870aa9a840fbf4c415858b3210d520511f8efb0327bb90e8aa0126f243
+  data.tar.gz: 16fa93602fc35d288ddeef831f50f3e3878714ea4fae6547064c9be0a148fad0
 SHA512:
-  metadata.gz: 6e90186e86e894822888b13cbaabe6c8c2a752d73a477cb13b39be5fd785d9b92082e5e403de91790f43c19a78f83bb85253e4fa0475f086f64e90258b2593b3
-  data.tar.gz: f297c4b6a5fad64c57984fc063778cddda2404f3861e75b8a16c4fff83986caedc44354bba11a58551c5815dae0c9785b5f8d159a9a7e0930d2f080341b69579
+  metadata.gz: 0ae69c163d580612c4a8f4f4cf8d9c460aebc24270a67f4a245d25f4fceb2988075664aab8f97977ae19b75d90c42e1cdb03ad2591bd546c5acf9f8562d5852f
+  data.tar.gz: 23132fff12f59d0bb72b073e6729ccb5f439f5c59493d8a3f8f7026dbb45cf20e56f1cac34b07bc8a2d10e757b4cd6efe60bc93180f22d2a4764db597913a1b6

data/app/cells/decidim/address/online.erb

@@ -5,8 +5,8 @@
       <div class="address">
         <div class="address__location"><%= t(model.type_of_meeting, scope: "decidim.meetings.meetings.filters.type_values") %></div>
         <% if display_online_meeting_url? %>
-          <a href="<%= model.online_meeting_url %>" target="_blank" rel="noopener noreferrer" class="address__hints underline break-all">
-            <%= model.online_meeting_url %>
+          <a href="<%= online_meeting_url %>" target="_blank" rel="noopener noreferrer" class="address__hints underline break-all">
+            <%= online_meeting_url %>
           <% end %>
           </a>
       </div>

data/app/cells/decidim/address_cell.rb

@@ -43,6 +43,10 @@ module Decidim
       HTML
     end
 
+    def online_meeting_url
+      URI::Parser.new.escape(model.online_meeting_url)
+    end
+
     def display_online_meeting_url?
       return true unless model.respond_to?(:online?)
       return true unless model.respond_to?(:iframe_access_level_allowed_for_user?)

data/app/cells/decidim/card_g/show.erb

@@ -1,7 +1,7 @@
 <%= link_to resource_path, class: classes[:default], id: resource_id do %>
   <div class="<%= classes[:img] %>">
     <% if has_image? %>
-      <%= image_tag resource_image_path, alt: alt_title %>
+      <%= image_tag resource_image_url, alt: alt_title %>
     <% else %>
       <%= external_icon "media/images/placeholder-card-g.svg", class: "card__placeholder-g" %>
     <% end %>

data/app/cells/decidim/card_g_cell.rb

@@ -52,12 +52,15 @@ module Decidim
       @id_base_name ||= resource.class.name.gsub(/\ADecidim::/, "").underscore.split("/").join("__")
     end
 
-    def resource_image_path
+    def resource_image_url
+      # Backwards compatibility.
+      return resource_image_path if respond_to?(:resource_image_path)
+
       nil
     end
 
     def has_image?
-      resource_image_path.present?
+      resource_image_url.present?
     end
 
     def show_description?

data/app/cells/decidim/card_l/image.erb

@@ -1,6 +1,6 @@
 <div class="card__list-image">
-  <% if resource_image_path.present? %>
-    <%= image_tag resource_image_path, class: "w-full h-full object-cover" %>
+  <% if has_image? %>
+    <%= image_tag resource_image_url, class: "w-full h-full object-cover" %>
   <% else %>
     <div class="w-full h-full relative">
       <div class="w-full h-full bg-primary opacity-10 absolute top-0 left-0 z-10">

data/app/cells/decidim/card_l_cell.rb

@@ -66,12 +66,15 @@ module Decidim
       "#{class_base_name}__#{class_name}"
     end
 
-    def resource_image_path
+    def resource_image_url
+      # Backwards compatibility.
+      return resource_image_path if respond_to?(:resource_image_path)
+
       nil
     end
 
     def has_image?
-      resource_image_path.present?
+      resource_image_url.present?
     end
 
     def has_link_to_resource?

data/app/cells/decidim/content_blocks/hero_cell.rb

@@ -15,7 +15,7 @@ module Decidim
       end
 
       def background_image
-        model.images_container.attached_uploader(:background_image).path(variant: :big)
+        model.images_container.attached_uploader(:background_image).variant_url(:big)
       end
 
       private

data/app/cells/decidim/content_blocks/highlighted_content_banner/show.erb

@@ -1,4 +1,4 @@
-<section id="highlighted_content_banner" class="home__section-image" style="--hero-image:url('<%= current_organization.attached_uploader(:highlighted_content_banner_image).path %>');">
+<section id="highlighted_content_banner" class="home__section-image" style="--hero-image:url('<%= current_organization.attached_uploader(:highlighted_content_banner_image).url %>');">
   <div class="home__section-content-banner home__section">
     <div>
       <h2 class="home__section-content-banner__title">

data/app/cells/decidim/content_blocks/participatory_space_hero_cell.rb

@@ -32,9 +32,9 @@ module Decidim
       # If it is called from the landing page content block, use the background image defined there
       # Else, use the banner image defined in the space (for assemblies)
       def image_path
-        return model.images_container.attached_uploader(:background_image).path if model.respond_to?(:images_container)
+        return model.images_container.attached_uploader(:background_image).url if model.respond_to?(:images_container)
 
-        attached_uploader(:banner_image).path
+        attached_uploader(:banner_image).url
       end
 
       def has_hashtag?

data/app/cells/decidim/nav_links/show.erb

@@ -1,12 +1,12 @@
 <div class="participatory-space__nav-container">
-  <button id="dropdown-trigger-participatory-space" data-component="dropdown" data-target="dropdown-menu-participatory-space" data-auto-close="true" data-scroll-to-menu="true">
+  <button id="dropdown-trigger-participatory-space" data-component="dropdown" data-target="dropdown-menu-participatory-space" data-auto-close="true" data-scroll-to-menu="true" data-open-md="true">
     <span><%= t("decidim.searches.filters.jump_to") %></span>
     <%= icon "arrow-down-s-line" %>
     <%= icon "arrow-up-s-line" %>
   </button>
-  <ul id="dropdown-menu-participatory-space" class="participatory-space__nav" aria-hidden="true">
+  <ul id="dropdown-menu-participatory-space" class="participatory-space__nav">
     <% model.each do |item| %>
-      <li>
+      <li role="menuitem">
         <%= link_to item[:url], class: "participatory-space__nav-item" do %>
           <%= item[:name] %>
           <%= icon "arrow-right-line" %>

data/app/cells/decidim/resource_types_filter/show.erb

@@ -1,5 +1,5 @@
 <div id="<%= id %>" class="filter-container">
-  <button id="dropdown-trigger-resource" data-component="dropdown" data-target="dropdown-menu-resource" data-auto-close="true">
+  <button id="dropdown-trigger-resource" data-component="dropdown" data-target="dropdown-menu-resource" data-open-md="true">
     <% resource_types.each do |resource_type| %>
       <span data-value="<%= resource_type[0] %>" class="<%= "is-active" if filter_param == resource_type[0] %>">
         <%= text_with_resource_icon(*resource_type) %>
@@ -8,15 +8,14 @@
     <%= icon "arrow-down-s-line" %>
     <%= icon "arrow-up-s-line" %>
   </button>
-  <%= filter_form_for filter, form_path, :class => "new_filter", :id => "dropdown-menu-resource", "aria-hidden" => true do |form| %>
-    <%= form.collection_radio_buttons(
-          filter_param_key,
-          resource_types,
-          :first,
-          :last,
-          { checked: filter_param }
-        ) do |builder|
-            builder.label { builder.radio_button(class: "reset-defaults", hidden: true) + content_tag(:span, text_with_resource_icon(builder.value, builder.text), class: "filter") }
-      end %>
-  <% end %>
+  <ul id="dropdown-menu-resource">
+    <% resource_types.each do |resource_type| %>
+      <li role="menuitem">
+        <%= link_to decidim.last_activities_path(filter: { with_resource_type: resource_type[0] } ), class: "filter#{" is-active" if filter_param == resource_type[0]}" do %>
+          <span class="sr-only"><%= resource_type[1] %></span>
+          <%= text_with_resource_icon(*resource_type) %>
+        <% end %>
+      </li>
+    <% end %>
+  </ul>
 </div>

data/app/commands/decidim/create_omniauth_registration.rb

@@ -55,7 +55,12 @@ module Decidim
         # If user has left the account unconfirmed and later on decides to sign
         # in with omniauth with an already verified account, the account needs
         # to be marked confirmed.
-        @user.skip_confirmation! if !@user.confirmed? && @user.email == verified_email
+        if !@user.confirmed? && @user.email == verified_email
+          @user.skip_confirmation!
+          @user.after_confirmation
+        end
+        @user.tos_agreement = "1"
+        @user.save!
       else
         @user.email = (verified_email || form.email)
         @user.name = form.name
@@ -69,10 +74,11 @@ module Decidim
           @user.avatar.attach(io: file, filename:)
         end
         @user.skip_confirmation! if verified_email
-      end
+        @user.tos_agreement = "1"
+        @user.save!
 
-      @user.tos_agreement = "1"
-      @user.save!
+        @user.after_confirmation if verified_email
+      end
     end
 
     def create_identity

data/app/controllers/concerns/decidim/devise_controllers.rb

@@ -21,6 +21,7 @@ module Decidim
       include Decidim::SafeRedirect
       include NeedsSnippets
       include UserBlockedChecker
+      include ActiveStorage::SetCurrent
 
       helper Decidim::TranslationsHelper
       helper Decidim::MetaTagsHelper

data/app/controllers/concerns/decidim/paginable.rb

@@ -7,7 +7,7 @@ module Decidim
   module Paginable
     extend ActiveSupport::Concern
 
-    OPTIONS = [10, 20, 50, 100].freeze
+    OPTIONS = [25, 50, 100].freeze
 
     included do
       helper_method :per_page, :page_offset

data/app/controllers/decidim/application_controller.rb

@@ -24,6 +24,7 @@ module Decidim
     include DisableRedirectionToExternalHost
     include NeedsPasswordChange
     include LinkedResourceReference
+    include ActiveStorage::SetCurrent
 
     helper Decidim::MetaTagsHelper
     helper Decidim::DecidimFormHelper

data/app/helpers/decidim/menu_helper.rb

@@ -57,7 +57,7 @@ module Decidim
         self,
         element_class: "font-semibold underline",
         active_class: "is-active",
-        container_options: { class: "space-y-4 break-inside-avoid" },
+        container_options: { class: "space-y-4 break-inside-avoid", role: :menu },
         label: t("layouts.decidim.footer.decidim_title")
       )
     end

data/app/helpers/decidim/paginate_helper.rb

@@ -11,16 +11,14 @@ module Decidim
     def decidim_paginate(collection, paginate_params = {})
       return if collection.total_pages <= 1
 
+      per_page = (params[:per_page] || paginate_params[:per_page] || Decidim::Paginable::OPTIONS.first).to_i
+
       content_tag :div, class: "flex flex-col-reverse md:flex-row items-center justify-between gap-1 py-8 md:py-16", data: { pagination: "" } do
         template = ""
-        template += render(partial: "decidim/shared/results_per_page", formats: [:html]) if collection.total_pages.positive?
+        template += render(partial: "decidim/shared/results_per_page", locals: { per_page: }, formats: [:html]) if collection.total_pages.positive?
         template += paginate collection, window: 2, outer_window: 1, theme: "decidim", params: paginate_params
         template.html_safe
       end
     end
-
-    def per_page
-      params[:per_page].to_i || Decidim::Paginable::OPTIONS.first
-    end
   end
 end

data/app/models/decidim/attachment.rb

@@ -84,11 +84,12 @@ module Decidim
     #
     # Returns String.
     def url
-      if file?
-        attached_uploader(:file).path
-      elsif link?
-        link
-      end
+      @url ||=
+        if file?
+          attached_uploader(:file).url
+        elsif link?
+          link
+        end
     end
 
     # The URL to download the thumbnail of the file. Only works with images.
@@ -97,7 +98,7 @@ module Decidim
     def thumbnail_url
       return unless photo?
 
-      attached_uploader(:file).path(variant: :thumbnail)
+      @thumbnail_url ||= attached_uploader(:file).variant_url(:thumbnail)
     end
 
     # The URL to download the a big version of the file. Only works with images.
@@ -106,7 +107,7 @@ module Decidim
     def big_url
       return unless photo?
 
-      attached_uploader(:file).path(variant: :big)
+      @big_url ||= attached_uploader(:file).variant_url(:big)
     end
 
     def set_content_type_and_size

data/app/models/decidim/component.rb

@@ -14,7 +14,10 @@ module Decidim
 
     belongs_to :participatory_space, polymorphic: true
 
-    default_scope { order(arel_table[:weight].asc, arel_table[:manifest_name].asc) }
+    scope :registered_component_manifests, -> { where(manifest_name: Decidim.component_registry.manifests.collect(&:name)) }
+    scope :registered_space_manifests, -> { where(participatory_space_type: Decidim.participatory_space_registry.manifests.collect(&:model_class_name)) }
+
+    default_scope { registered_component_manifests.registered_space_manifests.order(arel_table[:weight].asc, arel_table[:manifest_name].asc) }
 
     delegate :organization, :categories, to: :participatory_space
 

data/app/models/decidim/content_block.rb

@@ -60,8 +60,8 @@ module Decidim
     #
     #     # This is how you can access the image data, just like with any other
     #     # uploader field. You can use the uploader variants too.
-    #     content_block.images_container.attached_uploader(:my_image).path
-    #     content_block.images_container.attached_uploader(:my_image).path(variant: :big)
+    #     content_block.images_container.attached_uploader(:my_image).url
+    #     content_block.images_container.attached_uploader(:my_image).variant_url(:big)
     #
     #     # This will delete the attached image
     #     content_block.images_container.my_image = nil

data/app/models/decidim/user.rb

@@ -278,6 +278,18 @@ module Decidim
       false
     end
 
+    def after_confirmation
+      return unless organization.send_welcome_notification?
+
+      Decidim::EventsManager.publish(
+        event: "decidim.events.core.welcome_notification",
+        event_class: WelcomeNotificationEvent,
+        resource: self,
+        affected_users: [self],
+        extra: { force_email: true }
+      )
+    end
+
     protected
 
     # Overrides devise email required validation.
@@ -296,18 +308,6 @@ module Decidim
       super
     end
 
-    def after_confirmation
-      return unless organization.send_welcome_notification?
-
-      Decidim::EventsManager.publish(
-        event: "decidim.events.core.welcome_notification",
-        event_class: WelcomeNotificationEvent,
-        resource: self,
-        affected_users: [self],
-        extra: { force_email: true }
-      )
-    end
-
     private
 
     # Changes default Devise behaviour to use ActiveJob to send async emails.

data/app/packs/src/decidim/a11y.js

@@ -59,7 +59,6 @@ const createDropdown = (component) => {
   const dropdownOptions = {};
   dropdownOptions.dropdown = component.dataset.target;
   dropdownOptions.hover = component.dataset.hover === "true";
-  dropdownOptions.isOpen = component.dataset.open === "true";
   dropdownOptions.autoClose = component.dataset.autoClose === "true";
 
   // This snippet allows to disable the dropdown based on the current viewport
@@ -78,6 +77,17 @@ const createDropdown = (component) => {
     return
   }
 
+  dropdownOptions.isOpen = component.dataset.open === "true";
+
+  const isOpen = Object.keys(screens).some((key) => {
+    if (!isScreenSize(key)) {
+      return false;
+    }
+    return Boolean(component.dataset[`open-${key}`.replace(/-([a-z])/g, (str) => str[1].toUpperCase())]);
+  });
+
+  dropdownOptions.isOpen = dropdownOptions.isOpen || isOpen;
+
   if (!component.id) {
     // when component has no id, we enforce to have it one
     component.id = `dropdown-${Math.random().toString(36).substring(7)}`
@@ -104,20 +114,6 @@ const createDropdown = (component) => {
     });
   }
 
-  // Disable focus on children elements so we can pass the AXE accessibility tests
-  const dropdownMenu = document.getElementById(dropdownOptions.dropdown);
-  if (dropdownMenu.getAttribute("aria-hidden") === "true") {
-    dropdownMenu.
-      querySelectorAll("a, input, button").
-      forEach((element) => { element.tabIndex = -1 })
-  }
-
-  component.addEventListener("click", () => {
-    dropdownMenu.
-      querySelectorAll("a, input, button").
-      forEach((element) => { element.tabIndex = 0 })
-  })
-
   Dropdowns.render(component.id, dropdownOptions);
 }
 

data/app/packs/src/decidim/attachments/file_or_link_tabs.js

@@ -36,10 +36,14 @@ const initializeTabs = (container) => {
     updateTabsState(container);
   });
 
-  uploadsContainer.addEventListener("DOMSubtreeModified", () => {
-    updateTabsState(container);
-    console.log("DOMSubtreeModified");
+  const observer = new MutationObserver((mutationsList) => {
+    mutationsList.forEach((mutation) => {
+      if (mutation.type === "childList") {
+        updateTabsState(container);
+      }
+    });
   });
+  observer.observe(uploadsContainer, {childList: true, subtree: true});
 
   updateTabsState(container);
 };

data/app/packs/src/decidim/input_character_counter.js

@@ -74,7 +74,7 @@ export default class InputCharacterCounter {
 
       // If input is a hidden for WYSIWYG editor add it at the end
       if (this.$input.parent().is(".editor")) {
-        this.$input.parent().after(this.$target);
+        this.$input.parent().append(container);
       } else {
         const wrapper = document.createElement("span")
         wrapper.className = "input-character-counter"

data/app/packs/stylesheets/decidim/_dropdown.scss

@@ -72,6 +72,15 @@
 .dropdown {
   @apply absolute border-2 border-gray-3 rounded min-w-max p-4 drop-shadow-md text-left z-10;
 
+  /*
+    NOTE: the calculated value is the sum of the arrow offset position plus the half of the arrow size:
+    - offset position: 20%
+    - arrow size: 1.5rem
+  */
+  --arrow-offset: 20%;
+  --arrow-size: 1.5rem;
+  --arrow-visible-size: var(--arrow-size) * 0.5;
+
   & > * {
     @apply relative z-10 p-3.5 first:pt-1.5 last:pb-1.5;
   }
@@ -90,15 +99,6 @@
     }
   }
 
-  /*
-    NOTE: the calculated value is the sum of the arrow offset position plus the half of the arrow size:
-    - offset position: 20%
-    - arrow size: 1.5rem
-  */
-  --arrow-offset: 20%;
-  --arrow-size: 1.5rem;
-  --arrow-visible-size: var(--arrow-size) * 0.5;
-
   &__bottom {
     @apply top-full right-0 mt-3 translate-x-[calc(var(--arrow-offset)-var(--arrow-visible-size))] before:content-[''] before:absolute before:right-[var(--arrow-offset)] before:-top-2 before:w-[var(--arrow-size)] before:h-[var(--arrow-size)] before:rotate-45 before:bg-white before:rounded before:border-2 before:border-gray-3 after:content-[''] after:absolute after:left-0 after:top-0 after:w-full after:h-full after:bg-white;
   }

data/app/packs/stylesheets/decidim/_filters.scss

@@ -1,6 +1,8 @@
 .filter {
+  @apply flex gap-2 p-1.5 relative;
+
   label {
-    @apply flex items-center gap-2 p-1.5 rounded cursor-pointer relative;
+    @apply flex items-center gap-2 p-1.5 rounded cursor-pointer relative w-full;
   }
 
   &-container {

data/app/packs/stylesheets/decidim/_footer.scss

@@ -3,7 +3,7 @@ footer {
     @apply bg-gray-4;
 
     &__top {
-      @apply hidden flex flex-col lg:flex-row lg:block gap-8 container py-10;
+      @apply hidden lg:flex flex-row gap-8 container py-10;
     }
 
     &__down {

data/app/packs/stylesheets/decidim/_forms.scss

@@ -53,16 +53,16 @@
   }
 
   select {
-    &:not(.reset-defaults) {
-      @apply pr-8;
-    }
-
     @apply appearance-none;
 
     background-image: url("data:image/svg+xml,%3csvg xmlns='http://www.w3.org/2000/svg' fill='none' viewBox='0 0 12 8'%3e%3cpath fill='%233E4C5C' d='M5.99962 4.97656L10.1246 0.851562L11.303 2.0299L5.99962 7.33323L0.696289 2.0299L1.87462 0.851562L5.99962 4.97656Z'/%3e%3c/svg%3e");
     background-position: right 1rem center;
     background-repeat: no-repeat;
     background-size: 0.75rem;
+
+    &:not(.reset-defaults) {
+      @apply pr-8;
+    }
   }
 }
 

data/app/packs/stylesheets/decidim/_tooltip.scss

@@ -1,6 +1,16 @@
 [role="tooltip"] {
   @apply absolute bg-black z-10 px-4 py-2 w-max max-w-xs rounded text-left text-white;
 
+  /*
+    NOTE: the calculated value is the sum of the arrow offset position plus the half of the arrow size:
+    - offset position: 20%
+    - arrow size: 16px
+  */
+  --arrow-offset: 20%;
+  --arrow-size: 16px;
+  --arrow-visible-size: var(--arrow-size) * 0.5;
+  --arrow-margin: var(--arrow-visible-size) * 1.4142135623730951; // due to the rotation, the margin is SQRT2 times the visible size
+
   & > * {
     @apply relative z-20;
   }
@@ -41,16 +51,6 @@
     }
   }
 
-  /*
-    NOTE: the calculated value is the sum of the arrow offset position plus the half of the arrow size:
-    - offset position: 20%
-    - arrow size: 16px
-  */
-  --arrow-offset: 20%;
-  --arrow-size: 16px;
-  --arrow-visible-size: var(--arrow-size) * 0.5;
-  --arrow-margin: var(--arrow-visible-size) * 1.4142135623730951; // due to the rotation, the margin is SQRT2 times the visible size
-
   &.top {
     @apply -translate-x-[calc(100%-var(--arrow-offset))] -translate-y-[calc(100%+var(--arrow-margin))] before:content-[''] before:absolute before:-z-10 before:right-[calc(var(--arrow-offset)-var(--arrow-visible-size))] before:-bottom-[var(--arrow-visible-size)] before:w-[var(--arrow-size)] before:h-[var(--arrow-size)] before:rotate-45 before:bg-black before:rounded-br;
   }

data/app/packs/stylesheets/decidim/editor.scss

@@ -21,7 +21,7 @@
 }
 
 .editor-container {
-  @apply editor-props editor-suggestions-props flex flex-col mb-6 border editor-border;
+  @apply editor-props editor-suggestions-props flex flex-col mt-4 border editor-border;
 
   &.editor-disabled {
     .editor-input .ProseMirror {

data/app/presenters/decidim/menu_item_presenter.rb

@@ -26,7 +26,7 @@ module Decidim
     delegate :content_tag, :safe_join, :link_to, :active_link_to_class, :is_active_link?, :icon, to: :@view
 
     def render
-      content_tag :li, class: link_wrapper_classes do
+      content_tag :li, role: :menuitem, class: link_wrapper_classes do
         output = if url == "#"
                    [content_tag(:span, composed_label, class: "sidebar-menu__item-disabled")]
                  else

data/app/services/decidim/open_data_exporter.rb

@@ -51,13 +51,14 @@ module Decidim
             headers.push(*exporter.headers)
             exported = exporter.export
 
-            tmpfile = Tempfile.new("#{export_manifest.name}-#{component.id}-")
-            tmpfile.write(exported.read)
-            # Do not delete the file when the reference is deleted
-            ObjectSpace.undefine_finalizer(tmpfile)
-            tmpfile.close
-
-            collection.push(tmpfile.path)
+            tmpdir = Dir::Tmpname.create(export_manifest.name.to_s) do
+              # just get an empty file name
+            end
+            filename = File.join(tmpdir, "#{component.id}.csv")
+            Dir.mkdir(tmpdir)
+            File.write(filename, exported.read)
+
+            collection.push(filename)
           end
         end
       end

data/app/views/decidim/manifests/show.json.erb

@@ -8,22 +8,22 @@
   "background_color": "#FFFFFF",
   "icons": [
     {
-      "src": "<%= current_organization.attached_uploader(:favicon).variant_path :small %>",
+      "src": "<%= current_organization.attached_uploader(:favicon).variant_url :small %>",
       "sizes": "32x32",
       "type": "image/png"
     },
     {
-      "src": "<%= current_organization.attached_uploader(:favicon).variant_path :medium %>",
+      "src": "<%= current_organization.attached_uploader(:favicon).variant_url :medium %>",
       "sizes": "180x180",
       "type": "image/png"
     },
     {
-      "src": "<%= current_organization.attached_uploader(:favicon).variant_path :big %>",
+      "src": "<%= current_organization.attached_uploader(:favicon).variant_url :big %>",
       "sizes": "192x192",
       "type": "image/png"
     },
     {
-      "src": "<%= current_organization.attached_uploader(:favicon).variant_path :huge %>",
+      "src": "<%= current_organization.attached_uploader(:favicon).variant_url :huge %>",
       "sizes": "512x512",
       "type": "image/png"
     }

data/app/views/decidim/pages/_tabbed.html.erb

@@ -11,16 +11,16 @@
 
   <div class="vertical-tabs">
     <nav>
-      <button id="dropdown-trigger-pages" data-component="dropdown" data-target="dropdown-menu-pages" data-auto-close="true">
+      <button id="dropdown-trigger-pages" data-component="dropdown" data-target="dropdown-menu-pages" data-open-md="true" data-auto-close="true">
         <span>
           <%= translated_attribute(page.title) %>
         </span>
         <%= icon "arrow-down-s-line" %>
         <%= icon "arrow-up-s-line" %>
       </button>
-      <ul id="dropdown-menu-pages" class="vertical-tabs__list" aria-hidden="true">
+      <ul id="dropdown-menu-pages" class="vertical-tabs__list" role="menu">
         <% pages.each do |sibling| %>
-          <li class="<%= "is-active" if page == sibling %>">
+          <li class="<%= "is-active" if page == sibling %>" role="menuitem">
             <%= link_to translated_attribute(sibling.title), page_path(sibling.slug) %>
           </li>
         <% end %>