decidim-core 0.21.0 → 0.22.0

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

@@ -1,24 +1,27 @@
-<article class="card card--widget">
-  <ul class="card-data">
-    <li class="card-data__item">
-      <div class="card__link text-center">
-        <%= resource_icon resource, class: "icon--large" %>
-        <span class="text-medium mt-xs"><%= created_at %></span>
-      </div>
-    </li>
-    <li class="card-data__item card-data__item--expand">
-      <div>
-        <%= author %>
+<div class="card card--activity">
+  <div class="card__content">
+    <div class="card__text">
+      <div class="text-medium">
         <% if title.present? %>
-          <span class="text-small"><%= title %></span>
-        <% end %>
-        <%= link_to resource_link_path, class: "card__link" do %>
-          <h5 class="card__title mb-none">
-            <%= html_truncate resource_link_text, length: 80 %>
-          </h5>
+          <%= title %>
         <% end %>
-        <%= description %>
       </div>
-    </li>
-  </ul>
-</article>
+    </div>
+    <div class="card__author inline-block">
+      <%= author_data = author %>
+    </div><div class="card__activity-date inline-block <%= "before-content" if author_data.present? %>">
+      <span class="text-medium mt-xs"><%= created_at %></span>
+    </div>
+    <div class="card__header mb-none">
+      <%= link_to resource_link_path, class: "card__link" do %>
+        <span class="card__title mb-none">
+          <% @sanitized_resource_link_text = html_truncate decidim_sanitize(resource_link_text, strip_tags: true), length: 80 %>
+          <%= @sanitized_resource_link_text %>
+        </span>
+      <% end %>
+      <% if description.present? && @sanitized_resource_link_text.present? %>
+        <%= html_truncate decidim_sanitize(description, strip_tags: true), length: 80 %>
+      <% end %>
+    </div>
+  </div>
+</div>

data/app/cells/decidim/activity_cell.rb

@@ -76,6 +76,8 @@ module Decidim
     def user
       return resource.normalized_author if resource.respond_to?(:normalized_author)
       return resource.author if resource.respond_to?(:author)
+      # As Proposals have Coauthorable concern instead of Authorable
+      return resource.identities.first if resource.respond_to?(:identities)
 
       model.user_lazy if resource.respond_to?(:user)
     end

data/app/cells/decidim/address_cell.rb

@@ -13,7 +13,7 @@ module Decidim
     private
 
     def resource_icon
-      icon "meetings", class: "icon--big"
+      icon "meetings", class: "icon--big", role: "img", "aria-hidden": true
     end
   end
 end

data/app/cells/decidim/amendable/amend_button_card_cell.rb

@@ -31,12 +31,8 @@ module Decidim
         end
       end
 
-      def decidim
-        Decidim::Core::Engine.routes.url_helpers
-      end
-
       def button_classes
-        "amend_button_card_cell button secondary hollow expanded button--icon button--sc"
+        "amend_button_card_cell button hollow expanded button--icon button--sc"
       end
     end
   end

data/app/cells/decidim/amendable/promote_button_card_cell.rb

@@ -44,7 +44,7 @@ module Decidim
       end
 
       def button_classes
-        "amend_button_card_cell button secondary hollow expanded button--icon button--sc"
+        "amend_button_card_cell button expanded button--icon button--sc"
       end
     end
   end

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

@@ -1,7 +1,7 @@
 <div class="row column">
   <%= content_tag :div, class: "callout announcement mb-sm #{callout_class} cell-announcement" do %>
     <% if announcement.class == Hash && announcement.key?(:title) %>
-      <h5><%= decidim_sanitize translated_attribute announcement[:title] %></h5>
+      <span><%= decidim_sanitize translated_attribute announcement[:title] %></span>
 
       <p><%= decidim_sanitize translated_attribute announcement[:body] %></p>
     <% else %>

data/app/cells/decidim/author/comments.erb

