diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss index d551462b22..bb41eda51c 100644 --- a/app/assets/stylesheets/common.scss +++ b/app/assets/stylesheets/common.scss @@ -64,7 +64,13 @@ time[title] { .table-success { --bs-table-bg: rgb(var(--bs-success-rgb), .25); } - .table-primary, .table-secondary, .table-success { + .table-warning { + --bs-table-bg: rgb(var(--bs-warning-rgb), .25); + } + .table-danger { + --bs-table-bg: rgb(var(--bs-danger-rgb), .25); + } + .table-primary, .table-secondary, .table-success, .table-warning, .table-danger { --bs-table-color: initial; border-color: inherit; } diff --git a/app/views/notes/index.html.erb b/app/views/notes/index.html.erb index d5efe0d79c..03e112f279 100644 --- a/app/views/notes/index.html.erb +++ b/app/views/notes/index.html.erb @@ -1,9 +1,11 @@ <% content_for :heading do %>

<%= t ".heading", :user => @user.display_name %>

-

<%= t ".subheading_html", - :user => link_to(@user.display_name, @user), - :submitted => tag.span(t(".subheading_submitted"), :class => "px-2 py-1 bg-primary bg-opacity-25"), - :commented => tag.span(t(".subheading_commented"), :class => "px-2 py-1 bg-body") %>

+

<%= t ".legend_html", + :user => link_to(@user.display_name, @user), + :created => tag.span(t(".legend_created"), :class => "px-2 py-1 bg-danger bg-opacity-25"), + :created_and_resolved => tag.span(t(".legend_created_and_resolved"), :class => "px-2 py-1 bg-warning bg-opacity-25"), + :resolved => tag.span(t(".legend_resolved"), :class => "px-2 py-1 bg-success bg-opacity-25"), + :interacted => tag.span(t(".legend_interacted"), :class => "px-2 py-1 bg-body") %>

<% end %> <% if @notes.empty? %> @@ -24,7 +26,15 @@ <% @notes.each do |note| -%> - class="table-primary"<% end %>> + <% opened_by_user = note.author == @user %> + <% closed_by_user = note.comments.last&.author == @user && note.comments.last&.event == "closed" %> + <%= tag.tr :class => if opened_by_user && closed_by_user + "table-warning" + elsif opened_by_user + "table-danger" + elsif closed_by_user + "table-success" + end do %> <% if note.closed? %> <%= image_tag("closed_note_marker.svg", :alt => "closed", :width => 25, :height => 40) %> @@ -37,7 +47,7 @@ <%= note.comments.first.body.to_html %> <%= friendly_date_ago(note.created_at) %> <%= friendly_date_ago(note.updated_at) %> - + <% end %> <% end -%> diff --git a/config/locales/en.yml b/config/locales/en.yml index 20588e5a89..ff1f861b6b 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -2957,9 +2957,11 @@ en: index: title: "Notes submitted or commented on by %{user}" heading: "%{user}'s Notes" - subheading_html: "Notes %{submitted} or %{commented} by %{user}" - subheading_submitted: "submitted" - subheading_commented: "commented on" + legend_html: "Notes %{created}, %{created_and_resolved}, %{resolved} or %{interacted} by %{user}" + legend_created: "created" + legend_created_and_resolved: "created and resolved" + legend_resolved: "resolved" + legend_interacted: "with other interactions" no_notes: No notes id: "Id" creator: "Creator" diff --git a/test/controllers/notes_controller_test.rb b/test/controllers/notes_controller_test.rb index 4092ad7326..4d476737e5 100644 --- a/test/controllers/notes_controller_test.rb +++ b/test/controllers/notes_controller_test.rb @@ -67,6 +67,69 @@ def test_index_success assert_response :not_found end + def test_index_interaction_created + user = create(:user) + create(:note) do |note| + create(:note_comment, :note => note, :author => user) + end + + get user_notes_path(user) + assert_response :success + assert_dom "table.note_list tbody tr.table-danger:not(.table-warning):not(.table-success)", :count => 1 + end + + def test_index_interaction_created_and_resolved + user = create(:user) + create(:note) do |note| + create(:note_comment, :note => note, :author => user) + create(:note_comment, :note => note, :author => user, :event => "closed") + end + + get user_notes_path(user) + assert_response :success + assert_dom "table.note_list tbody tr:not(.table-danger).table-warning:not(.table-success)", :count => 1 + end + + def test_index_interaction_resolved + user = create(:user) + other_user = create(:user) + create(:note) do |note| + create(:note_comment, :note => note, :author => other_user) + create(:note_comment, :note => note, :author => user, :event => "closed") + end + + get user_notes_path(user) + assert_response :success + assert_dom "table.note_list tbody tr:not(.table-danger):not(.table-warning).table-success", :count => 1 + end + + def test_index_interaction_resolved_and_reopened_by_other_user + user = create(:user) + other_user = create(:user) + create(:note) do |note| + create(:note_comment, :note => note, :author => other_user) + create(:note_comment, :note => note, :author => user, :event => "closed") + create(:note_comment, :note => note, :author => other_user, :event => "reopened") + end + + get user_notes_path(user) + assert_response :success + assert_dom "table.note_list tbody tr:not(.table-danger):not(.table-warning):not(.table-success)", :count => 1 + end + + def test_index_interaction_commented + user = create(:user) + other_user = create(:user) + create(:note) do |note| + create(:note_comment, :note => note, :author => other_user) + create(:note_comment, :note => note, :author => user, :event => "commented") + end + + get user_notes_path(user) + assert_response :success + assert_dom "table.note_list tbody tr:not(.table-danger):not(.table-warning):not(.table-success)", :count => 1 + end + def test_index_paged user = create(:user)