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.]] }