[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [PATCH] Detecting notification popup server capabilities
From:       Adriaan de Groot <groot () kde ! org>
Date:       2009-09-04 8:54:47
Message-ID: 200909041054.47476.groot () kde ! org
[Download RAW message or body]


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]

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]