<br><br><div class="gmail_quote">2008/12/11 Arnd Baecker <span dir="ltr"><<a href="mailto:arnd.baecker@web.de">arnd.baecker@web.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="Wj3C7c">On Thu, 11 Dec 2008, Gilles Caulier wrote:<br>
<br>
> Hi all,<br>
><br>
> I have just tried to play with new QPainter effect from Qt4. Why not to add<br>
> a little 3D border around icon item like this :<br>
><br>
> <a href="http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-01.png" target="_blank">http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-01.png</a><br>

> <a href="http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-02.png" target="_blank">http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-02.png</a><br>

> <a href="http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-03.png" target="_blank">http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-03.png</a><br>

><br>
> Patch relevant is very small:<br>
><br>
> <a href="http://digikam3rdparty.free.fr/misc.tarballs/3dbordericonviewkde4.patch" target="_blank">http://digikam3rdparty.free.fr/misc.tarballs/3dbordericonviewkde4.patch</a><br>
><br>
> What do you you think about ?<br>
<br>
</div></div>To be honest: I don't like it at all, please make anything<br>
like this optional ....</blockquote><div><br>I don't want to add an option just for that... <br><br>I have fully rewrittent his patch now. Untouched thumbnail loader  and fixed against icon item implementation (like it must be done in the future by pure Qt4 Model/view code)<br>
<br>The frame is now very very discret (3 pixels no more, and code is the same than GWenview + some optimization.<br><br>Look the screenshot :<br><br><a href="http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-04.png">http://digikam3rdparty.free.fr/Screenshots/temp/3dbordericonkde4/3dbordericonviewkde4-04.png</a><br>
<br>Patch is at the same place:<br><br><a href="http://digikam3rdparty.free.fr/misc.tarballs/3dbordericonviewkde4.patch" target="_blank">http://digikam3rdparty.free.fr/misc.tarballs/3dbordericonviewkde4.patch</a><br><br>Of course revert previous patch before to compile with this one...<br>
<br>I hope this one will have a better audience (:=)))<br><br>Best<br><br>Gilles</div></div><br>