Page MenuHomePhabricator
Create Task
Maniphest T7923

enum Efl.Ui.Scrollbar_Mode
Closed, ResolvedPublic
Actions

Description

enum @beta Efl.Ui.Scrollbar_Mode
{
   auto = 0, [[Visible if necessary]]
   on, [[Always visible]]
   off, [[Always invisible]]
   last [[For internal use only]]
}

Details

Commits
D10008 / rEFLda594413bef4: declare first round of classes stable

Related Objects
Search...

StatusAssignedTask
 Resolvedbu5hm4nT8096 efl.ui.collection
 ResolvedNoneT7870 efl.ui.grid
 ResolvedNoneT7881 efl.ui.list
 ResolvedcedricT7885 efl.ui.list_view
 ResolvedNoneT7891 efl.ui.scroller
 ResolvedzmikeT7884 efl.ui.scrollbar
 ResolvedNoneT7923 enum Efl.Ui.Scrollbar_Mode
segfaultxavi created this task.May 8 2019, 2:32 AM
segfaultxavi triaged this task as TODO priority.
woohyun added a subscriber: eagleeye.Jun 18 2019, 4:43 AM

@eagleeye
Let's take a look at this together.

Currently, I cannot find any problem in this definition.

segfaultxavi added a comment.Jul 8 2019, 3:09 AM

I would very much like that this enum disappears and we use Efl.Ui.Layout_Orientation instead. We do not need to use the inverted part of the enum.

eagleeye added a comment.Jul 11 2019, 11:57 PM

I think we can't use Efl.Ui.Layout_Orientation in this case. Because this enum is related to mode of scrollbar not direction.

segfaultxavi added a comment.Jul 12 2019, 1:27 AM

Yeah, you are right. My bad. This enum is still needed.

woohyun added a comment.Aug 14 2019, 4:25 AM

I think this looks ok - because legacy scroller has proved that this enum is needed with this definition.

Does anyone have another idea on this ?

@eagleeye
If there would be any expected change on this, please share your idea.

zmike added a comment.Aug 29 2019, 7:48 AM

This seems fine to me

zmike moved this task from Backlog to Trivial on the efl: api board.Aug 29 2019, 7:48 AM
zmike moved this task from Trivial to Stabilized on the efl: api board.Sep 13 2019, 12:15 PM