reminder: KGlobalSettings cleanup
Matthew Woehlke
mw_triad at users.sourceforge.net
Wed Jun 20 17:25:54 BST 2007
Matthew Woehlke wrote:
> createApplicationPalette(KConfigGroup,int) - currently broken, probably
> needs to take a KConfig instead. KColorScheme needs to change if this is
> going to work at all. Used only by KDM which may genuinely need
> something like it. (It's BC to re-add something, not BC to remove a
> useless method!)
Right. This *needs* to die, it is broken and (newly) wrong. Instead
there should be createApplicationPalette(KConfig), and of course
KColorScheme needs to (optionally) take a KConfig also, where appropriate.
A week ago I would have said that colorscm should be using this also
(not sure why it isn't!), but now KGS doesn't do anything but hand off
to KColorScheme... :-) (But in the future, it would be nice to use a UI
preview ala the styles cm, possible even the same one, and then we would
want the KGS overload again.)
--
Matthew
"Nobody expects the Spanish Inquisition!" -- Monty Python
More information about the kde-core-devel
mailing list