Merge pull request #4839 from consul/comments_count_investment

Add comments count on budget investments

app/assets/stylesheets/application.scss

@@ -22,6 +22,7 @@
 @import "annotator_overrides";
 @import "autocomplete_overrides";
 @import "banner";
+@import "comments_count";
 @import "datepicker_overrides";
 @import "layout";
 @import "participation";

app/assets/stylesheets/comments_count.scss

@@ -0,0 +1,26 @@
+.comments-count {
+  @include has-fa-icon(comments, regular);
+
+  &::before {
+    margin-right: 0.3rem;
+  }
+
+  .legislation-comments & {
+
+    a {
+      text-decoration: underline;
+    }
+
+    &::before {
+      color: #999;
+    }
+  }
+
+  .comment-header & {
+    @include has-fa-icon(comment, solid);
+
+    &::before {
+      color: #838383;
+    }
+  }
+}

app/assets/stylesheets/legislation_process.scss

@@ -153,10 +153,6 @@
   .debate-meta a {
     font-size: $small-font-size;
     color: #6d6d6d;
-
-    .icon-comments {
-      margin-right: rem-calc(3);
-    }
   }
 
   .debate-info {
@@ -607,20 +603,10 @@
         }
 
         .comment-header {
-          color: #838383;
           padding-bottom: rem-calc(8);
           margin-bottom: rem-calc(16);
           border-bottom: 1px solid $border;
 
-          .comment-number {
-            color: $text;
-            display: inline-block;
-          }
-
-          .icon-comment {
-            margin-right: rem-calc(8);
-          }
-
           a .icon-expand {
             color: #838383;
             font-size: $small-font-size;
@@ -806,13 +792,12 @@
   .comment {
     margin-bottom: rem-calc(48);
 
-    a {
+    .expand-context a {
       span {
         text-decoration: underline;
       }
 
-      .icon-expand,
+      .icon-expand {
-      .icon-comments {
         text-decoration: none;
         color: #999;
         line-height: 0;
@@ -822,10 +807,6 @@
         margin-left: rem-calc(4);
       }
 
-      .icon-comments {
-        margin-right: rem-calc(4);
-      }
-
       &:active,
       &:focus,
       &:hover {
@@ -975,10 +956,6 @@
     }
   }
 
-  .comments-count {
-    @include has-fa-icon(comments, regular);
-  }
-
   .question-title:not(:only-child) {
     margin-bottom: $line-height / 2;
   }

app/assets/stylesheets/participation.scss

@@ -486,6 +486,7 @@
 .debate,
 .proposal,
 .budget-investment,
+.budget-investment-show,
 .legislation,
 .communities-show {
   margin: $line-height / 4 0;

app/components/budgets/investments/info_component.html.erb

@@ -1,5 +1,11 @@
 <p class="investment-project-info">
   <%= l investment.created_at.to_date %>
+  <span class="bullet">&nbsp;&bull;&nbsp;</span>
+
+  <%= render Shared::CommentsCountComponent.new(
+    investment.comments_count,
+    url: budget_investment_path(investment.budget, investment, anchor: "comments")
+  ) %>
 
   <% if investment.author.hidden? || investment.author.erased? %>
     <span class="bullet">&nbsp;&bull;&nbsp;</span>

app/components/shared/comments_count_component.html.erb

@@ -0,0 +1,3 @@
+<span class="comments-count">
+  <%= link_to_if url.present?, text, url %>
+</span>

app/components/shared/comments_count_component.rb

@@ -0,0 +1,12 @@
+class Shared::CommentsCountComponent < ApplicationComponent
+  attr_reader :comments_count, :url
+
+  def initialize(comments_count, url: nil)
+    @comments_count = comments_count
+    @url = url
+  end
+
+  def text
+    t("shared.comments", count: comments_count)
+  end
+end

app/views/admin/debates/show.html.erb

@@ -17,8 +17,10 @@
     <span class="bullet">&nbsp;&bull;&nbsp;</span>
     <%= l @debate.created_at.to_date %>
     <span class="bullet">&nbsp;&bull;&nbsp;</span>
-    <span class="icon-comments"></span>&nbsp;
+    <%= render Shared::CommentsCountComponent.new(
-    <%= link_to t("debates.show.comments", count: @debate.comments_count), debate_path(@debate, anchor: "comments") %>
+      @debate.comments_count,
+      url: debate_path(@debate, anchor: "comments")
+    ) %>
     <span class="bullet">&nbsp;&bull;&nbsp;</span>
     <span class="js-flag-actions">
       <%= render "shared/flag_actions", flaggable: @debate %>

app/views/budgets/investments/_investment_detail.html.erb

@@ -5,6 +5,8 @@
   <span class="bullet">&nbsp;&bull;&nbsp;</span>
   <%= l investment.created_at.to_date %>
   <span class="bullet">&nbsp;&bull;&nbsp;</span>
+  <%= render Shared::CommentsCountComponent.new(investment.comments_count, url: "#comments") %>
+  <span class="bullet">&nbsp;&bull;&nbsp;</span>
   <%= investment.heading.name %>
   <span class="bullet">&nbsp;&bull;&nbsp;</span>
   <% if local_assigns[:preview].nil? %>

app/views/debates/_debate.html.erb

@@ -7,10 +7,12 @@
           <div class="debate-content">
             <h3><%= link_to debate.title, debate %></h3>
             <p class="debate-info">
-              <span class="icon-comments"></span>&nbsp;
-              <%= link_to t("debates.debate.comments", count: debate.comments_count), debate_path(debate, anchor: "comments") %>
-              <span class="bullet">&nbsp;&bull;&nbsp;</span>
               <%= l debate.created_at.to_date %>
+              <span class="bullet">&nbsp;&bull;&nbsp;</span>
+              <%= render Shared::CommentsCountComponent.new(
+                debate.comments_count,
+                url: debate_path(debate, anchor: "comments")
+              ) %>
 
               <% if debate.author.hidden? || debate.author.erased? %>
                 <span class="bullet">&nbsp;&bull;&nbsp;</span>

app/views/debates/show.html.erb

@@ -26,8 +26,7 @@
           <span class="bullet">&nbsp;&bull;&nbsp;</span>
           <%= l @debate.created_at.to_date %>
           <span class="bullet">&nbsp;&bull;&nbsp;</span>
-          <span class="icon-comments"></span>&nbsp;
+          <%= render Shared::CommentsCountComponent.new(@debate.comments_count, url: "#comments") %>
-          <%= link_to t("debates.show.comments", count: @debate.comments_count), "#comments" %>
           <span class="bullet">&nbsp;&bull;&nbsp;</span>
           <span class="js-flag-actions">
             <%= render "shared/flag_actions", flaggable: @debate %>

app/views/legislation/annotations/_comment_header.html.erb

@@ -1,9 +1,4 @@
-<span class="icon-comment" aria-hidden="true"></span>
+<%= render Shared::CommentsCountComponent.new(annotation.comments.roots.count) %>
-
-<div class="comment-number">
-  <%= t("legislation.annotations.comments.comments_count",
-      count: annotation.comments.roots.count) %>
-</div>
 
 <span id="annotation-link" data-sub-annotation-ids="8">
   <%= render "annotation_link", annotation: annotation %>

app/views/legislation/annotations/_form.html.erb

@@ -1,7 +1,6 @@
 <div class="comment-box">
   <div class="comment-header">
-    <span class="icon-comment" aria-hidden="true"></span>
+    <%= render Shared::CommentsCountComponent.new(0) %>
-    <div class="comment-number"><%= t("legislation.annotations.comments.comments_count", count: 0) %></div>
   </div>
 
   <div class="comments-wrapper">

app/views/legislation/annotations/comments.js.erb

@@ -14,11 +14,11 @@ if ($(".comment").length == 0) {
 
   $("#annotation-link a").attr("href", new_annotation_link)
 
-  var current_comment_text = $(".comment-number").text()
+  var current_comment_text = $(".comments-count").text()
   var current_comment_count = current_comment_text.match(/\d+/)[0]
   var new_comment_count = parseInt(current_comment_count) + parseInt(<%= @annotation.comments.roots.count %>)
   var new_comment_count_text = current_comment_text.replace(/(\d+)/, new_comment_count);
-  $(".comment-number").text(new_comment_count_text)
+  $(".comments-count").text(new_comment_count_text)
 }
 
 <%= render "comments_box_form", comment: @comment, annotation: @annotation %>

app/views/legislation/annotations/index.html.erb

@@ -18,7 +18,7 @@
     <% @annotations.each do |annotation| %>
       <div class="comment">
         <strong><%= t("legislation.annotations.index.comments_about") %></strong>
-        <span class="float-right">
+        <span class="float-right expand-context">
           <%= link_to legislation_process_draft_version_path(@process, @draft_version, anchor: "annotation-id-#{annotation.id}") do %>
             <span><%= t("legislation.annotations.index.see_in_context") %></span>
             <span class="icon-expand" aria-hidden="true"></span>
@@ -27,12 +27,10 @@
         <div class="comment-section">
           <%= sanitize(annotation.context).presence || annotation.quote %>
         </div>
-        <%= link_to legislation_process_draft_version_annotation_path(@process, @draft_version, annotation) do %>
+        <%= render Shared::CommentsCountComponent.new(
-          <span class="icon-comments" aria-hidden="true"></span>
+          annotation.comments_count,
-          <span>
+          url: legislation_process_draft_version_annotation_path(@process, @draft_version, annotation)
-            <%= t("legislation.annotations.index.comments_count", count: annotation.comments_count) %>
+        ) %>
-          </span>
-        <% end %>
       </div>
     <% end %>
 

app/views/legislation/draft_versions/_comments_panel.html.erb

@@ -7,8 +7,7 @@
   <div id="comments-box" class="comments-box-container" style="display: none;">
     <div class="comment-box">
       <div class="comment-header">
-        <span class="icon-comment" aria-hidden="true"></span>
+        <span class="comments-count"><%= t("legislation.draft_versions.show.loading_comments") %></span>
-        <div class="comment-number"><%= t("legislation.draft_versions.show.loading_comments") %></div>
       </div>
     </div>
   </div>

app/views/legislation/processes/_summary_allegations.html.erb

@@ -14,10 +14,10 @@
               <%= annotation.quote %>
             </div>
 
-            <span class="comments-count">
+            <%= render Shared::CommentsCountComponent.new(
-              <%= link_to t("legislation.summary.comments", count: annotation.comments.count),
+              annotation.comments_count,
-                          polymorphic_path(annotation, anchor: "comments") %>
+              url: polymorphic_path(annotation, anchor: "comments")
-            </span>
+            ) %>
           </div>
 
           <%= render "summary_comments", comments: annotation_comments %>

app/views/legislation/processes/_summary_debate.html.erb

@@ -10,10 +10,10 @@
         <div class="debate-summary">
           <div class="question-title">
             <h4><%= link_to question.title, polymorphic_path(question) %></h4>
-            <span class="comments-count">
+            <%= render Shared::CommentsCountComponent.new(
-              <%= link_to t("legislation.summary.comments", count: question.comments.count),
+              question.comments.count,
-                          polymorphic_path(question, anchor: "comments") %>
+              url: polymorphic_path(question, anchor: "comments")
-            </span>
+            ) %>
           </div>
 
           <%= render "summary_comments", comments: question.best_comments %>

app/views/legislation/processes/summary.xlsx.axlsx

@@ -6,7 +6,7 @@ xlsx_package.workbook.add_worksheet(name: "Summary") do |sheet|
   if @process.debate_phase.enabled? && @process.questions.any?
     sheet.add_row [t("legislation.summary.debate_phase"), t("legislation.summary.debates", count: @process.questions.count)], style: title
     @process.questions.each do |question|
-      sheet.add_row [question.title, t("legislation.summary.comments", count: question.comments.count)], style: link
+      sheet.add_row [question.title, t("shared.comments", count: question.comments.count)], style: link
       sheet.add_hyperlink location: legislation_process_question_url(question.process, question), ref: sheet.rows.last.cells.first
       sheet.add_hyperlink location: polymorphic_url(question, anchor: "comments"), ref: sheet.rows.last.cells.last
       sheet.add_row [t("legislation.summary.most_voted_comments")] if question.best_comments.any?
@@ -34,7 +34,7 @@ xlsx_package.workbook.add_worksheet(name: "Summary") do |sheet|
                    t("legislation.summary.top_comments", count: @comments.count)], style: title
     @comments.group_by(&:commentable).each do |annotation, annotation_comments|
       sheet.add_row [t("legislation.annotations.index.comments_about")]
-      sheet.add_row [annotation.quote, t("legislation.summary.comments", count: annotation.comments.count)]
+      sheet.add_row [annotation.quote, t("shared.comments", count: annotation.comments.count)]
       sheet.add_hyperlink location: polymorphic_url(annotation, anchor: "comments"), ref: sheet.rows.last.cells.last
       sheet.rows.last.cells.last.style = link
 

app/views/legislation/proposals/_proposal.html.erb

@@ -19,11 +19,15 @@
           <% cache [locale_and_user_status(proposal), "index", proposal, proposal.author] do %>
             <h3><%= link_to proposal.title, legislation_process_proposal_path(proposal.legislation_process_id, proposal) %></h3>
             <p class="proposal-info">
-              <span class="icon-comments"></span>&nbsp;
-              <%= link_to t("proposals.proposal.comments", count: proposal.comments_count), legislation_process_proposal_path(proposal.legislation_process_id, proposal, anchor: "comments") %>
-
-              <span class="bullet">&nbsp;&bull;&nbsp;</span>
               <%= l proposal.created_at.to_date %>
+              <span class="bullet">&nbsp;&bull;&nbsp;</span>
+              <%= render Shared::CommentsCountComponent.new(
+                proposal.comments_count,
+                url: legislation_process_proposal_path(
+                       proposal.legislation_process_id,
+                       proposal,
+                       anchor: "comments")
+              ) %>
 
               <% if proposal.author.hidden? || proposal.author.erased? %>
                 <span class="bullet">&nbsp;&bull;&nbsp;</span>

app/views/legislation/proposals/show.html.erb

@@ -44,8 +44,7 @@
           <span class="bullet">&nbsp;&bull;&nbsp;</span>
           <%= l @proposal.created_at.to_date %>
           <span class="bullet">&nbsp;&bull;&nbsp;</span>
-          <span class="icon-comments"></span>&nbsp;
+          <%= render Shared::CommentsCountComponent.new(@proposal.comments_count, url: "#comments") %>
-          <%= link_to t("proposals.show.comments", count: @proposal.comments_count), "#comments" %>
 
           <% if current_user %>
             <span class="bullet">&nbsp;&bull;&nbsp;</span>

app/views/legislation/questions/_question.html.erb

@@ -6,9 +6,10 @@
     <h4><%= link_to question.title, legislation_process_question_path(question.process, question) %></h4>
   </div>
   <div class="debate-meta">
-    <span class="icon-comments" aria-hidden="true"></span> <span class="debate-comments">
+    <%= render Shared::CommentsCountComponent.new(
-      <%= link_to t("legislation.questions.question.comments", count: question.comments.count),
+      question.comments.count,
-                  legislation_process_question_path(question.process, question) %>
+      url: legislation_process_question_path(question.process, question, anchor: "comments")
-    </span> · <span class="debate-date"><%= format_date question.created_at %></span>
+    ) %>
+    · <span class="debate-date"><%= l question.created_at.to_date %></span>
   </div>
 </div>

app/views/proposals/_info.html.erb

@@ -6,8 +6,7 @@
 
   <% unless @proposal.selected? %>
     <span class="bullet">&nbsp;&bull;&nbsp;</span>
-    <span class="icon-comments"></span>&nbsp;
+    <%= render Shared::CommentsCountComponent.new(@proposal.comments_count, url: "#comments") %>
-    <%= link_to t("proposals.show.comments", count: @proposal.comments_count), "#comments" %>
   <% end %>
 
   <% if current_user %>

app/views/proposals/_proposal.html.erb

@@ -21,12 +21,12 @@
           <% cache [locale_and_user_status(proposal), "index", proposal, proposal.author] do %>
             <h3><%= link_to proposal.title, namespaced_proposal_path(proposal) %></h3>
             <p class="proposal-info">
-              <span class="icon-comments"></span>&nbsp;
-              <%= link_to t("proposals.proposal.comments", count: proposal.comments_count),
-                          namespaced_proposal_path(proposal, anchor: "comments") %>
-
-              <span class="bullet">&nbsp;&bull;&nbsp;</span>
               <%= l proposal.created_at.to_date %>
+              <span class="bullet">&nbsp;&bull;&nbsp;</span>
+              <%= render Shared::CommentsCountComponent.new(
+                proposal.comments_count,
+                url: namespaced_proposal_path(proposal, anchor: "comments")
+              ) %>
 
               <% if proposal.author.hidden? || proposal.author.erased? %>
                 <span class="bullet">&nbsp;&bull;&nbsp;</span>

app/views/topics/_topic.html.erb

@@ -5,11 +5,13 @@
     <h3><%= link_to topic.title, community_topic_path(@community, topic) %></h3>
 
     <p class="topic-info">
-      <span class="icon-comments"></span>&nbsp;
-      <%= link_to t("community.show.topic.comments", count: topic.comments_count), community_topic_path(@community, topic, anchor: "comments") %>
-      <span class="bullet">&nbsp;&bull;&nbsp;</span>
       <%= I18n.l topic.created_at.to_date %>
       <span class="bullet">&nbsp;&bull;&nbsp;</span>
+      <%= render Shared::CommentsCountComponent.new(
+        topic.comments_count,
+        url: community_topic_path(@community, topic, anchor: "comments")
+      ) %>
+      <span class="bullet">&nbsp;&bull;&nbsp;</span>
       <%= topic.author.name %>
     </p>
 

app/views/topics/show.html.erb

@@ -14,9 +14,10 @@
         <span class="bullet">&nbsp;&bull;&nbsp;</span>
         <%= l(@topic.created_at.to_date) %>
         <span class="bullet">&nbsp;&bull;&nbsp;</span>
-        <span class="icon-comments"></span>&nbsp;
+        <%= render Shared::CommentsCountComponent.new(
-        <%= link_to t("community.show.topic.comments", count: @topic.comments_count),
+          @topic.comments_count,
-                    community_topic_path(@community, @topic, anchor: "comments") %>
+          url: community_topic_path(@community, @topic, anchor: "comments")
+        ) %>
       </div>
       <br>
       <p><%= @topic.description %></p>

config/locales/en/community.yml

@@ -25,10 +25,6 @@ en:
       topic:
         edit: Edit topic
         destroy: Delete topic
-        comments:
-          one: 1 comment
-          other: "%{count} comments"
-          zero: No comments
       author: Author
       back: Back to %{community} %{proposal}
     topic:

config/locales/en/general.yml

@@ -69,10 +69,6 @@ en:
       form:
         submit_button: Start a debate
     debate:
-      comments:
-        one: 1 comment
-        other: "%{count} comments"
-        zero: No comments
       votes:
         one: 1 vote
         other: "%{count} votes"
@@ -128,10 +124,6 @@ en:
       start_new: Start a debate
     show:
       author_deleted: User deleted
-      comments:
-        one: 1 comment
-        other: "%{count} comments"
-        zero: No comments
       comments_title: Comments
       edit_debate_link: Edit
       flag: This debate has been flagged as inappropriate by several users.
@@ -396,10 +388,6 @@ en:
         edit: "Before it gets shared you'll be able to change the text as you like."
         view_proposal: Not now, go to my proposal
       already_supported: You have already supported this proposal. Share it!
-      comments:
-        one: 1 comment
-        other: "%{count} comments"
-        zero: No comments
       support: Support
       support_label: "Support %{proposal}"
       support_title: Support this proposal
@@ -420,10 +408,6 @@ en:
     show:
       author_deleted: User deleted
       code: "Proposal code:"
-      comments:
-        one: 1 comment
-        other: "%{count} comments"
-        zero: No comments
       comments_tab: Comments
       dashboard_proposal_link: Dashboard
       flag: This proposal has been flagged as inappropriate by several users.
@@ -763,6 +747,10 @@ en:
         other: "%{count} languages in use"
     optional: "Optional fields"
     required: "Required fields"
+    comments:
+      one: "1 comment"
+      other: "%{count} comments"
+      zero: "No comments"
   social:
     facebook: "%{org} Facebook"
     twitter: "%{org} Twitter"

config/locales/en/legislation.yml

@@ -4,9 +4,6 @@ en:
       comments:
         see_all: See all
         see_complete: See complete
-        comments_count:
-          one: "%{count} comment"
-          other: "%{count} comments"
         replies_count:
           one: "%{count} reply"
           other: "%{count} replies"
@@ -18,9 +15,6 @@ en:
         title: Comments
         comments_about: Comments about
         see_in_context: See in context
-        comments_count:
-          one: "%{count} comment"
-          other: "%{count} comments"
       show:
         title: Comment
       version_chooser:
@@ -92,10 +86,6 @@ en:
         form:
           leave_comment: Leave your answer
       question:
-        comments:
-          zero: No comments
-          one: "%{count} comment"
-          other: "%{count} comments"
         debate: Debate
       show:
         answer_question: Submit answer
@@ -134,10 +124,6 @@ en:
         zero: "No proposals"
         one: "%{count} proposal"
         other: "%{count} proposals"
-      comments:
-        zero: "No comments"
-        one: "%{count} comment"
-        other: "%{count} comments"
       download: "Download summary"
       top_comments:
         zero: "No comments"

config/locales/es/general.yml

@@ -69,10 +69,6 @@ es:
       form:
         submit_button: Empieza un debate
     debate:
-      comments:
-        zero: Sin comentarios
-        one: 1 Comentario
-        other: "%{count} Comentarios"
       votes:
         zero: Sin votos
         one: 1 voto
@@ -128,10 +124,6 @@ es:
       start_new: Empezar un debate
     show:
       author_deleted: Usuario eliminado
-      comments:
-        zero: Sin comentarios
-        one: 1 Comentario
-        other: "%{count} Comentarios"
       comments_title: Comentarios
       edit_debate_link: Editar debate
       flag: Este debate ha sido marcado como inapropiado por varios usuarios.
@@ -396,10 +388,6 @@ es:
         edit: "Antes de que se publique podrás modificar el texto a tu gusto."
         view_proposal: Ahora no, ir a mi propuesta
       already_supported: '¡Ya has apoyado esta propuesta, compártela!'
-      comments:
-        zero: Sin comentarios
-        one: 1 Comentario
-        other: "%{count} Comentarios"
       support: Apoyar
       support_label: "Apoyar %{proposal}"
       support_title: Apoyar esta propuesta
@@ -420,10 +408,6 @@ es:
     show:
       author_deleted: Usuario eliminado
       code: "Código de la propuesta:"
-      comments:
-        zero: Sin comentarios
-        one: 1 Comentario
-        other: "%{count} Comentarios"
       comments_tab: Comentarios
       dashboard_proposal_link: Panel de control
       flag: Esta propuesta ha sido marcada como inapropiada por varios usuarios.
@@ -763,6 +747,10 @@ es:
         other: "%{count} idiomas en uso"
     optional: "Campos opcionales"
     required: "Campos obligatorios"
+    comments:
+      one: "1 comentario"
+      other: "%{count} comentarios"
+      zero: "Sin comentarios"
   social:
     facebook: "Facebook de %{org}"
     twitter: "Twitter de %{org}"

config/locales/es/legislation.yml

@@ -4,9 +4,6 @@ es:
       comments:
         see_all: Ver todos
         see_complete: Ver completo
-        comments_count:
-          one: "%{count} comentario"
-          other: "%{count} comentarios"
         replies_count:
           one: "%{count} respuesta"
           other: "%{count} respuestas"
@@ -18,9 +15,6 @@ es:
         title: Comentarios
         comments_about: Comentarios sobre
         see_in_context: Ver en contexto
-        comments_count:
-          one: "%{count} comentario"
-          other: "%{count} comentarios"
       show:
         title: Comentario
       version_chooser:
@@ -92,10 +86,6 @@ es:
         form:
           leave_comment: Deja tu respuesta
       question:
-        comments:
-          zero: Sin comentarios
-          one: "%{count} comentario"
-          other: "%{count} comentarios"
         debate: Debate
       show:
         answer_question: Enviar respuesta
@@ -134,10 +124,6 @@ es:
         zero: "No hay propuestas"
         one: "%{count} propuesta"
         other: "%{count} propuestas"
-      comments:
-        zero: "No hay comentarios"
-        one: "%{count} comentario"
-        other: "%{count} comentarios"
       download: "Descargar resumen"
       top_comments:
         zero: "No hay comentarios"

spec/components/shared/comments_count_component_spec.rb

@@ -0,0 +1,17 @@
+require "rails_helper"
+
+describe Shared::CommentsCountComponent do
+  it "renders a link when a URL is given" do
+    render_inline Shared::CommentsCountComponent.new(0, url: "http://www.url.com")
+
+    expect(page.find(".comments-count")).to have_content "No comments"
+    expect(page.find(".comments-count")).to have_link "No comments", href: "http://www.url.com"
+  end
+
+  it "renders plain text when no URL is given" do
+    render_inline Shared::CommentsCountComponent.new(1)
+
+    expect(page.find(".comments-count")).to have_content "1 comment"
+    expect(page.find(".comments-count")).not_to have_link "1 comment"
+  end
+end

spec/system/budgets/investments_spec.rb

@@ -62,7 +62,11 @@ describe "Budget Investments" do
     investments.each do |investment|
       within("#budget-investments") do
         expect(page).to have_content investment.title
-        expect(page).to have_css("a[href='#{budget_investment_path(budget, id: investment.id)}']", text: investment.title)
+        expect(page).to have_content investment.comments_count
+        expect(page).to have_link "No comments", href: budget_investment_path(budget, id: investment.id,
+                                                                                      anchor: "comments")
+        expect(page).to have_link investment.title, href: budget_investment_path(budget, id: investment.id)
+
         expect(page).not_to have_content(unfeasible_investment.title)
       end
     end
@@ -849,6 +853,7 @@ describe "Budget Investments" do
     expect(page).to have_content(investment.title)
     expect(page).to have_content(investment.description)
     expect(page).to have_content(investment.author.name)
+    expect(page).to have_content(investment.comments_count)
     expect(page).to have_content(investment.heading.name)
     within("#investment_code") do
       expect(page).to have_content(investment.id)

spec/system/legislation/questions_spec.rb

@@ -109,5 +109,15 @@ describe "Legislation" do
 
       expect(page).not_to have_selector(:link_or_button, "Submit answer")
     end
+
+    scenario "render link to questions comments with anchor" do
+      question = create(:legislation_question, process: process, title: "Question without comments")
+
+      visit legislation_process_path(process)
+
+      expect(page).to have_link "No comments", href: legislation_process_question_path(process,
+                                                                                       question,
+                                                                                       anchor: "comments")
+    end
   end
 end