Project

General

Profile

Actions

Feature #31531

closed

Change quote note icon to quotation mark icon

Added by Mizuki ISHIKAWA about 6 years ago. Updated 9 days ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

You can quote a note by clicking the comment icon.

I think this icon is not general as an icon representing a quote.
https://fontawesome.com/icons?d=gallery&q=quote
https://octicons.github.com/icon/quote/
https://material.io/tools/icons/?search=quote&icon=format_quote&style=baseline

I suggest changing to the Quotation marks that are used on many websites.
e.g.


Files

Actions #2

Updated by Go MAEDA 7 months ago

Since Redmine 6.0.0 started using Tabler Icons (#23980), the following icon would currently be a candidate for the new "Quote" icon.
https://tabler.io/icons/icon/quote

Actions #3

Updated by Mizuki ISHIKAWA 5 months ago

I replaced it with https://tabler.io/icons/icon/quote.
I think it's a good improvement as it makes the difference between regular comments and quotes clearer.

Actions #4

Updated by Mizuki ISHIKAWA 3 months ago

I have also created a patch for a filled quote icon version, considering the possibility that the quote icon may look like the number 9.
Since this is the first time we have included filled style icons in Redmine, we have made some changes for this purpose.

Actions #6

Updated by Marius BĂLTEANU about 2 months ago

  • Assignee set to Marius BĂLTEANU
  • Target version set to 6.1.0
Actions #7

Updated by Marius BĂLTEANU 11 days ago

  • Subject changed from Change quote note icon to quotation marks image to Change quote note icon to quotation mark icon
  • Status changed from New to Resolved
  • Resolution set to Fixed

Committed with a few changes.

Actions #8

Updated by Katsuya HIDAKA 10 days ago

Looks good. However, the following system test has been failing since commit r23809:
https://github.com/redmine/redmine/actions/runs/15354807677/job/43211407573#step:5:33

Failure:
ReactionsSystemTest#test_reaction_button_is_visible_on_property_changes_tab [test/system/reactions_test.rb:146]:
expected to find css "a.icon-comment" within #<Capybara::Node::Element tag="div" path="/HTML/BODY[1]/DIV[1]/DIV[4]/DIV[2]/DIV[3]/DIV[2]/DIV[1]"> but there were no matches

bin/rails test test/system/reactions_test.rb:116

Here's the fix:

diff --git a/test/system/reactions_test.rb b/test/system/reactions_test.rb
index cd3e2e871..96dd4cf81 100644
--- a/test/system/reactions_test.rb
+++ b/test/system/reactions_test.rb
@@ -129,7 +129,7 @@ class ReactionsSystemTest < ApplicationSystemTestCase
     within('#change-1') do
       assert_selector 'a.reaction-button'

-      assert_no_selector 'a.icon-comment'
+      assert_no_selector 'a.icon-quote'
       assert_no_selector 'span.drdn'
     end
     within("#change-#{journal_without_notes.id}") do
@@ -143,14 +143,14 @@ class ReactionsSystemTest < ApplicationSystemTestCase
     within('#change-1') do
       assert_selector 'a.reaction-button'

-      assert_selector 'a.icon-comment'
+      assert_selector 'a.icon-quote'
       assert_selector 'span.drdn'
     end
     within("#change-#{journal_without_notes.id}") do
       assert_selector 'a.reaction-button'
       assert_selector 'span.drdn'

-      assert_no_selector 'a.icon-comment'
+      assert_no_selector 'a.icon-quote'
     end
   end

Actions #9

Updated by Go MAEDA 10 days ago

Katsuya HIDAKA wrote in #note-8:

Looks good. However, the following system test has been failing since commit r23809:
https://github.com/redmine/redmine/actions/runs/15354807677/job/43211407573#step:5:33

Committed the fix in r23810. Thank you.

Actions #10

Updated by Marius BĂLTEANU 10 days ago

Thanks!

Actions #11

Updated by Marius BĂLTEANU 9 days ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF