[PATCH] fix KDialog to use platform native layout spacing and margins

[Christoph Feck]

Hello,

this is my first post to this list, so please be gentle with my mistakes :)

I noticed that KDialog uses its own hard coded sizes for spacing and margins, 
which I consider bad, because Qt styles can compute layouts depending on
widget types and pairs, so even a configurable value would be wrong.

This patch should "move" those fixed values to KStyle, meaning any style 
automatically inherits the KDE defaults (should they ever change).

Additionally, KDialog no longer sets these attributes on it's TopLayout, so 
that it can respect the style's layoutSpacingImplementation() method, which 
results in better platform native appearance e.g. on QMacStyle.

I already posted this to kde.bugs.org as bug #176473, since I was not 
subscribed to this list before, and am looking for someone who could review 
this patch. But I am adding it here, too, as I do not know if you prefer it 
here.

Thanks,
Christoph Feck (kdepepo on #kde-devel)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: layout.diff
Type: text/x-diff
Size: 5409 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20081129/3be8a9ce/attachment.diff>