[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-08-04 Thread Nicolas Fella
https://bugs.kde.org/show_bug.cgi?id=423852

Nicolas Fella  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|REOPENED|RESOLVED

--- Comment #5 from Nicolas Fella  ---


*** This bug has been marked as a duplicate of bug 422616 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-08-03 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=423852

bl...@vivaldi.net changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|RESOLVED|REOPENED
 Resolution|DUPLICATE   |---

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-08-03 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=423852

bl...@vivaldi.net changed:

   What|Removed |Added

 CC||bl...@vivaldi.net

--- Comment #4 from bl...@vivaldi.net ---
(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

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-07-04 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=423852

David Edmundson  changed:

   What|Removed |Added

 CC||k...@davidedmundson.co.uk

--- Comment #3 from David Edmundson  ---
>- 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.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-07-04 Thread Nicolas Fella
https://bugs.kde.org/show_bug.cgi?id=423852

Nicolas Fella  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|REPORTED|RESOLVED

--- Comment #2 from Nicolas Fella  ---


*** This bug has been marked as a duplicate of bug 422616 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 423852] Visual bug, various apps' notifications show HTML code

2020-07-04 Thread Nicolas Fella
https://bugs.kde.org/show_bug.cgi?id=423852

Nicolas Fella  changed:

   What|Removed |Added

 CC||nicolas.fe...@gmx.de

--- Comment #1 from Nicolas Fella  ---
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

-- 
You are receiving this mail because:
You are watching all bug changes.