[PATCH] Bug 158444: Pager can't display name of desktop

[Rafał Miłecki]

2008/3/13, Rafał Miłecki <zajec5 at gmail.com>:
> It's ready. Thanks for your tip about casting enum<->int. I wasn't
>  sure which style of enum should I use. Some apps in KDE use UPPER_CASE
>  values and other use First_letter_upper_only. Hope it doesn't matter
>  so much and patch may be accepted. Finally I used:

Ahh, this patch introduced some bug. After opening configure dialog a
few times there is list like a:

Desktop Number
Desktop Name
None
Desktop Number
Desktop Name
None
Desktop Number
Desktop Name
None

I've forgotten simple:
ui.displayedTextComboBox->clear();
line. Shame on me...

I send patch version 1.2 and really hope it's fine this time. Sorry
for a problem and mess!

-- 
Rafał Miłecki
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pager.desktops.names-1.2.patch
Type: application/octet-stream
Size: 10359 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080314/3371089e/attachment-0001.obj