decidim-core 0.28.1 → 0.29.0.rc1

data/app/presenters/decidim/component_attachment_collection_presenter.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+module Decidim
+  #
+  # Decorator attachments of a component acting as an attachment collection
+  #
+  class ComponentAttachmentCollectionPresenter < SimpleDelegator
+    include Decidim::TranslatableAttributes
+
+    def initialize(resource, opts = {})
+      super(resource)
+
+      @include_component_name = opts[:include_component_name]
+    end
+
+    def attachments
+      @attachments ||= if [resource_model_name, resource_table_name].all?(&:present?)
+                         base_query = Decidim::Attachment.joins(resource_join.join_sources)
+                                                         .where(resource_table_name => { decidim_component_id: __getobj__.id })
+                         if resource_model&.include?(Decidim::Publicable)
+                           base_query.where.not(resource_table_name => { published_at: nil })
+                         else
+                           base_query
+                         end
+                       else
+                         Decidim::Attachment.none
+                       end
+    end
+
+    def documents
+      @documents ||= attachments.with_attached_file.order(:weight).select(&:document?)
+    end
+
+    def documents_visible_for(user)
+      return documents unless resource_model.respond_to?(:visible_for)
+
+      attachments.merge(resource_model.visible_for(user)).with_attached_file.order(:weight).select(&:document?)
+    end
+
+    def unused?
+      documents.blank?
+    end
+
+    def id
+      "component-#{__getobj__.id}"
+    end
+
+    def name
+      base_name = I18n.t(manifest_name, scope: "decidim.application.documents.component_documents")
+
+      return base_name unless @include_component_name
+
+      "#{base_name} - #{translated_attribute(__getobj__.name)}"
+    end
+
+    def description; end
+
+    private
+
+    def resource_join
+      resource_table = Arel::Table.new(resource_table_name)
+      attachments_table = Decidim::Attachment.arel_table
+      attachments_table.join(resource_table).on(
+        resource_table[:id].eq(attachments_table[:attached_to_id]),
+        attachments_table[:attached_to_type].eq(resource_model_name)
+      )
+    end
+
+    def resource_model_name
+      @resource_model_name ||= Decidim.resource_registry.find(manifest_name)&.model_class_name
+    end
+
+    def resource_model
+      @resource_model ||= resource_model_name&.safe_constantize
+    end
+
+    def resource_table_name
+      @resource_table_name ||= resource_model&.table_name
+    end
+  end
+end

data/app/presenters/decidim/footer_menu_presenter.rb

@@ -4,7 +4,7 @@ module Decidim
   # A presenter to render footer menu
   class FooterMenuPresenter < MenuPresenter
     def render
-      content_tag(:nav, role: "navigation", "aria-label" => @options[:label]) do
+      content_tag(:nav, :role => "navigation", "aria-label" => @options[:label]) do
         safe_join([content_tag(:h2, @options[:label], class: "h5 mb-4"), render_menu])
       end
     end

data/app/presenters/decidim/home_stats_presenter.rb

@@ -45,7 +45,7 @@ module Decidim
       stats = {}
       Decidim.component_manifests.flat_map do |component|
         component
-          .stats.except([:supports_count])
+          .stats.except([:votes_count])
           .filter(conditions)
           .with_context(published_components)
           .each do |name, data|

data/app/presenters/decidim/log/resource_presenter.rb

@@ -10,6 +10,8 @@ module Decidim
     # overwrite `BasePresenter#resource_presenter` to return your custom resource presenter.
     # The only requirement for custom renderers is that they should respond to `present`.
     class ResourcePresenter
+      include Decidim::SanitizeHelper
+
       # Public: Initializes the presenter.
       #
       # resource - An instance of a model that can be located by
@@ -65,7 +67,11 @@ module Decidim
       #
       # Returns an HTML-safe String.
       def present_resource_name
