* daniel.za...@samsung.com <daniel.za...@samsung.com> [2013-05-23 13:45:38 +0300]:
> Hi Gustavo, Hey :) > > No problem, I will fix that. I just have some points to confirm with you: > - From now and only for functions relative to the interface, elm_scroller > will eo_do elm_scrollable_interface... instead of elm_obj_scroller... Yeppers. > - It means that I will remove all the op_ids from scroller that will be > added in the interface. Indeed. > - For the moment, the scrollable interface API is not public, i.e a > developer cannot use it in its application. As we want in some next-century > day remove the non Eo APIs, we have a clash here and so we may need to make > them public. Am I right? The use I left possible, at my time, was the possibility of a user also inheriting widgets and making them scrollable. He/she could not change the interface internals (nor extend it), though, just use it as is, like our widgets do. By keeping that as a minimum, I'm fine :) Thanks for fixing that. The patch you sent my privately looks ok, except for enlightenment/efl/src/lib/eo/eo.c:838 _eo_class_check_op_descs() Wrong order in Ops description for class 'elm_scroller'. Expected 0x1 and got 0x4 (and maybe other errors). Can you fix that? Thanks. > > JackDanielZ (alias Daniel) > > On 05/22/2013 06:43 PM, Gustavo Lima Chaves wrote: > >This is addressed to Elementary coders, especially Tom and (Jack)Daniel. > > > >When I released the new, widget classes enabled version of Elementary, > >I introduced the scrollable widgets, as seen on picture > >http://docs.enlightenment.org/auto/elementary/elm-widget-tree.png. All > >those would respond to all elm_scroller_ namespaced EAPI functions, > >like explicitly said at that widget's documentation -- > > > >"Its (non-deprecated) API functions, except for elm_scroller_add(), > >which gives basic scroller objects, are meant to be a basis for all > >other scrollable widgets (i.e. widgets implementing The Elementary > >Scrollable Interface). So, they will work both on pristine scroller > >widgets and on other "specialized" scrollable widgets." > > > >What happened after people stepped in to translate the Evas Smart > >Class based inheritance schema to eo, is that some of those > >elm_scroller_ functions meant to work on all scrollables DO NOT > >ANYMORE (they work only for the scroller, itself). That is an ABI > >break, which can't go on released. > > > >Daniel, could you fix that for us, please? All elm_scroller.c EAPI > >functions which had s_iface-> calls on old elm and now have eo_do() > >for elm_obj_scroller_ namespaced things are WRONG, like: > > > >- elm_scroller_region_get > >- elm_scroller_child_size_get > >- elm_scroller_page_relative_set (_get is OK) > > > >-- > >Gustavo Lima Chaves > >Senior Developer @ ProFUSION Embedded Systems > > > >------------------------------------------------------------------------------ > >Try New Relic Now & We'll Send You this Cool Shirt > >New Relic is the only SaaS-based application performance monitoring service > >that delivers powerful full stack analytics. Optimize and monitor your > >browser, app, & servers with just a few lines of code. Try New Relic > >and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may > >_______________________________________________ > >enlightenment-devel mailing list > >enlightenment-devel@lists.sourceforge.net > >https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > -- Gustavo Lima Chaves Senior Developer @ Intel - Open Source Technology Center ------------------------------------------------------------------------------ Try New Relic Now & We'll Send You this Cool Shirt New Relic is the only SaaS-based application performance monitoring service that delivers powerful full stack analytics. Optimize and monitor your browser, app, & servers with just a few lines of code. Try New Relic and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
