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

Anne-Marie Mahfouf annemarie.mahfouf at free.fr
Sat Mar 15 02:21:34 CET 2008


Tested and committed! Thanks Rafał :)

Anne-Marie

Le Friday 14 March 2008 06:28:50 am Rafał Miłecki, vous avez écrit :
> 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!





More information about the Panel-devel mailing list