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

List:       kde-core-devel
Subject:    Re: [PATCH] Shortcut for Show Desktop
From:       Ellis Whitehead <kde () ellisw ! net>
Date:       2001-06-25 2:22:18
[Download RAW message or body]

Hi Carsten,

> Besides that, the following made me a bit curious:
> keys->insertItem(i18n("Panel"), "Program:kicker", 0);
>
> I guess this is for grouping all those entries in the keybindings control
> module? Why is it misusing insertItem() for that?

Doing it "right" will require redesigning kcontrol/keys and altering some 
KGlobalAccel member variables.  It was too close to freeze to make such 
changes, though, so the insertItem() abomination is just a temporary 
solution. ;)  Presuming that I don't get kidnapped by international 
terrorists or burned at the stake by rabid farmers (niether of which are 
common occurances in urban Michigan, USA), this will be on the top 
of my priority list for the next KDE release.

Cheers,
Ellis

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