-        h.translated_attribute extra["title"]
+        if resource.present? && resource.respond_to?(:presenter) && resource.presenter.respond_to?(:title)
+          resource.presenter.title(html_escape: true)
+        else
+          decidim_escape_translated(extra["title"]).html_safe
+        end
       end
     end
   end

data/app/presenters/decidim/organization_presenter.rb

@@ -4,7 +4,7 @@ module Decidim
   # A general presenter to render organization logic to build a manifest
   class OrganizationPresenter < SimpleDelegator
     def html_name
-      name.html_safe
+      translated_attribute(name).html_safe
     end
 
     def translated_description

data/app/queries/decidim/metric_measure.rb

@@ -4,7 +4,7 @@ module Decidim
   # This class executes some measures related to Metrics, and return its results
   #
   #  - day: Date object
-  #  - resource: Object used to make measurements. Object class is dependant within each measure
+  #  - resource: Object used to make measurements. Object class is dependent within each measure
   class MetricMeasure
     def initialize(day, resource)
       @day = day.try(:to_date) || Time.zone.yesterday

data/app/queries/decidim/public_activities.rb

@@ -14,7 +14,7 @@ module Decidim
   #
   # :resource_name - an optional name for a resource for searching only that
   #    type of resources.
-  # :user - an optional `Decidim::User` that performed the activites
+  # :user - an optional `Decidim::User` that performed the activities
   # :current_user - an optional `Decidim::User` for defining whether to show the
   #   private log entries that relate to the current user.
   # :follows - a collection of `Decidim::Follow` resources. It will return any

data/app/queries/decidim/user_groups/admin_memberships.rb

@@ -2,7 +2,7 @@
 
 module Decidim
   module UserGroups
-    # Use this class to find the the admins of the given user group with the
+    # Use this class to find the admins of the given user group with the
     # "admin" role. It returns memberships.
     class AdminMemberships < Decidim::Query
       # Syntactic sugar to initialize the class and return the queried objects.

data/app/queries/decidim/user_groups/member_memberships.rb

@@ -2,7 +2,7 @@
 
 module Decidim
   module UserGroups
-    # Use this class to find the the members of the given user group with the
+    # Use this class to find the members of the given user group with the
     # "member" role. It returns memberships.
     class MemberMemberships < Decidim::Query
       # Syntactic sugar to initialize the class and return the queried objects.

data/app/services/decidim/action_logger.rb

@@ -2,7 +2,7 @@
 
 module Decidim
   # Use this class to log actions by any user. You probably should not
-  # use this class dfirectly, but rather use `Decidim.traceability` instead.
+  # use this class directly, but rather use `Decidim.traceability` instead.
   # Check the docs on `Decidim::Traceability` for more info.
   #
   # Usage:
@@ -14,7 +14,7 @@ module Decidim
     #
     # action - a String representing the name of the action
     # user - the Decidim::User that performed the action
-    # resource - the resource onn which the action was performed
+    # resource - the resource on which the action was performed
     # version_id - the ID of the `PaperTrail::Version` that was created on that action
     # resource_extra - a Hash with resource_extra info to be recorded
     #
@@ -27,7 +27,7 @@ module Decidim
     #
     # action - a String representing the name of the action
     # user - the Decidim::User that performed the action
-    # resource - the resource onn which the action was performed
+    # resource - the resource on which the action was performed
     # version_id - the ID of the `PaperTrail::Version` that was created on that action
     # resource_extra - a Hash with resource_extra info to be recorded
     def initialize(action, user, resource, version_id = nil, resource_extra = {})

data/app/services/decidim/log/diff_changeset_calculator.rb

@@ -20,7 +20,7 @@ module Decidim
     class DiffChangesetCalculator
       # original_changeset - a `changeset` from a PaperTrail::Version instance
       # fields_mapping - a Hash mapping attribute names and a type to render them
-      # i18n_labels_scope - a String representing the I18n scope where the attribtue
+      # i18n_labels_scope - a String representing the I18n scope where the attribute
       #   labels can be found
       def initialize(original_changeset, fields_mapping, i18n_labels_scope)
         @original_changeset = original_changeset
