[PATCH] Detecting notification popup server capabilities

Adriaan de Groot groot at kde.org
Fri Sep 4 09:54:47 BST 2009


On Friday 04 September 2009 12:05:02 am John Tapsell wrote:
>  notify->setActions( i18n("View,Ignore").split(',') );

Relies on , being preserved in the translations and would need a lot of 
context to make this understandable to translators. Use a QStringList instead, 
even if it's more typing for the developer.

> I can't think of a decent example where you'd want different text when
> no actions are available, but maybe also useful so that you could do
> notify->setNoActionText("")  or something for the action to be ignored

Do not use "" when you mean QString().

Yes, you're brainstorming API. Even so, there's API consistency to be thought 
of.

[ade]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 489 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090904/4c9b2e1a/attachment.sig>


More information about the kde-core-devel mailing list