Summary: | Visual bug, various apps' notifications show HTML code | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | gadzinisko |
Component: | System Tray | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED DUPLICATE | ||
Severity: | normal | CC: | blaze, kde, nicolas.fella |
Priority: | NOR | ||
Version: | 5.18.5 | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
gadzinisko
2020-07-03 20:28:32 UTC
I can confirm this for Clementine, haven't checked the others. According to the statusnotifier spec (https://freedesktop.org/wiki/Specifications/StatusNotifierItem/StatusNotifierItem/) it is legal to have HTML markup in the tooltip subtitle, but not in the tooltip title. The Plasma code adheres to that and does not allow HTML markup in the title. Clementine however sets a tooltip title with HTML markup and an empty tooltip subtitle, so it violates the specification. Plasma could therefore: - Do nothing and blame Clementine since that's where this should be addressed - Deviate from the spec and allow HTML markup in the title - Strip away HTML markup in the title Given that the latter two may result in undesirable unforseen results I'm inclined to advocate for the first option. See also https://github.com/clementine-player/Clementine/issues/6733 and https://bugs.kde.org/show_bug.cgi?id=422616 *** This bug has been marked as a duplicate of bug 422616 *** >- Do nothing and blame Clementine since that's where this should be addressed
Yes please. Even if we change you still have a bug in the app - if we deviate from the spec you end up with apps coding for that.
(In reply to David Edmundson from comment #3) > >- Do nothing and blame Clementine since that's where this should be addressed > > Yes please. Even if we change you still have a bug in the app - if we > deviate from the spec you end up with apps coding for that. That's not how you do this. You should fix QSystemTray first, patch every possible distro and only THEN break the UX when the impact is minimal. I do not see how the issue is "resolved" for now *** This bug has been marked as a duplicate of bug 422616 *** |