[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice
Subject:    [Bug 187681] shape shadows are not handled correctly
From:       Thomas Zander <zander () kde ! org>
Date:       2009-03-24 22:47:37
Message-ID: 20090324224737.DAC2415C6A () immanuel ! kde ! org
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=187681





--- Comment #5 from Thomas Zander <zander kde org>  2009-03-24 23:47:36 ---
On Sunday 22. March 2009 15.19.10 Jan Hambrecht wrote:
> > * 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.

I think it depends on how the user views the z-indeces on the objects.
The itemviews people got lots of bugreports when they took this approach and 
basically I think it depends on what your goal is.
If your goal is to illustrate the fact that one shape has a z-index higher than
another, then yes.
But we do our best to hide this fact and actually our code assumes that the 
user is not affected by z-indexes much.

Imagine us inserting 2 objects. One after the other.  We place them next to 
each other.  The user never choose which one is over the other and frankly 
doesn't care.  But in our code we force one to be above the other because we
auto-assign them indices.
If we then end up casting a shadow of one object on the other that is next to 
it, then that breaks the illusion of them being next to each other.
In effect we are showing an implementation detail.

Maybe we should have a shadow per layer, but I think that its wrong to have one
per shape that cast a shadow on another shape.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
____________________________________
koffice mailing list
koffice@kde.org
To unsubscribe please visit:
https://mail.kde.org/mailman/listinfo/koffice
[prev in list] [next in list] [prev in thread] [next in thread]