@@ -86,7 +86,7 @@ module Decidim
         locales.flat_map do |locale|
           previous_value = values.first.try(:[], locale)
           new_value = values.last.try(:[], locale)
-          if previous_value == new_value
+          if previous_value == new_value || (previous_value.nil? && new_value.blank?)
             nil
           else
             label = generate_label(attribute, locale)
@@ -119,7 +119,7 @@ module Decidim
       # Generates the label for the given attribute. If the `locale` is set,
       # it appends the locale at the end: `AttributeName (LocaleName)`.
       #
-      # attribute - A Symbol representing the attribute name. It will retrive
+      # attribute - A Symbol representing the attribute name. It will retrieve
       #   this key from the I18n scope set at `i18n_labels_scope`.
       # locale - a String representing the name of the locale.
       #

data/app/services/decidim/notifications_digest_sending_decider.rb

@@ -3,7 +3,7 @@
 module Decidim
   class NotificationsDigestSendingDecider
     class << self
-      def must_notify?(user, time: Time.now.utc)
+      def must_notify?(user, time = Time.now.utc)
         return true if user.digest_sent_at.blank?
 
         # Note that we are checking whether the notifications were sent at any

data/app/uploaders/decidim/organization_favicon_uploader.rb

@@ -26,7 +26,7 @@ module Decidim
     end
 
     def extension_allowlist
-      %w(png jpg jpeg ico)
+      %w(png jpg jpeg webp ico)
     end
   end
 end

data/app/uploaders/decidim/record_image_uploader.rb

@@ -5,11 +5,11 @@ module Decidim
   # limited content types than the defaults.
   class RecordImageUploader < ImageUploader
     def content_type_allowlist
-      %w(image/jpeg image/png)
+      %w(image/jpeg image/png image/webp)
     end
 
     def extension_allowlist
-      %w(jpeg jpg png)
+      %w(jpeg jpg png webp)
     end
   end
 end

data/app/validators/organization_present_validator.rb

@@ -8,7 +8,7 @@ class OrganizationPresentValidator < ActiveModel::Validations::FileContentTypeVa
     return if record.is_a?(Decidim::Organization)
     return if record.respond_to?(:organization) && record.organization.is_a?(Decidim::Organization)
 
-    record.errors.add attribute, I18n.t("carrierwave.errors.not_inside_organization")
+    record.errors.add attribute, I18n.t("decidim.errors.files.not_inside_organization")
   end
 
   def check_validity!; end

data/app/validators/passthru_validator.rb

@@ -69,7 +69,7 @@ class PassthruValidator < ActiveModel::EachValidator
   def validation_record_context(dummy, record)
     if dummy.is_a?(Decidim::Form)
       dummy.with_context(
-        current_organization: record.try(:current_organization) || record.try(:ganization),
+        current_organization: record.try(:current_organization) || record.try(:organization),
         current_participatory_space: record.try(:current_participatory_space) || record.try(:participatory_space),
         current_component: record.try(:current_component) || record.try(:component)
       )

data/app/validators/uploader_image_dimensions_validator.rb

@@ -28,7 +28,7 @@ class UploaderImageDimensionsValidator < ActiveModel::Validations::FileContentTy
     return unless uploader.validable_dimensions
     return if (image = extract_image(file)).blank?
 
-    record.errors.add attribute, I18n.t("carrierwave.errors.file_resolution_too_large") if image.dimensions.any? { |dimension| dimension > uploader.max_image_height_or_width }
+    record.errors.add attribute, I18n.t("decidim.errors.files.file_resolution_too_large") if image.dimensions.any? { |dimension| dimension > uploader.max_image_height_or_width }
   rescue MiniMagick::Error
     # The error may happen because of many reasons but most commonly the image
     # exceeds the default maximum dimensions set for ImageMagick when the
