decidim-core 0.23.3 → 0.24.1

data/app/assets/stylesheets/decidim/email.scss

@@ -1091,7 +1091,7 @@ a{
 }
 
 a:hover{
-  color: #147dc2;
+  color: var(--secondary);
 }
 
 a:active{
@@ -1246,7 +1246,13 @@ table.button.large table a{
 
 table.button.expand,
 table.button.expanded{
-  width: 100% !important;
+  width: 100%;
+}
+
+// Fix for Outlook to center the button
+table.button.expand > tr > td,
+table.button.expanded > tr > td{
+  text-align: center;
 }
 
 table.button.expand table,
@@ -1255,6 +1261,12 @@ table.button.expanded table{
   margin: auto;
 }
 
+// Fix for Outlook to center the button link text
+table.button.expand table > tr > td,
+table.button.expanded table > tr > td{
+  text-align: center;
+}
+
 table.button.expand table a,
 table.button.expanded table a{
   text-align: center;

data/app/assets/stylesheets/decidim/extras/_external-links.scss

@@ -1,6 +1,5 @@
 .external-link-container{
   position: relative;
-  display: inline-block;
 }
 
 .external-link-indicator{

data/app/assets/stylesheets/decidim/extras/_process_stats.scss

@@ -28,7 +28,7 @@
     .icon{
       width: 1.5em;
       height: 1.5em;
-      color: #599aa6;
+      color: var(--secondary);
       vertical-align: middle;
       margin-right: .5rem;
     }

data/app/assets/stylesheets/decidim/extras/_results-per-page.scss

@@ -9,6 +9,7 @@ $order-border: $border;
   flex-wrap: wrap;
 
   a:hover{
+    color: var(--secondary);
     text-decoration: underline;
   }
 

data/app/assets/stylesheets/decidim/modules/_author-avatar.scss

@@ -11,13 +11,13 @@ $author-data-color: $muted;
   @include flex;
   @include flex-align($x: left, $y: middle);
 
-  color: $author-data-color;
+  color: var(--secondary);
   font-size: .9rem;
   padding-left: 30px;
   position: relative;
 
   .author__name{
-    color: $anchor-color;
+    color: var(--secondary);
     margin-bottom: 0;
   }
 

data/app/assets/stylesheets/decidim/modules/_buttons.scss

@@ -339,7 +339,7 @@
   background-color: $muted;
 
   &:hover{
-    background-color: var(--secondary);
+    background-color: var(--primary);
   }
 
   &.hollow{
@@ -371,7 +371,7 @@
 
 .link{
   cursor: pointer;
-  color: $anchor-color;
+  color: var(--secondary);
   font-weight: 600;
 
   &:hover{

data/app/assets/stylesheets/decidim/modules/_cards.scss

@@ -89,6 +89,10 @@ a .card__title{
 .card__header{
   margin-bottom: $card-padding / 2;
 
+  .card__title{
+    margin-bottom: .5rem;
+  }
+
   &.collapse{
     margin-bottom: 0;
   }
@@ -957,6 +961,7 @@ a .card__title{
       font-size: $global-font-size;
       text-transform: none;
       justify-content: left;
+      flex-direction: row;
 
       strong{
         text-transform: uppercase;
@@ -1090,12 +1095,24 @@ a .card__title{
   @include flex;
   @include flex-align($x: left, $y: null);
 
+  flex-direction: column;
+
+  @include breakpoint(medium){
+    flex-direction: row;
+  }
+
   > *{
     padding: $global-margin;
     flex: 0 0 25%;
 
     &:not(:nth-child(4n)){
-      border-right: $card-border;
+      border-right: 0;
+      border-bottom: $card-border;
+
+      @include breakpoint(medium){
+        border-bottom: 0;
+        border-right: $card-border;
+      }
     }
   }
 }

data/app/assets/stylesheets/decidim/modules/_collapsible-list.scss

@@ -36,7 +36,7 @@
 
   .collapsible-list__see-more{
     display: none;
-    color: #599aa6;
+    color: $secondary;
   }
 
   .collapsible-list__see-more:hover{

data/app/assets/stylesheets/decidim/modules/_comments.scss

@@ -113,6 +113,10 @@ $comment-form-bg: $light-gray;
     color: $white;
     outline: none;
   }
+
+  form.button_to{
+    display: inline-block;
+  }
 }
 
 .comment__votes--up{
@@ -207,7 +211,7 @@ $comment-form-bg: $light-gray;
   }
 
   a:focus{
-    background-color: $secondary;
+    background-color: var(--secondary);
     color: $white;
     outline: none;
   }

data/app/assets/stylesheets/decidim/modules/_conference-speaker.scss

@@ -78,9 +78,10 @@
   @include breakpoint(medium){
     position: relative;
 
-    &:nth-of-type(4n + 3) .speaker-bio,
-    &:nth-of-type(4n + 4) .speaker-bio{
-      transform: translateY(-10%) translateX(-50%);
+    &:nth-of-type(4) .speaker-bio,
+    &:nth-of-type(4n + 5) .speaker-bio,
+    &:nth-of-type(5n) .speaker-bio,{
+      transform: translateY(-10%) translateX(-60%);
     }
   }
 }

data/app/assets/stylesheets/decidim/modules/_data-picker.scss

@@ -78,6 +78,13 @@
       }
     }
   }
+
+  .picker-values{
+    .label.primary{
+      background: #eee;
+      color: #202734;
+    }
+  }
 }
 
 #data_picker-modal{

data/app/assets/stylesheets/decidim/modules/_extra.scss

@@ -30,6 +30,10 @@
   line-height: 1;
 }
 
+.extra__date-container{
+  margin-bottom: 1rem;
+}
+
 .extra__month{
   display: block;
   font-size: 1rem;

data/app/assets/stylesheets/decidim/modules/_forms.scss

@@ -46,6 +46,10 @@ label,
   margin-top: $form-spacing * -1;
 }
 
+#registration_user_nickname_characters{
+  margin-top: 1rem;
+}
+
 .form-input-extra-before{
   margin-bottom: $form-spacing / 2;
 }

data/app/assets/stylesheets/decidim/modules/_loading-spinner.scss

@@ -10,3 +10,37 @@
     vertical-align: middle;
   }
 }
+
+.spinner-container{
+  position: relative;
+  cursor: wait;
+
+  &::before{
+    content: " ";
+    position: absolute;
+    top: 0;
+    left: 0;
+    width: 100%;
+    height: 100%;
+    background-color: rgba($white, .8);
+    z-index: 1;
+  }
+
+  &::after{
+    -webkit-animation: animation-spin 800ms infinite linear;
+    animation: animation-spin 800ms infinite linear;
+    width: 25px;
+    height: 25px;
+    box-sizing: border-box;
+    border-radius: 50%;
+    border: 3px solid #e8e8e8;
+    border-right-color: var(--primary);
+    display: inline-block;
+    position: absolute;
+    content: "";
+    vertical-align: middle;
+    z-index: 1;
+    top: 47%;
+    left: 47%;
+  }
+}

data/app/assets/stylesheets/decidim/modules/_modules.scss

@@ -52,6 +52,7 @@
 @import "decidim/modules/versions";
 @import "decidim/modules/sticky";
 @import "decidim/modules/notification";
+@import "decidim/modules/space-stats";
 
 //Process elements
 @import "decidim/modules/process-header";
@@ -64,7 +65,6 @@
 @import "decidim/modules/inline-filters";
 @import "decidim/modules/reference";
 @import "decidim/modules/status-labels";
-@import "decidim/modules/process-stats";
 
 //Conference elements
 @import "decidim/modules/conference-nav";

data/app/assets/stylesheets/decidim/modules/_opinion-toggle.scss

@@ -50,6 +50,6 @@
   &:hover,
   &:focus,
   &.is-active{
-    background-color: $secondary;
+    background-color: var(--secondary);
   }
 }

data/app/assets/stylesheets/decidim/modules/_reveal.scss

@@ -73,3 +73,9 @@
 .reveal__trigger{
   cursor: pointer;
 }
+
+.reveal__buttons{
+  display: flex;
+  justify-content: space-between;
+  padding: 0 1rem;
+}

data/app/assets/stylesheets/decidim/modules/_signup.scss

@@ -66,6 +66,7 @@
       // this is the prefix div
       @extend .input-group-label;
 
+      padding: 0;
       border-#{$global-right}: 0;
       width: 8%;
       height: 3rem;

data/app/assets/stylesheets/decidim/modules/{_process-stats.scss → _space-stats.scss}

@@ -1,4 +1,4 @@
-.process-stats{
+.space-stats{
   overflow: hidden;
   margin: 2rem 0;
   display: flex;

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

@@ -1,4 +1,4 @@
-<div class="card card--activity">
+<div class="card card--activity" id="<%= element_id %>">
   <div class="card__content">
     <div class="card__text">
       <div class="text-medium">

data/app/cells/decidim/activity_cell.rb

@@ -36,9 +36,10 @@ module Decidim
       resource_title = resource.try(:resource_title) || resource.try(:title)
       return if resource_title.blank?
 
-      if resource_title.is_a?(String)
+      case resource_title
+      when String
         resource_title
-      elsif resource_title.is_a?(Hash)
+      when Hash
         translated_attribute(resource_title)
       end
     end
@@ -50,9 +51,10 @@ module Decidim
       resource_description = resource.try(:resource_description) || resource.try(:description)
       return if resource_description.blank?
 
-      resource_description = if resource_description.is_a?(String)
+      resource_description = case resource_description
+                             when String
                                resource_description
-                             elsif resource_description.is_a?(Hash)
+                             when Hash
                                translated_attribute(resource_description)
                              end
 
@@ -66,7 +68,7 @@ module Decidim
 
     # The text to show as the link to the resource.
     def resource_link_text
-      translated_attribute(resource.title)
+      decidim_html_escape(translated_attribute(resource.title))
     end
 
     def created_at
@@ -82,6 +84,12 @@ module Decidim
       model.user_lazy if resource.respond_to?(:user)
     end
 
+    delegate :action, to: :model
+
+    def element_id
+      "action-#{model.id}"
+    end
+
     private
 
     def published?
@@ -101,9 +109,10 @@ module Decidim
     def author
       return unless show_author? && user.is_a?(UserBaseEntity)
 
-      presenter = if user.is_a?(Decidim::User)
+      presenter = case user
+                  when Decidim::User
                     UserPresenter.new(user)
-                  elsif user.is_a?(Decidim::UserGroup)
+                  when Decidim::UserGroup
                     UserGroupPresenter.new(user)
                   end
 
@@ -120,7 +129,7 @@ module Decidim
 
     def participatory_space_link
       link_to(
-        translated_attribute(participatory_space.title),
+        decidim_html_escape(translated_attribute(participatory_space.title)),
         resource_locator(participatory_space).path
       )
     end

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

@@ -1,7 +1,7 @@
-<% if model.respond_to? :location %>
-  <strong><%= translated_attribute model.location %></strong><br>
+<% if has_location? %>
+  <strong><%= location %></strong><br>
 <% end %>
-<span><%= model.address %></span><br>
-<% if model.respond_to? :location_hints %>
-  <span><%= translated_attribute model.location_hints %></span>
+<span><%= address %></span><br>
+<% if has_location_hints? %>
+  <span><%= location_hints %></span>
 <% end %>

data/app/cells/decidim/address_cell.rb

@@ -5,11 +5,32 @@ module Decidim
   class AddressCell < Decidim::ViewModel
     include Cell::ViewModel::Partial
     include LayoutHelper
+    include Decidim::SanitizeHelper
 
     def details
       render
     end
 
+    def has_location?
+      model.respond_to?(:location)
+    end
+
+    def has_location_hints?
+      model.respond_to?(:location_hints)
+    end
+
+    def location_hints
+      decidim_sanitize(translated_attribute(model.location_hints))
+    end
+
+    def location
+      decidim_sanitize(translated_attribute(model.location))
+    end
+
+    def address
+      decidim_sanitize(translated_attribute(model.address))
+    end
+
     private
 
     def resource_icon