diff --git a/app/cells/decidim/plans/author/flag.erb b/app/cells/decidim/plans/author/flag.erb index 7a001f15..24a4a958 100644 --- a/app/cells/decidim/plans/author/flag.erb +++ b/app/cells/decidim/plans/author/flag.erb @@ -1,5 +1,5 @@ <% if flaggable? %> <% end %> diff --git a/app/cells/decidim/plans/collapsible_authors/show.erb b/app/cells/decidim/plans/collapsible_authors/show.erb index 64bd754d..4c08773a 100644 --- a/app/cells/decidim/plans/collapsible_authors/show.erb +++ b/app/cells/decidim/plans/collapsible_authors/show.erb @@ -1,17 +1,18 @@ -
- <%= cell( - "decidim/collapsible_list", - list, - cell_name: cell_name, - cell_options: options.merge(has_actions: false), - size: size - ) %> +
+ <% visible_authors.each do |author| %> + <%= cell "decidim/plans/author", author, layout: :compact %> + <% end %> + + <% if model.length > MAX_ITEMS_STACKED %> + + + + <% end %>
-<% if actionable? %> -
- <% author_cell= cell("decidim/plans/author", model, from: from_context) %> - <%= author_cell.(:date) %> - <%= author_cell.(:flag) %> - <%= author_cell.(:withdraw) %> -
-<% end %> diff --git a/app/cells/decidim/plans/plan_add_authors/show.erb b/app/cells/decidim/plans/plan_add_authors/show.erb index d948e947..05a9319f 100644 --- a/app/cells/decidim/plans/plan_add_authors/show.erb +++ b/app/cells/decidim/plans/plan_add_authors/show.erb @@ -26,8 +26,8 @@
-
-
+
+
<%= f.submit t(".next") %>
diff --git a/app/cells/decidim/plans/plan_notification_cell.rb b/app/cells/decidim/plans/plan_notification_cell.rb index 2747ab9f..def4e9e6 100644 --- a/app/cells/decidim/plans/plan_notification_cell.rb +++ b/app/cells/decidim/plans/plan_notification_cell.rb @@ -34,11 +34,11 @@ def status_class def icon_key if model.accepted? - "circle-check" + "checkbox-circle-line" elsif model.rejected? - "warning" + "error-warning-line" else - "flag" + "flag-line" end end diff --git a/app/cells/decidim/plans/plan_view/actions.erb b/app/cells/decidim/plans/plan_view/actions.erb index 31c0d164..601d872c 100644 --- a/app/cells/decidim/plans/plan_view/actions.erb +++ b/app/cells/decidim/plans/plan_view/actions.erb @@ -50,7 +50,7 @@ <% if flaggable? %>
  • diff --git a/app/cells/decidim/plans/plan_view/share_modal.erb b/app/cells/decidim/plans/plan_view/share_modal.erb index 302f8024..38dba370 100644 --- a/app/cells/decidim/plans/plan_view/share_modal.erb +++ b/app/cells/decidim/plans/plan_view/share_modal.erb @@ -1,7 +1,8 @@ @@ -22,7 +23,8 @@

    <%= t("decidim.shared.share_modal.share_link") %>:

    -
    diff --git a/app/cells/decidim/plans/plan_view/show.erb b/app/cells/decidim/plans/plan_view/show.erb index 7ae93f5e..410e6cf4 100644 --- a/app/cells/decidim/plans/plan_view/show.erb +++ b/app/cells/decidim/plans/plan_view/show.erb @@ -48,7 +48,6 @@
    - <%== cell("decidim/plans/plan_m", plan, full_badge: true).badge %> <%= contents %> <%= cell("decidim/plans/tags", plan, context: { extra_classes: ["tags--plan"] }) %>
    diff --git a/app/cells/decidim/plans/plan_view_cell.rb b/app/cells/decidim/plans/plan_view_cell.rb index 8bd2b539..b8114e7f 100644 --- a/app/cells/decidim/plans/plan_view_cell.rb +++ b/app/cells/decidim/plans/plan_view_cell.rb @@ -8,6 +8,7 @@ class PlanViewCell < Decidim::ViewModel include Decidim::ApplicationHelper # For presenter include Decidim::LayoutHelper # For the icon helper include Decidim::FormFactory + include Decidim::SocialShareButtonHelper include Decidim::TooltipHelper include Decidim::MetaTagsHelper include Decidim::Plans::LinksHelper @@ -188,7 +189,7 @@ def organization_share_image_url scope_name: :homepage, manifest_name: :hero ).try(:images_container) - container.attached_uploader(:background_image).url if container && container.background_image && container.background_image.attached? + container.attached_uploader(:background_image).path if container && container.background_image && container.background_image.attached? end end diff --git a/lib/decidim/plans/engine.rb b/lib/decidim/plans/engine.rb index 0acfedc1..6b70aaea 100644 --- a/lib/decidim/plans/engine.rb +++ b/lib/decidim/plans/engine.rb @@ -209,6 +209,7 @@ class Engine < ::Rails::Engine initializer "decidim_plans.register_icons", after: "decidim_core.add_social_share_services" do Decidim.icons.register(name: "focus-3-line", icon: "focus-3-line", category: "system", description: "", engine: :core) + Decidim.icons.register(name: "links-line", icon: "links-line", category: "system", description: "", engine: :core) end end end diff --git a/spec/cells/decidim/plans/collapsible_authors_cell_spec.rb b/spec/cells/decidim/plans/collapsible_authors_cell_spec.rb index 0e25485d..6a36b91b 100644 --- a/spec/cells/decidim/plans/collapsible_authors_cell_spec.rb +++ b/spec/cells/decidim/plans/collapsible_authors_cell_spec.rb @@ -16,10 +16,6 @@ controller Decidim::Plans::PlansController - before do - expect(my_cell).to receive(:cell).with("decidim/collapsible_list", any_args) - end - it "renders collapsible list" do subject end diff --git a/spec/cells/decidim/plans/plan_notification_cell_spec.rb b/spec/cells/decidim/plans/plan_notification_cell_spec.rb index 57002c3d..58ca6b0a 100644 --- a/spec/cells/decidim/plans/plan_notification_cell_spec.rb +++ b/spec/cells/decidim/plans/plan_notification_cell_spec.rb @@ -30,7 +30,7 @@ it "renders the show" do expect(subject).to have_css(".callout__title") - expect(subject).to have_css("svg.icon--circle-check.icon[aria-label='circle-check']") + expect(subject).to have_css("svg[role=\"img\"][aria-hidden=\"true\"] use[href*=\"#ri-checkbox-circle-line\"]") expect(subject).to have_content("This proposal has been accepted") within ".callout__content" do expect(subject).to have_content(translated(plan.title))