@@ -40,7 +40,7 @@ class UploaderImageDimensionsValidator < ActiveModel::Validations::FileContentTy
     # Note that the error can also happen because of other reasons than only
     # the image dimensions being too large. But as we do not really know the
     # reason every time, we default to that error.
-    record.errors.add attribute, I18n.t("carrierwave.errors.file_cannot_be_processed")
+    record.errors.add attribute, I18n.t("decidim.errors.files.file_cannot_be_processed")
   end
 
   def extract_image(file)

data/app/views/decidim/account/show.html.erb

@@ -4,8 +4,8 @@
 <%= render layout: "layouts/decidim/shared/layout_user_profile" do %>
   <% if current_user.unconfirmed_email.present? %>
     <%# NOTE: announcement cell clean_body method will purge the button %>
-    <div id="email-change-pending" class="flash secondary" data-announcement>
-      <span class="flash__message flex-none block leading-relaxed">
+    <div id="email-change-pending" class="flash warning" data-announcement>
+      <div class="flash__message flex-none block leading-relaxed">
         <p>
           <strong><%= t("decidim.account.email_change.title") %></strong>
         </p>
@@ -18,7 +18,7 @@
             resend_link: link_to(t("decidim.account.email_change.send_again"), resend_confirmation_instructions_account_path, role: :button, method: :post, remote: true),
             cancel_link: link_to(t("decidim.account.email_change.cancel"), cancel_email_change_account_path, role: :button, method: :post, remote: true)) %>
         </p>
-      </span>
+      </div>
     </div>
   <% end %>
 

data/app/views/decidim/amendments/_wizard_header.html.erb

@@ -1,14 +1,13 @@
 <div class="flex justify-center">
   <h1 class="title-decorator my-12">
     <%= wizard_header_title %>
-    <%= "(#{similar_resources_count})" if local_assigns[:similar_resources_count] %>
   </h1>
 </div>
 
 <ol id="wizard-steps" class="wizard-steps">
   <% (1..total_steps).each do |wizard_step| %>
     <% attributes = @current_step == wizard_step ? { "aria-current": "step" } : {} %>
-    <%= content_tag :li, t(wizard_step, scope: "decidim.amendments.wizard_step_form.steps"), attributes %>
+    <%= content_tag :li, t(wizard_step, scope: "decidim.amendments.wizard_step_form.steps"), attributes.merge(class: "w-1/2") %>
   <% end %>
 </ol>
 

data/app/views/decidim/application/_collection.html.erb

@@ -12,7 +12,9 @@
 </button>
 
 <div id="dropdown-menu-documents-<%= attachment_collection.id %>" class="documents__collection-content" aria-hidden="true">
-  <p><%= decidim_escape_translated(attachment_collection.description) %></p>
+  <% if (description = attachment_collection.description).present? %>
+    <p><%= decidim_escape_translated(description) %></p>
+  <% end %>
 
   <div class="documents__container">
     <% documents.each do |document| %>

data/app/views/decidim/application/_component_documents.html.erb

@@ -0,0 +1,4 @@
+<% documents = collection.documents_visible_for(current_user) %>
+<% if documents.any? %>
+  <%= render partial: "decidim/application/collection", formats: [:html], locals: { attachment_collection: collection, documents: } %>
+<% end %>

data/app/views/decidim/application/_document.html.erb

@@ -7,17 +7,31 @@
       <div class="card__list-text"><%= decidim_escape_translated(document.description) %></div>
     <% end %>
     <div class="card__list-metadata">
-      <span><%= icon "file-text-line" %><%= document.file_type %></span>
-      <span><%= icon "scales-2-line" %><%= number_to_human_size(document.file_size) %></span>
+      <div><%= icon "file-text-line" %><%= document.file_type %></div>
+      <% if document.file? %>
+        <div><%= icon "scales-2-line" %><%= number_to_human_size(document.file_size) %></div>
+      <% end %>
     </div>
   </div>