@@ -1,6 +1,6 @@
 <% if commentable? %>
   <%= link_to "#{resource_locator(from_context).path}#comments", title: t("decidim.author.comments", count: from_context.comments.count) do %>
-    <%= icon "comment-square", class: "icon--small", aria_label: t("decidim.author.comments", count: from_context.comments.count), role: "img" %>
+    <%= icon "comment-square", class: "icon--small", role: "img", "aria-hidden": true %>
     <%= from_context.comments.count %> <%= t("decidim.author.comments", count: from_context.comments.count) %>
   <% end %>
 <% end %>

data/app/cells/decidim/author/endorsements.erb

@@ -0,0 +1,6 @@
+<% if endorsable? %>
+  <%= link_to "#{resource_locator(from_context).path}#list-of-endorsements", title: t("decidim.author.endorsements",count: from_context.endorsements_count) do %>
+    <%= icon "bullhorn", class: "icon--small", role: "img", "aria-hidden": true %>
+    <%= from_context.endorsements_count %> <%= t("decidim.author.endorsements", count: from_context.endorsements_count) %>
+  <% end %>
+<% end %>

data/app/cells/decidim/author/flag.erb

@@ -1,5 +1,8 @@
 <% if flagable? %>
-  <button type="button" data-open="<%= current_user.present? ? "flagModal" : "loginModal" %>" title="<%= t("report", scope: "decidim.proposals.proposals.show") %>" aria-controls="<%= current_user.present? ? "flagModal" : "loginModal" %>" aria-haspopup="true" tabindex="0">
-    <%= icon "flag", aria_label: t("report", scope: "decidim.proposals.proposals.show"), class: "icon--small" %>
+  <button type="button" class="link-alt" data-open="<%= current_user.present? ? "flagModal" : "loginModal" %>" title="<%= t("report", scope: "decidim.proposals.proposals.show") %>" aria-controls="<%= current_user.present? ? "flagModal" : "loginModal" %>" aria-haspopup="true" tabindex="0">
+    <%= icon "flag", aria_hidden: true, class: "icon--small", role: "img", "aria-hidden": true %>
+    <span class="show-for-sr">
+      <%= t("report", scope: "decidim.proposals.proposals.show") %>
+    </span>
   </button>
 <% end %>

data/app/cells/decidim/author/profile_inline.erb

@@ -17,7 +17,6 @@
     data-keep-on-hover="true"
     data-allow-html="true"
     data-template-classes="light expanded"
-    tabindex="0"
     data-tip-text='<%= h render(:profile_minicard) %>'>
     <%= model.name %>
   </span>

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

@@ -9,6 +9,8 @@
 
       <%= render :comments %>
 
+      <%= render :endorsements %>
+
       <%= render :flag %>
 
       <%= render :withdraw %>

data/app/cells/decidim/author/withdraw.erb

@@ -1,6 +1,6 @@
 <% if withdrawable? %>
-  <%= action_authorized_link_to :withdraw, withdraw_path, method: :put, class: "title-action__action button small hollow", title: t("withdraw_btn_hint", scope: "decidim.proposals.proposals.show"), data: { confirm: t("withdraw_confirmation", scope: "decidim.proposals.proposals.show") }  do %>
+  <%= action_authorized_link_to :withdraw, withdraw_path, method: :put, class: "title-action__action button small hollow", title: t("withdraw_btn_hint", scope: "decidim.proposals.proposals.show"), data: { confirm: t("withdraw_confirmation_html", scope: "decidim.proposals.proposals.show") } do %>
     <%= t("withdraw_proposal", scope: "decidim.proposals.proposals.show") %>
-    <%= icon "x" %>
+    <%= icon "x", role: "img", "aria-hidden": true %>
   <% end %>
 <% end %>

data/app/cells/decidim/author_cell.rb

@@ -55,28 +55,25 @@ module Decidim
     end
 
     def creation_date?
-      return true if posts_controller?
       return unless from_context
