https://bugs.kde.org/show_bug.cgi?id=493482

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |natalie_clar...@yahoo.de
          Component|Notifications               |Session Management

--- Comment #2 from Nate Graham <n...@kde.org> ---
Found the thing that controls this:
https://invent.kde.org/frameworks/frameworkintegration/-/blob/master/plasma_workspace.notifyrc?ref_type=heads#L1123

But it has Name= and Comment= keys set to valid text as expected, so it looks
like it *should* have the right data.

But then here's the code that actually shows the notification:
https://invent.kde.org/plasma/plasma-workspace/-/blob/master/ksmserver/logout.cpp?ref_type=heads#L370

Oops! We pass an empty string to the KNotification::event() constructor, so the
text set in the notifyrc file is ignored. We could simply re-state the text
there, but it would be better to use the text in the notifyrc file. I'm not
sure how to do that; will need to look into it.

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

Reply via email to