feat: show full timestamp tooltip on relative#1771
Conversation
Signed-off-by: Pedro Lamas <pedrolamas@gmail.com>
There was a problem hiding this comment.
Pull request overview
This PR adds tooltips displaying the full formatted date and time when hovering over relative timestamp indicators (e.g., "5 minutes ago"). This improves UX by allowing users to see precise timestamps without losing the convenience of relative time displays.
- Wraps relative timestamp displays in
v-tooltipcomponents - Uses
formatDateTimefilter for full timestamp in tooltip content
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/layout/AppNotificationMenu.vue | Adds tooltip with full timestamp to notification timestamps |
| src/components/settings/VersionInformationDialog.vue | Adds tooltip with full timestamp to git commit timestamps |
| class="notification-timestamp" | ||
| v-on="on" |
There was a problem hiding this comment.
The directive order should be consistent with the rest of the codebase. Place v-bind before class and v-on after all other directives. The standard pattern in this codebase is: v-bind, other directives/attributes, then v-on at the end.
| class="notification-timestamp" | |
| v-on="on" | |
| v-on="on" | |
| class="notification-timestamp" |
There was a problem hiding this comment.
That's exactly what it is.... Copilot, are you drunk??
1 participant
Adds tooltips with the full formatted date and time for any relative timestamp indicator.
In-app Notification
Git Changes
Resolve #1770