<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <title></title>

</head>

<body>

Maks Orlovich wrote:<br>

<blockquote type="cite"

 cite="mid200209021150.24232.mo002j@mail.rochester.edu">

  <blockquote type="cite">

    <pre wrap="">First of all I have to ask why this somewhat "hacky" tweak is needed.

Couldn't it simply be fixed in the styles, liquid seems to manage it.

    </pre>

  </blockquote>

  <pre wrap=""><!---->

Because the style doesn't even draw labels, and for many other objects it  

can't check parenting anyway, plus doing the check every time is somewhat of 

an overhead. So the style has to handle this with event filters installed in 

::polish(), and it either has to walk up the widget tree upto the root for 

every widget that is created checking for toolbar, or do something like this. 

And performance matters, so the name check is there. The reason Liquid 

doesn't need to do this is that it, IIRC, simply uses the stiple to draw the 

background, and so Qt draws it for it. However, changing the background brush 

requires dozens of hacks to get the alignment right. 

  </pre>

  <blockquote type="cite">

    <pre wrap="">Lastly, after trying it it unfortunately doesn't seem to work. The

background is still not right.

    </pre>

  </blockquote>

  <pre wrap=""><!---->

It certainly works for KAddressBook -- could you perhaps compare what you're 

doing with it?

  </pre>

</blockquote>

Ok, I've looked around and it seems to work now. That just leaves the artifacts

of the problem with the inactive QPushButton popup and the wrongly painted

missing toolbar handle on toolbars which doesn't have the maintindow as the

parent.<br>

<br>

/Mauritz<br>

Underscore AB<br>

</body>

</html>