From: cnook <kim...@gm...> - 2011-03-22 10:36:31
|
Dear All, Hello :) Thanks for your response. I have attached "elm_diskselector" patch using API and example code is also included. Please review this again. Thanks always. Sincerely, Shinwoo Kim. 2011/3/22 Tom Hacohen <tom...@pa...> > You can load default from the theme and modify it with API... > > On Tue, 2011-03-22 at 13:47 +0900, Daniel Juyung Seo wrote: > > Hello. > > > > > then several different theme file have to be existed. In this case, API > > > would be more efficient. > > > > Hmm... you're right. It will be more flexible to use API for item count. > > If we use previously suggested idea, we need to prepare many item > > styles for several cases. > > And that doesn't look that flexible. > > > > And one more comment. > > You don't need to write separate sample application. > > Just add the sample to elementary_test. > > > > Thanks. > > Daniel Juyung Seo (SeoZ) > > > > On Mon, Mar 21, 2011 at 10:39 PM, cnook <kim...@gm...> wrote: > > > Dear All, > > > > > > Hi~ > > > Thanks for your response always! > > > > > > I have changed the "elm_diskselector" based on your comments. > > > The "displayed item count" is described by using theme file. > > > And example source - example.tar.gz - have attached. > > > (If you have an example of the example :), then please inform and send > me.) > > > > > > By the way, I have something wondering about this way. > > > If several "elm_diskselector" are added with different "displayed item > > > count" on one same parent, > > > then several different theme file have to be existed. In this case, API > > > would be more efficient. > > > > > > I would like to know more about your opinion. Thanks > > > > > > Sincerely, > > > Shinwoo Kim. > > > > > > > > > 2011/3/19 Daniel Juyung Seo <seo...@gm...> > > >> > > >> Check the _theme_hook(). > > >> :) > > >> > > >> Thanks. > > >> Daniel Juyung Seo (SeoZ) > > >> > > >> On Sat, Mar 19, 2011 at 1:53 PM, cnook <kim...@gm...> wrote: > > >> > Dear All, > > >> > > > >> > Hi~ > > >> > > > >> > Thanks for your response. > > >> > > > >> > I have tried to use "data.item" and edje_object_data_get() API. It > works > > >> > fine. :) > > >> > > > >> > But I met a problem. When I use a new style for elm_diskselector, > > >> > edje_object_data_get() cannot retrieve "data.item" of new style. > > >> > > > >> > Only original (default) "data.item" value is retrieved. > > >> > > > >> > Would you please let me know what I forget? > > >> > > > >> > Thanks. > > >> > > > >> > > > >> > Sincerely, > > >> > Shinwoo Kim. > > >> > > > >> > > > >> > 2011/3/18 Daniel Juyung Seo <seo...@gm...> > > >> >> > > >> >> +3 ! > > >> >> Using group data in theme looks better because it reduces > source-gui > > >> >> dependences. > > >> >> > > >> >> group { name: "xx"; > > >> >> data.item: "count" "3"; > > >> >> > > >> >> You can fetch this data from c source using edje_object_data_get() > API. > > >> >> Please check other widgets for a reference. > > >> >> > > >> >> I checked the patch very briefly and here are some comments. > > >> >> > > >> >> 1. Indentation. > > >> >> ex) line 250 in elm_diskselector.diff > > >> >> > > >> >> 2. Blank lines. > > >> >> There are 2 blank lines at the end of diff file. 396, 397 lines. > > >> >> > > >> >> 3. Diff file. > > >> >> I think there is no rule for this but you can merge two diff > files > > >> >> to one diff file > > >> >> because they are patches for a one feature and one > > >> >> library(elementary). > > >> >> > > >> >> 4. Sample code. > > >> >> It will be better to have a sample code for a new feature in > > >> >> elementary_test. > > >> >> > > >> >> Other than that, looks ok :) > > >> >> > > >> >> Thanks. > > >> >> Daniel Juyung Seo (SeoZ) > > >> >> > > >> >> On Fri, Mar 18, 2011 at 5:57 AM, Tiago Falcao > > >> >> <dev...@ti...> > > >> >> wrote: > > >> >> > +2 ! > > >> >> > > > >> >> > When had see this widget in first time, I imagined it with option > to > > >> >> > many items but ever configured in theme. > > >> >> > If i'm right, this widget is a lot dependent of code and less of > > >> >> > theme > > >> >> > :( > > >> >> > > > >> >> > What you thing about this, Shinwoo Kim? > > >> >> > Gustavo suggested the easiest way, use group data. > > >> >> > > > >> >> > Thanks. > > >> >> > > > >> >> > > > >> >> > On Thu, Mar 17, 2011 at 4:47 PM, Gustavo Sverzut Barbieri > > >> >> > <bar...@pr...> wrote: > > >> >> >> On Thu, Mar 17, 2011 at 7:42 PM, Bruno Dilly < > bd...@pr...> > > >> >> >> wrote: > > >> >> >>> On Thu, Mar 17, 2011 at 10:45 AM, cnook <kim...@gm...> > wrote: > > >> >> >>>> Dear All, > > >> >> >>> > > >> >> >>> Hi Shinwoo Kim, > > >> >> >>> > > >> >> >>>> > > >> >> >>>> This is Shinwoo Kim, learned that I could contribute to EFL! > :) > > >> >> >>>> I'm pleased to inform you that the patch for the > > >> >> >>>> "elm_diskselector". > > >> >> >>>> > > >> >> >>>> Until now, the "elm_diskselector" only display 3 items at > once, > > >> >> >>>> if you accept this patch, the "elm_diskselector" can display > more > > >> >> >>>> than 3 > > >> >> >>>> items. > > >> >> >>> > > >> >> >>> Displaying more than 3 items is a nice improvement. > > >> >> >>> What do you think about the idea of getting the number of items > to > > >> >> >>> be > > >> >> >>> displayed from the theme ? > > >> >> >>> > > >> >> >>> Anyway, there is a typo on documentation (param num). Maybe a > > >> >> >>> getter > > >> >> >>> could be useful as well. > > >> >> >> > > >> >> >> +1 to get it from theme! > > >> >> >> > > >> >> >> I did something similar for ephoto in some older version, it had > > >> >> >> couple of swallow parts defined, like "elm.swallow.p%d", and a > > >> >> >> data.item: "count" "3", thus it would fill 3 swallows. > > >> >> >> > > >> >> >> I believe this is something up to the theme as it has the > knowledge > > >> >> >> on > > >> >> >> how to pack more items. > > >> >> >> > > >> >> >> > > >> >> >> -- > > >> >> >> Gustavo Sverzut Barbieri > > >> >> >> http://profusion.mobi embedded systems > > >> >> >> -------------------------------------- > > >> >> >> MSN: bar...@gm... > > >> >> >> Skype: gsbarbieri > > >> >> >> Mobile: +55 (19) 9225-2202 > > >> >> >> > > >> >> >> > > >> >> >> > > >> >> >> > ------------------------------------------------------------------------------ > > >> >> >> Colocation vs. Managed Hosting > > >> >> >> A question and answer guide to determining the best fit > > >> >> >> for your organization - today and in the future. > > >> >> >> http://p.sf.net/sfu/internap-sfd2d > > >> >> >> _______________________________________________ > > >> >> >> enlightenment-devel mailing list > > >> >> >> enl...@li... > > >> >> >> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > >> >> >> > > >> >> > > > >> >> > > > >> >> > > > >> >> > -- > > >> >> > Tiago Rezende Campos Falcão > > >> >> > http://www.tiagofalcao.com > > >> >> > -- > > >> >> > ProFUSION | embedded systems > > >> >> > Computer Systems Laboratory - IC - Unicamp > > >> >> > Grupo Pró Software Livre - Unicamp > > >> >> > Laboratory of Information Systems - IC - Unicamp > > >> >> > > > >> >> > > > >> >> > > > >> >> > > ------------------------------------------------------------------------------ > > >> >> > Colocation vs. Managed Hosting > > >> >> > A question and answer guide to determining the best fit > > >> >> > for your organization - today and in the future. > > >> >> > http://p.sf.net/sfu/internap-sfd2d > > >> >> > _______________________________________________ > > >> >> > enlightenment-devel mailing list > > >> >> > enl...@li... > > >> >> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > >> >> > > > >> > > > >> > > > > > > > > > > > > ------------------------------------------------------------------------------ > > Enable your software for Intel(R) Active Management Technology to meet > the > > growing manageability and security demands of your customers. Businesses > > are taking advantage of Intel(R) vPro (TM) technology - will your > software > > be a part of the solution? Download the Intel(R) Manageability Checker > > today! http://p.sf.net/sfu/intel-dev2devmar > > _______________________________________________ > > enlightenment-devel mailing list > > enl...@li... > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > |