Discussion:
[klipper] [Bug 390375] New: Klipper notifications visually broken since plasma 5.12
Add Reply
Till Schäfer
2018-02-13 13:38:47 UTC
Reply
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=390375

Bug ID: 390375
Summary: Klipper notifications visually broken since plasma
5.12
Product: klipper
Version: 5.12.0
Platform: Gentoo Packages
OS: Linux
Status: UNCONFIRMED
Severity: normal
Priority: NOR
Component: general
Assignee: ***@kde.org
Reporter: ***@uni-dortmund.de
Target Milestone: ---

Created attachment 110607
--> https://bugs.kde.org/attachment.cgi?id=110607&action=edit
screenshot of visually broken notification

After upgrading to plasma 5.12, the klipper notifications, which are issued
while changing the clipboard content to the next/previous history item are
visually broken. It seems that the newlines and the horizontal alignment are
gone and all text is just written in one line.

See the attached screenshot.
--
You are receiving this mail because:
You are watching all bug changes.
Martin Flöser
2018-02-13 19:35:32 UTC
Reply
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=390375

Martin Flöser <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |1.0
CC| |plasma-***@kde.org
Component|general |Notifications
Assignee|***@kde.org |***@privat.broulik.de
Product|klipper |plasmashell
--
You are receiving this mail because:
You are watching all bug changes.
David Edmundson
2018-02-14 01:05:50 UTC
Reply
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=390375

David Edmundson <***@davidedmundson.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@davidedmundson.co.uk

--- Comment #1 from David Edmundson <***@davidedmundson.co.uk> ---
Klipper is sending a table as text. See Klipper::cycleText

They aren't HTML tags that were ever permissable in the notification
specification. Klipper is in the wrong.

Since 5.12 we filter to only the tags that are allowed.

From a security POV, we could re-allow the table with the attribute filtering
we now have and it'd be safe. But I don't know if I want to.
--
You are receiving this mail because:
You are watching all bug changes.
Loading...