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

Flossy Cat <flossy-...@online.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression, usability

--- Comment #8 from Flossy Cat <flossy-...@online.de> ---
(In reply to Nate Graham from comment #5)
> Got it, thanks. I can see how that would be useful for powerful custom
> workflows.

Some technicalities: 

You set this bug report as "wish list", but it is NOT a FEATURE REQUEST.
It is about a REGRESSION, breaking existing usage and use-cases – as we
discussed.

Further the related bug report https://bugs.kde.org/show_bug.cgi?id=481068 is
about an actually buggy (incomplete) implementation.

If the REGRESSION is avoided by not removing the feature, the code handling the
%-escapes needs fixing.

Further the code should be improved by providing better provision of the
notification information to the executed process.
I suggest generally providing all information via 
* environment variables to the called process in the likeness of
»KDE_notifier_cmd_«_"PARAMETER" where parameter is "Title", "WinID", etc.
* STDIN – simply the complete visible text in the notification window, perhaps
amended with additional info available (a "key=value" version is already
provided by the environment variables)

I consider it much simpler to provide all these parameter passing in parallel
than have a GUI component for selection.

Finally the functionality NEEDS to be documented: KDE can't expect widespread
use (or even discovery) of functionality like this, if there is no
documentation …

I can easily support in doing
1. tests
2. the English and German documentation

I can support with coding if guided (I know C++ well enough, but not the KDE
libraries).

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

Reply via email to