-  <%= link_to document.url,
-              target: "_blank",
-              rel: "noopener noreferrer",
-              class: "button button__sm button__transparent-secondary flex-none",
-              title: t("decidim.application.document.download"),
-              data: { "external-link": false } do %>
-    <%= t("decidim.application.document.download") %>
-    <%= icon "download-line" %>
+  <% if document.file? %>
+    <%= link_to document.url,
+                target: "_blank",
+                rel: "noopener noreferrer",
+                class: "button button__sm button__transparent-secondary flex-none",
+                title: t("decidim.application.document.download"),
+                data: { "external-link": false } do %>
+      <%= t("decidim.application.document.download") %>
+      <%= icon "download-line" %>
+    <% end %>
+  <% else %>
+    <%= link_to document.url,
+                target: "_blank",
+                rel: "noopener noreferrer",
+                class: "button button__sm button__transparent-secondary flex-none",
+                title: t("decidim.application.document.visit_link"),
+                data: { "external-link": true } do %>
+      <%= t("decidim.application.document.visit_link") %>
+    <% end %>
+
   <% end %>
 </div>

data/app/views/decidim/block_user_mailer/notify.html.erb

@@ -1,7 +1,7 @@
 <p class="email-greeting"><%= t ".hello" %></p>
 
-<p class="email-instructions"><%= t ".body_1", organization_name: h(@organization.name) %></p>
+<p class="email-instructions"><%= t ".body_1", organization_name: organization_name(@organization) %></p>
 
 <p class="email-instructions"><%= t ".body_2", justification: h(@justification) %></p>
 
-<p class="email-closing"><%= t(".greetings", organization_name: h(@organization.name), organization_url: decidim.root_url(host: @organization.host)).html_safe %></p>
+<p class="email-closing"><%= t(".greetings", organization_name: organization_name(@organization), organization_url: decidim.root_url(host: @organization.host)).html_safe %></p>

data/app/views/decidim/devise/invitations/edit.html.erb

@@ -14,7 +14,7 @@
     <div class="form__wrapper">
       <%= f.hidden_field :invitation_token %>
 
-      <%= f.text_field :nickname, help_text: t("devise.invitations.edit.nickname_help", organization: current_organization.name), required: "required", autocomplete: "nickname" %>
+      <%= f.text_field :nickname, help_text: t("devise.invitations.edit.nickname_help", organization: current_organization_name), required: "required", autocomplete: "nickname" %>
 
       <% if f.object.class.require_password_on_accepting %>
         <%= render partial: "decidim/account/password_fields", locals: { form: f, user: :user } %>
@@ -24,7 +24,7 @@
     <div class="form__wrapper-block">
       <h4 class="h4"><%= t("tos_title", scope: "decidim.devise.registrations.new") %></h4>
 
-      <div class="tos-text">
+      <div>
         <% terms_of_service_summary_content_blocks.each do |content_block| %>
           <%= cell content_block.manifest.cell, content_block %>
         <% end %>

data/app/views/decidim/devise/omniauth_registrations/new.html.erb

@@ -18,7 +18,7 @@
     <div class="form__wrapper">
       <%= f.text_field :name, help_text: t("decidim.devise.omniauth_registrations.new.username_help"), autocomplete: "name", placeholder: "John Doe" %>
 
-      <%= f.text_field :nickname, help_text: t("decidim.devise.omniauth_registrations.new.nickname_help", organization: current_organization.name), autocomplete: "nickname", placeholder: "johndoe" %>
+      <%= f.text_field :nickname, help_text: t("decidim.devise.omniauth_registrations.new.nickname_help", organization: current_organization_name), autocomplete: "nickname", placeholder: "johndoe" %>
 
       <%= f.email_field :email, autocomplete: "email", placeholder: t("placeholder_email", scope: "decidim.devise.shared") %>
 

data/app/views/decidim/devise/registrations/new.html.erb

@@ -43,7 +43,7 @@
     <div id="card__tos" class="form__wrapper-block">
       <h2 class="h4"><%= t("decidim.devise.registrations.new.tos_title") %></h2>
 
