[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-bugs-dist
Subject: [Bug 187681] shape shadows are not handled correctly
From: Jan Hambrecht <jaham () gmx ! net>
Date: 2009-03-22 14:19:10
Message-ID: 20090322141910.C05F1158F2 () immanuel ! kde ! org
[Download RAW message or body]
https://bugs.kde.org/show_bug.cgi?id=187681
Jan Hambrecht <jaham@gmx.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jaham@gmx.net
--- Comment #4 from Jan Hambrecht <jaham gmx net> 2009-03-22 15:19:08 ---
> Shadows are done incorrect.
> * A shape that is transparent but has a border gets its border drawn inside the
> object. (it should be clipped to outside the shape) The text shape shows this.
Why should a transparent shape cover the shadow? After all it is transparent,
which means one can see through.
> * You get overlapping shapes painting their shadows on top of each other. I
> expect the border to be below all shapes. See the green shape being covered by
> a shadow. It should not be.
I think it is correct if they have different z-indeces. If their z-index is the
same they should not cast a shadow on each other. The latter probably has to be
fixed.
> * A shape with content that has an alpha channel becomes unreadable because the
> shadow shows through. See the yellow shape.
Again, why should transparency prevent to see the shadow behind the shape?
>
> See the textshape in the screenshot;
> * A shape that has a transparent fill doesn't have a shadow. (it should)
In the case of the text shape, no it does not have a transparent fill, it has
no fill. If you set a fill, the shadow will be shown.
> * A shape without a border gets a one-line border drawn anyway. It shouldn't.
This was also fixed by my last commit.
--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic