remove QtGui dependence of kdecore

[Olivier Goffart]

Le mercredi 3 janvier 2007 20:55, Aaron J. Seigo a écrit :
> On Wednesday 03 January 2007 6:30, Olivier Goffart wrote:
> > - KDebug
> >     has an operator with QWidget,  could be moved into kdeui, but in
> > which include ?   probably better to remove it,  it just add the geometry
>
> it also prints out the name and is a rather useful feature of KDebug. 

But the QObject operator would do that if the QWidget operator is not present.

> it would be nice to move this to a commonly included header in kdeui. or
> even create a kdebugui.h or similar in kdeui/kernel though i'm not sure what
> we'd add to it over time.

kdebugui.h would probably be never included.

another possibility would be to leave the declaration in kdecore, and only put 
the implementation in kdeui.  
This is just dangerous if one link against QtGui and not kdeui.


> > - KConfigBase and KConfigSkeleton
> >      Allow to store QColor, QFont and probably more. This is probably the
> > more difficult to get ride of.  I have no idea what to do here.
>
> this is indeed more problematic; we've moved to mostly using QVariant in
> the branch which brings the usage of QColor down to two instances in
> KConfig itself one of which is easily resolvable and one which is trickier.

How ready is the branch ? 
When do you think it could be merged in trunk ?

> in the latter case, using QVariant::convert(Type) might help but it would
> bring several problems with it:
> 
>  - it would use a different storage format than the one in kde3 (hex #RGB
> as opposed to decimal r,g,b,a)
>  - it doesn't support a format that includes a value for alpha

For readability of config file, i prefer the #RGB format,  anyway, the fact it 
doesn't support alpha channel is very annoying.
can't trolltech fix that by implementing #ARGB ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20070103/229c2cfe/attachment.sig>