Page MenuHomePhabricator

enum Efl.Ui.Scrollbar_Direction
Closed, ResolvedPublic


enum @beta Efl.Ui.Scrollbar_Direction
   horizontal = 0,
segfaultxavi triaged this task as TODO priority.

Let's take a look at this together.

Currently, I cannot find any problem in this definition.

Shouldn't all users of this class use Efl.Ui.Layout_Orientation instead?
For example, Efl.Ui.Scrollbar should implement the Efl.Ui.Layout_Orientable interface instead of defining its own enum with the same values.


Could you check the idea of @segfaultxavi ?

If there would be no matter with axit_bitmask and inverted in Efl.Ui.Layout_Orientation -
I also think it's good idea.

Scrollbars can just ignore the inverted flag. We will document it. I think users will understand :)

If Efl.Ui.Scrollbar implements Efl.Ui.Layout_Orientable, it makes conflict on efl_ui_image_zoomable.eo.
Because efl_ui_image_zoomable.eo implements Efl.Gfx.Image_Orientable also, there are two orientation method with same name.
Do you have some ideas of avoiding conflict?

eagleeye moved this task from Backlog to Evaluating on the efl: api board.Jun 26 2019, 11:08 PM

Hmmmm... you are correct. We had not found any such case before.

I guess the only solution is to rename the conflicting properties to image_orientation and layout_orientation.
The C method names should not change (because of the repeated word) and the C# names will be clearer (object.ImageOrientation).

Do you want to provide a patch, @eagleeye?

This sounds a bit confusing to me, shouldn't this be Scrollbar_Position ? Its not really the orientation ...

This selectes whether the scrollbar appears at the bottom or at the side of the scroller, but also whether the scrollbar is horizontal or vertical, no? I think it is very related to orientation.
Now that D9299 has landed the idea from T7924#137454 (above) can be implemented, I think.

Sorry, I don't understand what you mean :/

Okay - i misunderstood the comment in Sorry. Yes that would be possible.

zmike closed this task as Resolved.Aug 29 2019, 7:47 AM
zmike claimed this task.

Alright this is gone