-      return unless proposals_controller? || collaborative_drafts_controller?
-      return unless show_action?
+      return unless proposals_controller? || collaborative_drafts_controller? || posts_controller?
+      return unless show_action? && (from_context.respond_to?(:published_at) || from_context.respond_to?(:created_at))
 
       true
     end
 
     def creation_date
-      date_at = if proposals_controller?
-                  from_context.published_at
-                else
-                  from_context.created_at
-                end
+      date_at = from_context.try(:published_at) || from_context.try(:created_at)
 
       l date_at, format: :decidim_short
     end
 
     def commentable?
-      return unless posts_controller?
+      from_context && from_context.class.include?(Decidim::Comments::Commentable)
+    end
 
-      true
+    def endorsable?
+      from_context && from_context.class.include?(Decidim::Endorsable)
     end
 
     def author_classes
@@ -85,13 +82,12 @@ module Decidim
 
     def actionable?
       return false if options[:has_actions] == false
-      return true if user_author? && posts_controller?
 
-      true if withdrawable? || flagable?
+      (user_author? && posts_controller?) || withdrawable? || flagable?
     end
 
     def user_author?
-      true if "Decidim::UserPresenter".include? model.class.to_s
+      "Decidim::UserPresenter".include?(model.class.to_s)
     end
 
     def profile_path?

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

@@ -1,14 +1,14 @@
 <div class="column">
-  <article class="card card--badge absolutes badge-<%= badge.name %>">
+  <div class="card card--badge absolutes badge-<%= badge.name %>">
     <div class="right badge-tip">
-      <div data-tooltip data-position="top" title="<%= tooltip %>" aria-describedby="<%= badge.name %>-tooltip" data-yeti-box="<%= badge.name %>-tooltip" data-toggle="<%= badge.name %>-tooltip" data-resize="<%= badge.name %>-tooltip" class="has-tip" data-t="<%= badge.name %>-t">
-        <%= icon "info" %>
+      <div data-tooltip data-position="top" title="<%= tooltip %>" data-yeti-box="<%= badge.name %>-tooltip" data-toggle="<%= badge.name %>-tooltip" data-resize="<%= badge.name %>-tooltip" class="has-tip" data-t="<%= badge.name %>-t">
+        <%= icon "info", role: "img", "aria-hidden": true %>
       </div>
     </div>
     <div class="card__content">
       <div class="card__header" style="opacity: <%= opacity %>;">
         <div class="badge-container">
-          <%= image_tag badge.image, class: "badge__logo" %>
+          <%= image_tag badge.image, class: "badge__logo", alt: "#{badge_name} badge_image" %>
         </div>
       </div>
       <div class="card__text text-center">
@@ -21,7 +21,7 @@
     <div class="card__footer card__footer--transparent">
         <div class="p-xs">
           <div class="flex--sbc">
-            <strong class="text-uppercase muted">
+            <strong class="muted">
               <%= level_title %>
             </strong>
             <div class="badge-level">
@@ -32,5 +32,5 @@
           </div>
         </div>
     </div>
-  </article>
+  </div>
 </div>

data/app/cells/decidim/badge/small.erb

@@ -1,5 +1,6 @@
 <%=
   image_tag badge.image,
     class: "badge__logo--small external-icon",
-    title: "#{badge_name} (#{level_title}): #{description}"
+    title: "#{badge_name} (#{level_title}): #{description}",
+    alt: "#{badge_name} badge_image"
 %>

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

@@ -1,11 +1,11 @@
 <div class="column">
-  <article class="card">
+  <div class="card">
     <div class="card__content">
       <% if title.present? %>
         <div class="card__header">
-          <h5 class="card__title">
+          <span class="card__title">
             <%= title %>
-          </h5>
+          </span>
         </div>
       <% end %>
 
@@ -15,5 +15,5 @@
         </p>
       <% end %>
     </div>
-  </article>
+  </div>
 </div>

data/app/cells/decidim/card_m/comments_counter.erb

@@ -1,3 +1,3 @@
-<%= icon("comment-square", class: "icon--small") %>
+<%= icon("comment-square", class: "icon--small", aria_label: "#{t("decidim.comments.comments_count")}:", role: "img") %>
 <%= " " %>
 <%= comments_count %>

