enum @beta Efl.Orient
{
[[An orientation type, to rotate visual objects.
Not to be confused with @Efl.Ui.Dir which is meant for widgets, rather
than images and canvases. This enum is used to rotate images, videos and
the like.
See also @Efl.Orientation.
]]
none = 0, [[Default, same as up]]
up = 0, [[Orient up, do not rotate.]]
right = 90, [[Orient right, rotate 90 degrees counter clock-wise.]]
down = 180, [[Orient down, rotate 180 degrees.]]
left = 270, [[Orient left, rotate 90 degrees clock-wise.]]
}Description
Description
Details
Details
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | None | T7866 efl.ui.nstate | ||
| Resolved | None | T7846 efl.ui.button | ||
| Resolved | zmike | T7951 Efl.Ui.Text_Alert_Popup | ||
| Resolved | zmike | T7952 Efl.Ui.Scroll_Alert_Popup | ||
| Resolved | zmike | T7948 Efl.Ui.Alert_Popup | ||
| Resolved | None | T7873 efl.ui.image | ||
| Resolved | None | T7878 efl.orientation | ||
| Resolved | bu5hm4n | T8096 efl.ui.collection | ||
| Resolved | None | T7870 efl.ui.grid | ||
| Resolved | None | T7881 efl.ui.list | ||
| Resolved | None | T8097 efl_ui_spin_button | ||
| Resolved | None | T7897 efl.ui.spin | ||
| Open | None | T7510 evaluate stabilization potential of efl.ui classes and dependencies | ||
| Open | None | T7849 efl.ui.text | ||
| Open | CHAN | T7868 efl.ui.clock | ||
| Resolved | zmike | T7947 Efl.Ui.Anchor_Popup | ||
| Resolved | None | T7902 efl.ui.popup | ||
| Resolved | CHAN | T7869 efl.ui.datepicker | ||
| Resolved | None | T7880 efl.ui.bg | ||
| Resolved | cedric | T7885 efl.ui.list_view | ||
| Resolved | None | T7891 efl.ui.scroller | ||
| Resolved | None | T7893 efl.ui.slider | ||
| Resolved | CHAN | T7901 efl.ui.timepicker | ||
| Resolved | None | T7905 efl.ui.item | ||
| Resolved | None | T7919 enum Efl.Orient |
Comment Actions
Besides the comment in T7878 regarding the possible merging of this class with Efl.Flip, I am not sure {up, down, left, right} are sensible names for a rotation. What does it mean that an icon is oriented "to the right"?
How about we rename the enum ROTATION_0, ROTATION_90, etc ?
Comment Actions
After a lengthy brainstorming session:
- Rename this enum to Efl.Gfx.Image_Orientation
- Turn it into a bitfield and include flipping values in both axes
- Use it in all places where Efl.Orientation was used except in Efl.Ui.Layout_Base.theme_rotation_apply, that should be turned into an int.
- Delete Efl.Orientation and Efl.Flip.
Comment Actions
After that commit, it now looks like this and I think it's good to go:
enum @beta Efl.Gfx.Orientation
{
[[An orientation type, to rotate and flip images.
This is similar to EXIF's orientation.
Directional values ($up, $down, $left, $right) indicate the final direction in
which the top of the image will be facing (e.g. a picture of a house will have
its roof pointing to the right if the $right orientation is used).
Flipping values ($flip_horizontal and $flip_vertical) can be additionaly added
to produce a mirroring in each axis.
Not to be confused with @Efl.Ui.Dir which is meant for widgets, rather
than images and canvases. This enum is used to rotate images, videos and
the like.
]]
none = 0, [[Default, same as up, do not rotate.]]
up = 0, [[Orient up, do not rotate.]]
right = 1, [[Orient right, rotate 90 degrees clock-wise.]]
down = 2, [[Orient down, rotate 180 degrees.]]
left = 3, [[Orient left, rotate 270 degrees clock-wise.]]
rotation_bitmask = 3, [[Bitmask that can be used to isolate rotation values, that is, $none, $up, $down, $left and $right.]]
flip_horizontal = 4, [[Mirror horizontally. Can be added to the other values.]]
flip_vertical = 8, [[Mirror vertically. Can be added to the other values.]]
flip_bitmask = 12 [[Bitmask that can be used to isolate flipping values, that is, $flip_vertical and $flip_horizontal.]]
}