<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-15"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
On 01/30/2010 05:49 PM, Pino Toscano wrote:
<blockquote cite="mid:201001310149.43970.pino@kde.org" type="cite">
<pre wrap="">Alle domenica 31 gennaio 2010, Hugo Pereira Da Costa ha scritto:
</pre>
<blockquote type="cite">
<pre wrap="">To me, the plain background you get when calling autofillbackground
(using the palette.color( widget->backgroundrole() ), is a feature, not
a bug.
</pre>
</blockquote>
<pre wrap="">
See, color() != brush(). This Qt behaviour is basically bugging every widget
which sets the autoFill and has the background role in the palette with
something else than a color.
</pre>
</blockquote>
ok, I agree with this.<br>
<br>
But that wont fix the issue with oxygen. its main window background is
not a brush, passed via the palette. <br>
Maybe it could be made so (though I'm not sure about clipping), but it
is not at the moment. Right now its plain painting (with QPainter) in
re-implemented QEvent::Paint for appropriate (top level) widgets. <br>
<br>
<blockquote cite="mid:201001310149.43970.pino@kde.org" type="cite">
<pre wrap=""></pre>
<blockquote type="cite">
<pre wrap="">if okular wants to leave it this way:
- no I wont consider it as an oxygen bug and wont fix any bug report on
that (notably, not to break gwenview)
</pre>
</blockquote>
<pre wrap="">
As it looks to me, it is a Qt bug (forcing the use of a color from the palette
instead of using the brush).
</pre>
<blockquote type="cite">
<pre wrap="">- will live with that, and patch my local okular.
I was just suggesting.
</pre>
</blockquote>
<pre wrap="">
Me as well.
... but well, why I am caring about it at all, given I'm not the Okular
maintainer?!?! shame.
</pre>
</blockquote>
me neither ! I was just thinking that if a 4 lines patch (when properly
indented) can fix the appearance of okular with oxygen, and bespin (the
only two decos which use background gradients AFAK), without breaking
anything else, then its worth the change. <br>
I usually have nothing against fixing bugs in oxygen style (make a svn
log on kstyles/oxygen/oxygen.cpp :-))<br>
And most of all, I really really don't want to enter in any kind of
conflict with valuable devs. <br>
<br>
<blockquote cite="mid:201001310149.43970.pino@kde.org" type="cite">
<pre wrap="">Feel free to commit it, if anybody else wants it. (In that case, please at
least fix the indentation, thanks.)
</pre>
<pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
Okular-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Okular-devel@kde.org">Okular-devel@kde.org</a>
<a class="moz-txt-link-freetext" href="https://mail.kde.org/mailman/listinfo/okular-devel">https://mail.kde.org/mailman/listinfo/okular-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>