Skip to content

Commit 7353d83

Browse files
committed
Remove no-op tooltip icon definition
Button tooltip does not support icons, futher the linked code would fail with a NoMethodError as `EmojiReactions.emoji` is not defined (note the plural definition) which should be singular `EmojiReaction.emoji` This was just forgotten, never gets executed.
1 parent 306b24c commit 7353d83

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

app/components/work_packages/activities_tab/journals/item_component/reactions.html.erb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@
2222
classes: "op-reactions-button"
2323
)) do |button|
2424
button.with_tooltip(text: number_of_user_reactions_text(data[:users]),
25-
test_selector: "reaction-tooltip-#{reaction}") do
26-
button.with_icon(EmojiReactions.emoji(reaction), size: :small)
25+
test_selector: "reaction-tooltip-#{reaction}")
2726
end
2827
"#{EmojiReaction.emoji(reaction)} #{data[:count]}"
2928
end

0 commit comments

Comments
 (0)