data/app/cells/decidim/card_m/header.erb

@@ -1,15 +1,15 @@
 <div class="card__header">
   <% if has_link_to_resource? %>
     <%= link_to resource_path, class: "card__link" do %>
-      <h5 class="card__title">
+      <div class="card__title">
         <%= title %>
-      </h5>
+      </div>
     <% end %>
   <% else %>
     <div class="card__link">
-      <h5 class="card__title">
+      <div class="card__title">
         <%= title %>
-      </h5>
+      </div>
     </div>
   <% end %>
 

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

@@ -1,5 +1,5 @@
 <% if resource_image_path.present? %>
   <%= link_to resource_path do %>
-    <%= image_tag resource_image_path, class: "card__image" %>
+    <%= image_tag resource_image_path, class: "card__image", alt: title %>
   <% end %>
 <% end %>

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

@@ -1,5 +1,5 @@
 <div class="column" id="<%= dom_id(model) %>">
-  <article class="card <%= card_classes %>">
+  <div class="card <%= card_classes %>">
     <%= render :label if has_label? %>
     <%= render :top if render_top? %>
     <%= render :image if has_image? %>
@@ -19,5 +19,5 @@
     <%= render :data %>
     <%= render :status %>
     <%= render :footer %>
-  </article>
+  </div>
 </div>

data/app/cells/decidim/card_m_cell.rb

@@ -117,7 +117,8 @@ module Decidim
     end
 
     def creation_date_status
-      l(model.created_at.to_date, format: :decidim_short)
+      explanation = content_tag(:strong, t("activemodel.attributes.common.created_at"))
+      "#{explanation}<br>#{l(model.created_at.to_date, format: :decidim_short)}"
     end
 
     def follow_status
@@ -127,7 +128,7 @@ module Decidim
     def comments_count_status
       return render_comments_count unless has_link_to_resource?
 
-      link_to resource_path do
+      link_to resource_path, "aria-label" => "#{t("decidim.comments.comments_count")}: #{comments_count}", title: t("decidim.comments.comments_count") do
         render_comments_count
       end
     end

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

@@ -7,12 +7,12 @@
         <%= card_for element, cell_options %>
       <% end %>
     <% end %>
-    <span class="collapsible-list__see-more">
+    <span class="card__text--paragraph collapsible-list__see-more">
       <%= t(hidden_elements_count_i18n_key, count: hidden_elements_count) %>
     </span>
     <span class="text-small">
-      <a class="collapsible-list__see-more" data-toggle="collapsible-list-<%= seed %>"><%= t("decidim.collapsible_list.see_more") %></a>
-      <a class="collapsible-list__see-less" data-toggle="collapsible-list-<%= seed %>"><%= t("decidim.collapsible_list.see_less") %></a>
+      <span class="collapsible-list__see-more" data-toggle="collapsible-list-<%= seed %>"><%= t("decidim.collapsible_list.see_more") %></span>
+      <span class="collapsible-list__see-less" data-toggle="collapsible-list-<%= seed %>"><%= t("decidim.collapsible_list.see_less") %></span>
     </span>
   </div>
 <% else %>

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

@@ -1,12 +1,12 @@
 <section class="footer__subhero extended subhero home-section">
   <div class="row">
     <div class="columns small-centered large-10">
-      <h2 class="heading3"><%= t("decidim.pages.home.footer_sub_hero.footer_sub_hero_headline", organization: current_organization.name) %></h2>
-      <h5 class="heading4"><%== t("decidim.pages.home.footer_sub_hero.footer_sub_hero_body") %></h5>
+      <h3 class="heading3"><%= t("decidim.pages.home.footer_sub_hero.footer_sub_hero_headline", organization: current_organization.name) %></h3>
+      <h4 class="heading4"><%== t("decidim.pages.home.footer_sub_hero.footer_sub_hero_body") %></h4>
       <% if current_organization.sign_up_enabled? && !current_user %>
