[Panel-devel] kwallet support for twitter

[Chani]

On December 21, 2007 13:06:21 Aaron J. Seigo wrote:
> On Thursday 20 December 2007, Chani wrote:
> > ok, I've got it working now.
> > two problems:
> > if it fails to write to the wallet, it silently writes to the config.
> > need to add a confirmation; what's the standard way to do that?
>
> no standard way. just fail silently back to the config file imho.

once it gets into the config file you have to remove the plasmoid to get it 
out, and personally I don't want my password there if I mistype my kwallet 
password too many times or something - so I added a confirmation.

>
> > if I try to open the wallet in init(), it fails because there's no view
> > and kwallet requires a window id. I hacked around this with a timer, but
> > I don't like it.
>
> pass in 0 for the WId.

okey dokey.

I've found a bug: adding a new twitter plasmoid doesn't give it a sane default 
size. I guess I should set something in the constructor.

-- 
This message brought to you by evyl bananas, and the number 3.
www.chani3.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: twitter.diff
Type: text/x-diff
Size: 7541 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071221/ed621cbc/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071221/ed621cbc/attachment.pgp