-      <div class="tos-text">
+      <div>
         <% terms_of_service_summary_content_blocks.each do |content_block| %>
           <%= cell content_block.manifest.cell, content_block %>
         <% end %>

data/app/views/decidim/devise/shared/_login_boxes.html.erb

@@ -1,6 +1,6 @@
 <div class="login__box-container">
   <button type="button" class="login__box" data-dialog-open="loginModal">
-    <%= icon "login-box-line", class: "login__box-icon" %>
+    <%= icon "user-line", class: "login__box-icon" %>
     <p class="login__box-title">
       <%= t("already_have_an_account?", scope: ) %>
     </p>

data/app/views/decidim/doorkeeper/authorizations/new.html.erb

@@ -8,7 +8,7 @@
 
   <div class="login__info">
     <p>
-      <%= t("connect_your_account_html", scope: "decidim.doorkeeper.authorizations.new", organization: current_organization.name) %>
+      <%= t("connect_your_account_html", scope: "decidim.doorkeeper.authorizations.new", organization: current_organization_name) %>
     </p>
 
     <p><%= t("this_application_will_be_able_to", scope: "decidim.doorkeeper.authorizations.new") %></p>

data/app/views/decidim/endorsements/_identity.html.erb

@@ -1,4 +1,4 @@
-<button type="button" class="<%= selected ? "is-selected" : "" %>" data-user-identity data-method="<%= http_method.to_s.upcase %>" data-create_url="<%= create_url %>" data-destroy_url="<%= destroy_url %>">
+<button type="button" class="<%= "is-selected" if selected %>" data-user-identity data-method="<%= http_method.to_s.upcase %>" data-create_url="<%= create_url %>" data-destroy_url="<%= destroy_url %>">
   <%== cell "decidim/author", identity, skip_profile_link: true %>
   <%= icon("checkbox-circle-line") %>
 </button>

data/app/views/decidim/endorsements/update_buttons_and_counters.js.erb

@@ -1,5 +1,4 @@
 updateEndorsementBlock();
-
 function updateEndorsementBlock() {
   var $endorsementBlock = $('#resource-<%= resource.id %>-endorsement-block');
   var $endorsementListTrigger = $('#resource-<%= resource.id %>-endorsement-block #dropdown-trigger');
@@ -24,4 +23,6 @@ function updateEndorsementBlock() {
   } else {
     $endorsementIdentitiesButton[0].innerHTML = '<%= j(cell("decidim/endorsement_buttons", resource).button_content).strip.html_safe %>';
   }
+  document.dispatchEvent(new CustomEvent("ajax:loaded", { detail: $endorsementBlock[0] }));
+  document.dispatchEvent(new CustomEvent("ajax:loaded", { detail: $endorsementListGrid[0] }));
 }

data/app/views/decidim/gamification/badges/index.html.erb

@@ -1,40 +1,41 @@
-<div class="wrapper">
-  <div class="row column mb-m">
-    <h1><%= t ".title" %></h1>
-    <p>
-      <%= t ".page_description" %>
+<% add_decidim_meta_tags(
+  title: t(".title"),
+  description: t(".page_description")
+) %>
+
+<main class="layout-1col cols-10">
+
+  <header class="text-center py-10">
+    <h1 class="title-decorator inline-block text-left mb-12">
+      <%= t ".title" %>
+    </h1>
+    <p class="text-lg text-gray-2">
+    <%= t ".page_description" %>
     </p>
-  </div>
+  </header>
+
   <% @badges.each do |badge| %>
