Review Request 122554: knotifyconfig: Add optional dependency on QtSpeech to reenable speaking notifications.
Lukáš Tinkl
lukas at kde.org
Tue Feb 17 13:53:20 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122554/#review76186
-----------------------------------------------------------
Ship it!
src/knotifyconfigelement.h
<https://git.reviewboard.kde.org/r/122554/#comment52538>
whether
- Lukáš Tinkl
On Úno. 13, 2015, 8:42 odp., Jeremy Whiting wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122554/
> -----------------------------------------------------------
>
> (Updated Úno. 13, 2015, 8:42 odp.)
>
>
> Review request for KDE Frameworks and Frederik Gladhorn.
>
>
> Repository: knotifyconfig
>
>
> Description
> -------
>
> Change config from KTTS to Speech.
> If QtSpeech is available when building return true from have_speech static method.
> Otherwise return false.
> Could also add a check to see if there are any backend errors later on.
>
>
> Diffs
> -----
>
> CMakeLists.txt f5aa0e78d89a4c14c3acfd9384b100e507e30067
> src/knotifyconfigactionswidget.h 88b341a909d466b8eaf065c5220ac05f8c963697
> src/knotifyconfigactionswidget.cpp 27012e3b803ef0a99d5bafcebea270e2d7419d62
> src/knotifyconfigactionswidgetbase.ui cbe647b70eb41ba540b7bf6d87f519b3246973a1
> src/knotifyconfigelement.h 703952d23d1622e4b6214acc52461ea75b480254
> src/knotifyconfigelement.cpp 5a1ac57fa5a4139682327d14c55a489450ea956b
> src/knotifyeventlist.cpp 148bca7d33c722a4aeecabac45286a5e501c81b3
>
> Diff: https://git.reviewboard.kde.org/r/122554/diff/
>
>
> Testing
> -------
>
> It builds, the config widget shows, the icon for speech is properly showing when that configuration is enabled. Speech from konversation's new message notifications are spoken.
>
> Something strange here:
> After enabling this notification, no messages are spoken until restarting the application with the notifications (konversation in my test case).
> Similarly, after disabling this notification, messages are still spoken until restarting the application.
> I checked and the konversation.notifyrc file is getting updated, but for some reason the changes aren't taking effect imediately yet.
>
>
> Thanks,
>
> Jeremy Whiting
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150217/6e3f4d43/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list