Feature #39529
open
Add tooltips for clarifying non-notifiable user mentions
0%
Description
Currently, when a user without permission to view a particular issue is mentioned, they do not receive a notification email. However, visually, the mention link appears identical to others, which can be misleading as there's no indication that no email was sent.
In issue Feature #39528: Change mention link color for users without access to view issues, I proposed changing the background color of mention links for such users to make it clear they are non-notifiable. However, I believe that visual change alone may not sufficiently convey the intent to users. Therefore, I suggest using tooltips to display a message clarifying the status of the mention.
Yet, since tooltips are only used for displaying title attributes right now, expanding their use might require some discussion, which is why I have created a separate ticket for this matter.
Files
Related issues
Updated by Mizuki ISHIKAWA about 1 year ago
- File feature-39528.patch feature-39528.patch added
Updated by Mizuki ISHIKAWA about 1 year ago
- File feature-39529.patch feature-39529.patch added
- File screenshot.png screenshot.png added
Sorry, https://www.redmine.org/attachments/31460 file was attached by mistake.
After applying the attached patch, the tooltip will appear as shown in the screenshot.
Updated by Mizuki ISHIKAWA 12 months ago
- File feature-39529-v2.patch feature-39529-v2.patch added
- File screenshot 2024-01-04 16.00.10.png screenshot 2024-01-04 16.00.10.png added
- File screenshot 2024-01-04 16.00.20.png screenshot 2024-01-04 16.00.20.png added
I changed it based on the UI for when there is an invalid watcher.
Updated by Go MAEDA 4 months ago
- File feature-39529-v3.patch feature-39529-v3.patch added
I have modified the patch with the following changes:
- Shortened long lines of code
- Fixed a missing translation error for
:notice_mention_watcher
Updated by Go MAEDA 4 months ago
- Related to Feature #39528: Change mention link color for users without access to view issues added
Updated by Marius BĂLTEANU 4 months ago
Maybe we can improve the current translation or add a new one that can be used by both mention and watcher.
Updated by Go MAEDA 4 months ago
Marius BĂLTEANU wrote in #note-8:
Maybe we can improve the current translation or add a new one that can be used by both mention and watcher.
Do you mean something like this?
notice_invalid_notification: "Invalid %{type}: User will not receive %{type} notifications because they do not have access to view this object."
Updated by Go MAEDA about 2 months ago
Marius BĂLTEANU wrote in #note-8:
Maybe we can improve the current translation or add a new one that can be used by both mention and watcher.
I would like to commit this patch for 6.0.0 without modifying the translation.
How about we consider unifying the translations in 6.1.0?
Unification like #note-9 may cause a capitalization issue reported in #37072.
Updated by Go MAEDA about 2 months ago
- Target version changed from 6.0.0 to 6.1.0
The patch should be updated to use a vector icon instead of app/assets/images/warning.png. See #23980.