-    <div class="row column mb-m">
-      <div class="column medium-2">
-        <div class="card card--badge absolutes">
-          <div class="card__content">
-            <div class="card__header">
-              <div class="badge-container">
-                <%= image_tag badge.image %>
-              </div>
-            </div>
-            <div class="card__text text-center">
-              <strong><%= badge.translated_name %></strong>
-            </div>
+    <div class="mb-12">
+      <h2 class="h4 mb-4"><%= t ".badge_title", name: badge.translated_name %></h2>
+      <div class="grid grid-cols-10 text-gray-2 leading-relaxed">
+        <div class="col-span-2">
+          <div class="flex justify-center items-center">
+            <%= image_tag badge.image, class: "w-32", alt: badge.translated_name %>
+          </div>
+          <div class="text-center mt-4">
+            <strong><%= badge.translated_name %></strong>
           </div>
         </div>
-      </div>
-      <div class="column medium-10">
-        <h6>
-          <strong><%= t ".badge_title", name: badge.translated_name %></strong>
-        </h6>
-        <p><%= badge.description(current_organization.name) %></p>
-        <h6>
-          <strong><%= t ".how" %></strong>
-        </h6>
-        <ol>
-          <% badge.conditions.each do |condition| %>
-            <li><%= condition %></li>
-          <% end %>
-        </ol>
+        <div class="col-span-7 ml-2">
+          <p><%= badge.description(current_organization_name) %></p>
+          <p class="mt-4"><strong><%= t ".how" %></strong></p>
+          <ol class="list-decimal">
+            <% badge.conditions.each do |condition| %>
+              <li><%= condition %></li>
+            <% end %>
+          </ol>
+        </div>
       </div>
     </div>
   <% end %>
-</div>
+</main>

data/app/views/decidim/links/_modal.html.erb

@@ -16,6 +16,6 @@
     <button class="button button__lg button__transparent-secondary" data-dialog-close="external-domain-warning">
       <%= t("decidim.links.warning.cancel") %>
     </button>
-    <%= link_to t("decidim.links.warning.proceed"), external_url.to_s , target: "_blank", class: "button button__lg button__secondary" %>
+    <%= link_to t("decidim.links.warning.proceed"), external_url.to_s , target: "_blank", class: "button button__lg button__secondary", rel: "nofollow noopener noreferrer" %>
   </div>
 <% end %>

data/app/views/decidim/links/new.html.erb

@@ -1,3 +1,5 @@
+<% add_decidim_page_title t("decidim.links.warning.title") %>
+
 <%= render layout: "layouts/decidim/shared/layout_center" do %>
 
   <div class="flex justify-center">
@@ -12,7 +14,7 @@
   </code>
 
   <div class="form__wrapper-block">
-    <%= link_to t("decidim.links.warning.proceed"), external_url.to_s, class: "button button__lg button__secondary ml-auto" %>
+    <%= link_to t("decidim.links.warning.proceed"), external_url.to_s, class: "button button__lg button__secondary ml-auto", rel: "nofollow noopener noreferrer" %>
   </div>
 
 <% end %>

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

@@ -5,7 +5,7 @@
   "display": "<%= organization_params.pwa_display %>",
   "start_url": "<%= organization_params.start_url %>",
   "theme_color": "<%= organization_params.colors["primary"] %>",
-  "background_color": "<%= organization_params.colors["primary"] %>",
+  "background_color": "#FFFFFF",
   "icons": [
     {
       "src": "<%= current_organization.attached_uploader(:favicon).variant_path :small %>",

data/app/views/decidim/messaging/conversations/_add_conversation_users.html.erb

@@ -12,10 +12,10 @@
           autocomplete="off"
           placeholder="<%= t("add_users_placeholder", scope: "decidim.messaging.conversations.add_conversation_users") %>"
           data-noresults="<%= t("no_results", scope: "decidim.messaging.conversations.add_conversation_users") %>"
-          data-selected="multiple-mentions-selected"
+          data-selected="conversation__modal-results"
           contenteditable="true"
           data-direct-messages-disabled="<%= t("participant_with_disabled_message_reception", scope: "decidim.messaging.conversations.add_conversation_users") %>">
     </div>
-    <ul class="conversation__modal-results multiple-mentions-selected"></ul>
+    <ul class="conversation__modal-results"></ul>
   </div>
 </div>