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

Rafał Miłecki zajec5 at gmail.com
Thu Mar 13 23:05:25 CET 2008


2008/3/12, Aaron J. Seigo <aseigo at kde.org>:
> cool =) i look forward to seeing the update.
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:
enum DisplayedText
{
Number,
Name,
None
};

>  also, we tend to use the review board on http://mattr.info for patch
>  submission. it prevents them getting lost in the nether regions of this list
>  and provides a nicer interface for commenting on patches. it's the prefered
>  place for patches (and everything get's cc'd to this list), though it's
>  ultimately up to you of course.
OK. In case of having next ideas for impoveming KDE, I'll use this
board at next time. Thanks again for tip.

-- 
Rafał Miłecki
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pager.desktops.names-1.1.patch
Type: application/octet-stream
Size: 10595 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080313/42dac34e/attachment.obj 


More information about the Panel-devel mailing list