elm_hoversel_scrollable_set/get() are removed with the next patch, and scrollability becomes default behavior of hoversel. Thanks for your opinion.
https://phab.enlightenment.org/D3241 2015-10-26 17:55 GMT+09:00 Davide Andreoli <d...@gurumeditation.it>: > 2015-10-26 1:31 GMT+01:00 Cedric BAIL <cedric.b...@free.fr>: > > > On Sun, Oct 25, 2015 at 9:23 AM, Davide Andreoli <d...@gurumeditation.it > > > > wrote: > > > 2015-10-23 7:20 GMT+02:00 Conrad Um <con...@gmail.com>: > > >> I'm sorry for inconvenience in using scrollable hoversel. > > >> > > >> Davide metioned that scrollable hoversel should be expand as large as > it > > >> can, but becomes scrollable after its size exceeds hover parent size. > > >> > > >> > > >> To say the conclusion first, that function is already implemented. > > >> > > >> If "max_size" is large enough, hoversel will try expanding to the > > >> "max_size", and shrinks again to fit its parent size. > > >> > > >> I thought that developers would determine how many items will show at > > the > > >> same time and customize theme, but it could be a cumbersome for those > > who > > >> do not want customize theme. > > >> > > >> > > >> The next patch removes the limitation with pixels, and makes hoversel > > >> expand as large as it can, but shrinks to fit its hover parent. > > >> > > >> https://phab.enlightenment.org/D3223 > > > > > > > > > Cool, thanks, this differential fix my use case. > > > At this point I wonder if we need the scrollable_set/get API at all, > > > souldn't it be scrollable all the time (when it does not fit)? > > > Is there a use case where you want it to be not scrollable? > > > > Preparing to push it. Now I agree here with your concern about > > scrollable_get/set. Do we really need it at all for any reason ? If > > that is part of the API added in 1.16, I think we should remove it. > > > > Yes, scrollable_set/get is a 1.16 API, and I also think we should remove it > > > > > > >> regards, > > >> > > >> Jee-Yong Um > > >> > > >> > > >> 2015년 10월 23일 금요일, Davide Andreoli<d...@gurumeditation.it>님이 작성한 메시지: > > >> > > >> > Let me try to explain better the problematic with the hoversel > > scrollable > > >> > implementation. > > >> > I will use some egitu screenshots to show a real case usage. > > >> > > > >> > This is clearly a case where hoversel scrollable is needed: > > >> > http://imgbin.org/images/27048.jpg > > >> > as you can see the list of branches (on the right) is too big to fit > > >> > in the window, thus the hoversel need to be set as scrollable. > > >> > > > >> > Now I set the hoversel to be scrollable.. this is the result: > > >> > http://imgbin.org/images/27049.jpg > > >> > as you can see the hoversel is way to short (60px fixed in theme), > > >> > and totally unusable as the scrolling is way to fast and finding the > > >> > wanted item is pratically impossible. > > >> > > > >> > What we really want here is that the scrollable hoversel expand its > > size > > >> > to the maximun size available in the hover_parent obj (or the > maximum > > >> > size of its contents if fit). > > >> > > > >> > To conclude I think this will need to be fixed (or reverted) in time > > for > > >> > the release, as it will be a mess to fix in a later time. Also > > remember > > >> > that this change (as it is now) require all the elm themes to be > > updated, > > >> > an thus re-updated if we will fix later. > > >> > > > >> > > > >> > > > >> > 2015-10-21 17:24 GMT+02:00 Stefan Schmidt <ste...@osg.samsung.com > > >> > <javascript:;>>: > > >> > > > >> > > Hello Jee-Yong Um. > > >> > > > > >> > > On 07/10/15 20:22, Davide Andreoli wrote: > > >> > > > 2015-08-04 16:59 GMT+02:00 Jee-Yong Um <jc9...@samsung.com > > >> > <javascript:;>>: > > >> > > > > > >> > > >> cedric pushed a commit to branch master. > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > > > >> > > > >> > > > http://git.enlightenment.org/core/elementary.git/commit/?id=e8264becb8f6f36e3c62c5fc5e4a46bbdbb9cf1f > > >> > > >> > > >> > > >> commit e8264becb8f6f36e3c62c5fc5e4a46bbdbb9cf1f > > >> > > >> Author: Jee-Yong Um <jc9...@samsung.com <javascript:;>> > > >> > > >> Date: Tue Aug 4 15:51:36 2015 +0200 > > >> > > >> > > >> > > >> elm_hoversel: add elm_hoversel_scroll_enabled_set/get() > API > > >> > > >> > > >> > > >> Summary: > > >> > > >> Currently, hoversel shows all items at the same time, but > to > > >> > limit > > >> > > >> the size of contents and make it scrollable is neccesary > to > > >> some > > >> > > users. > > >> > > >> elm_hoversel_scroll_enabled_set/get() API makes the > > contents of > > >> > > >> hoversel > > >> > > >> scrollable or not. > > >> > > >> > > >> > > > Surly a must-have feature (I also need the hoversel to be > > scrollable > > >> in > > >> > > > egitu). > > >> > > > > > >> > > > But I really don't like the behaviour of this implementation: > the > > >> usage > > >> > > of > > >> > > > the max_size > > >> > > > in edc seems wrong to me. > > >> > > > Now if you want the hover to be scrollable you end up with a > > really > > >> > > > small popup: 60 fixed px from the theme... Is this the wanted > > >> > behaviour? > > >> > > > how can > > >> > > > a theme designer know how many pixel are needed? > > >> > > > > > >> > > > I think that a "correct" implementation should expand the popup > to > > >> the > > >> > > > maximum > > >> > > > available space and then use the scroller if it still do not > fit. > > >> > > > > > >> > > > The current implementation is unusable in my case: > > >> > > > I have an hover that can contain up to (for example) 200 items, > > and > > >> if > > >> > I > > >> > > > set it scrollable > > >> > > > it only show 2 items at a time...really unusable... > > >> > > > > > >> > > > > >> > > Can you please comment on what Davide brings up here about your > > commit? > > >> > > > > >> > > regards > > >> > > Stefan Schmidt > > >> > > > > >> > > > > >> > > > > >> > > > >> > > > ------------------------------------------------------------------------------ > > >> > > _______________________________________________ > > >> > > enlightenment-devel mailing list > > >> > > enlightenment-devel@lists.sourceforge.net <javascript:;> > > >> > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > >> > > > > >> > > > >> > > > >> > > > ------------------------------------------------------------------------------ > > >> > _______________________________________________ > > >> > enlightenment-devel mailing list > > >> > enlightenment-devel@lists.sourceforge.net <javascript:;> > > >> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > >> > > > >> > > >> > > > ------------------------------------------------------------------------------ > > >> _______________________________________________ > > >> enlightenment-devel mailing list > > >> enlightenment-devel@lists.sourceforge.net > > >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > >> > > > > > > ------------------------------------------------------------------------------ > > > _______________________________________________ > > > enlightenment-devel mailing list > > > enlightenment-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > -- > > Cedric BAIL > > > > > > > ------------------------------------------------------------------------------ > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > ------------------------------------------------------------------------------ > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > ------------------------------------------------------------------------------ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel