-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123173/#review78175
-----------------------------------------------------------


>[Edit] Looks like the notification is working if the plasmoid is added 
>somewhere but not from within plasmawindowed.. any ideas?

KNotification checks the app's name and notification ID (see systemsettings -> 
notifications -> plasma workspace)
If that says "show a popup" it shows a popup. 
In no entry exists it will do nothing.

When called from plasmawindowed it will look up the wrong notification config 
and do nothing.
(maybe we should fake the componentdata in plasmawindowed so it pretends to be 
the shell for that reason)

- David Edmundson


On March 29, 2015, 2:20 p.m., Bernhard Friedreich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123173/
> -----------------------------------------------------------
> 
> (Updated March 29, 2015, 2:20 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> -------
> 
> Commit 1: Fixed codingstyle
> Commit 2: Fixed warning "Unable to assign int to QEasingCurve"
> Commit 3: Added notification on timer completion (not working yet)
> 
> 
> Diffs
> -----
> 
>   applets/timer/package/contents/ui/TimerDigit.qml 
> 94ec17d2c1e6fb02f6d8d3b1c0ea75f16b0fa405 
>   applets/timer/package/contents/ui/timer.qml 
> 09df67004f07352436a30195fb13c2550f8ebce7 
> 
> Diff: https://git.reviewboard.kde.org/r/123173/diff/
> 
> 
> Testing
> -------
> 
> Event is fired when the timer is finished (console.log output works) but the 
> notification won't show up.
> 
> I get this message on console: Are we the only client? (Both have to be true) 
>  false true
> 
> How can I send a notification from qml (without c++)? Am I on the right track?
> 
> [Edit] Looks like the notification is working if the plasmoid is added 
> somewhere but not from within plasmawindowed.. any ideas?
> 
> 
> Thanks,
> 
> Bernhard Friedreich
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to