-        <%= link_to decidim.new_user_registration_path, class: "button--sc link subhero-cta" do %>
+        <%= link_to decidim.new_user_registration_path, class: "button--sc link subhero-cta", title: t("decidim.pages.home.sub_hero.register_title") do %>
           <%= t("decidim.pages.home.footer_sub_hero.register") %>
-          <%= icon "chevron-right", aria_hidden: true %>
+          <%= icon "chevron-right", aria_hidden: true, role: "img" %>
         <% end %>
       <% end %>
     </div>

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

@@ -1,7 +1,7 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <%= settings_fields.translated :text_field, :welcome_text %>
+  <%= settings_fields.translated :text_field, :welcome_text, label: t(".welcome_text") %>
 <% end %>
 
 <% form.fields_for :images, form.object.images do |images_fields| %>
-  <%= images_fields.upload :background_image %>
+  <%= images_fields.upload :background_image, label: t(".background_image") %>
 <% end %>

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

@@ -1,16 +1,16 @@
 <section class="extended home-section">
   <div class="wrapper-home">
     <div class="row column text-center">
-      <h3 class="heading2"><%= t("decidim.pages.home.extended.how_to_participate") %></h3>
+      <h3 class="heading3"><%= t("decidim.pages.home.extended.how_to_participate") %></h3>
     </div>
     <div class="row small-up-1 medium-up-3 home-bullets">
       <div class="column">
         <div class="home-bullet">
           <div class="home-bullet__icon">
-            <%= icon "meetings" %>
+            <%= icon "meetings", role: "img", "aria-hidden": true %>
           </div>
           <div class="home-bullet__desc">
-            <h4 class="heading6"><%= t("decidim.pages.home.extended.meetings") %></h4>
+            <h4 class="heading4"><%= t("decidim.pages.home.extended.meetings") %></h4>
             <p><%= t("decidim.pages.home.extended.meetings_explanation") %></p>
           </div>
         </div>
@@ -18,10 +18,10 @@
       <div class="column">
         <div class="home-bullet">
           <div class="home-bullet__icon">
-            <%= icon "debates" %>
+            <%= icon "debates", role: "img", "aria-hidden": true %>
           </div>
           <div class="home-bullet__desc">
-            <h4 class="heading6"><%= t("decidim.pages.home.extended.debates") %></h4>
+            <h4 class="heading4"><%= t("decidim.pages.home.extended.debates") %></h4>
             <p><%= t("decidim.pages.home.extended.debates_explanation") %></p>
           </div>
         </div>
@@ -29,10 +29,10 @@
       <div class="column">
         <div class="home-bullet">
           <div class="home-bullet__icon">
-            <%= icon "proposals" %>
+            <%= icon "proposals", role: "img", "aria-hidden": true %>
           </div>
           <div class="home-bullet__desc">
-            <h4 class="heading6"><%= t("decidim.pages.home.extended.proposals") %></h4>
+            <h4 class="heading4"><%= t("decidim.pages.home.extended.proposals") %></h4>
             <p><%= t("decidim.pages.home.extended.proposals_explanation") %></p>
           </div>
         </div>
@@ -41,7 +41,7 @@
     <div class="row">
       <div class="columns small-centered small-10
                 smallmedium-8 medium-6 large-4">
-        <%= link_to t("decidim.pages.home.extended.more_info"), decidim.page_path("faq"), class: "button expanded hollow button--sc" %>
+        <%= link_to t("decidim.pages.home.extended.more_info", resource_name: translated_attribute(current_organization.name, current_organization)), decidim.page_path("faq"), class: "button expanded hollow button--sc" %>
       </div>
     </div>
   </div>

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

@@ -30,7 +30,7 @@ module Decidim
         valid_activities_count = 0
         @valid_activities = []
 
-        activities.each do |activity|
+        activities.includes([:user]).each do |activity|
           break if valid_activities_count == activities_to_show
 
           if activity.visible_for?(current_user)