After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 679465 - Add :z-position and deprecate :depth
Add :z-position and deprecate :depth
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: ClutterActor
unspecified
Other All
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2012-07-05 18:31 UTC by Emmanuele Bassi (:ebassi)
Modified: 2012-07-11 20:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
actor: Add :z-position and deprecate :depth (15.35 KB, patch)
2012-07-05 18:31 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review

Description Emmanuele Bassi (:ebassi) 2012-07-05 18:31:10 UTC
The ClutterActor:depth property has always been a bit of a misnomer:
actors are 2D flat surfaces, so they cannot have "depth"; the property
defines the position on the Z axis.

Another side effect of the :depth property is that it decides the
default paint and allocation order on insertion, and that setting it
will call the ClutterContainer.sort_depth_order() method. This has
proven to be a fairly bad design decision that we strung along from the
0.x days, as it gives a false impression of being able to change the
paint and allocation order simply by changing the position on the Z
axis — something that, in reality, requires depth testing to be enabled
during the paint sequence of an actor's parent.

For 2.0 we need a clean break from the side effects, and a better
defined interface.

ClutterActor:z-position is essentially what ClutterActor:depth is, but
doesn't call into ClutterContainer, and has a more apt name.
Comment 1 Emmanuele Bassi (:ebassi) 2012-07-05 18:31:12 UTC
Created attachment 218117 [details] [review]
actor: Add :z-position and deprecate :depth
Comment 2 Emmanuele Bassi (:ebassi) 2012-07-11 20:16:18 UTC
pushed to master