Feature #31531
closedChange quote note icon to quotation mark icon
0%
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
Updated by Mizuki ISHIKAWA about 6 years ago
Updated by Mizuki ISHIKAWA 5 months ago
- File 0001-Change-quote-icon.patch 0001-Change-quote-icon.patch added
- File screenshot 2025-01-15 17.02.13.png screenshot 2025-01-15 17.02.13.png added
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.
Updated by Mizuki ISHIKAWA 3 months ago
- File 0001-Change-the-quote-icon-to-filled-quote-icon.patch 0001-Change-the-quote-icon-to-filled-quote-icon.patch added
- File screenshot 2025-03-03 15.43.31.png screenshot 2025-03-03 15.43.31.png added
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.
Updated by Mizuki ISHIKAWA 3 months ago
- File 0001-Change-the-quote-icon-to-filled-quote-icon.patch 0001-Change-the-quote-icon-to-filled-quote-icon.patch added
Slightly modified patch.
Updated by Marius BĂLTEANU about 2 months ago
- Assignee set to Marius BĂLTEANU
- Target version set to 6.1.0
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.
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