No comments? I guess we already discussed this heavily. If there is no comments in a couple of days, I'll commit this.
Thanks. Daniel Juyung Seo (SeoZ) On Thu, Mar 31, 2011 at 12:22 PM, cnook <[email protected]> wrote: > Hi All, > > Please check latest patch set. > > Sincerely, > Shinwoo Kim. > > > 2011/3/28 cnook <[email protected]> > >> Hi All, >> >> Thanks for your response and suggestion always. >> I have attached new patch file which has following modification. >> >> - remove warning message >> - support setting by theme and api >> >> Others are also reasonable suggestion, some of them especially vertical >> mode would be fine. But till now there is no design for that as raster >> mentioned. setting min/max "display item num" is reasonable also but user >> can set content size differently. So if elm_diskselector restrict its >> min/max "display item num", it would occur that user cannot set "display >> item num" more than max value even though there is enough space. >> >> Thanks again. >> >> Sincerely, >> Shinwoo Kim. >> >> >> 2011/3/25 Carsten Haitzler <[email protected]> >> >>> On Wed, 23 Mar 2011 08:19:13 -0700 Daniel Juyung Seo < >>> [email protected]> >>> said: >>> >>> you're going to hate me :) >>> >>> you did go the right way - theme defines default, code can override... >>> BUT.. >>> you are ALSO missing a fit POLICY. you are fitting just N items in the >>> visible >>> region. i think... you are missing the ability to say "fit as many items >>> ads >>> you can" either based on largest item in list (homogeneous layout in the >>> box) >>> or have all items a different size (actually i might argue that this is >>> dubiously useful except maybe in a case where MOST items are small and 1 >>> or 2 >>> are longer eg a list like: 1,2,3,4,5,NONE). >>> >>> so THEME should provide the default FIT policy (fixed count, best fit, >>> compact >>> fit). fixed count uses the theme number defined as you did already and >>> code can >>> override that, and other policies as above. also code should be allowed >>> to >>> override this too. >>> >>> (and yes a horizontal scrolling list of items where they change size >>> horizontally is not that great as gustavo mentioned... but i know that >>> you >>> didn't design the widget, so not a lot of use going on about that - BUT >>> his >>> point implies that we should have a vertical diskselector mode too... use >>> that >>> for horizontally expanding items, and vice-versa). >>> >>> > I attached a screen shot :) >>> > This is a screen shot from elementary_test "Disk Selector" which cnook >>> > attached. >>> > >>> > Btw, cnook, I have one comment. >>> > _item_click_cb()'s parameter is wrong. >>> > Please check elm_diskselector.c code. >>> > >>> > elm_diskselector.c: In function ‘_item_new’: >>> > elm_diskselector.c:137: warning: passing argument 4 of >>> > ‘edje_object_signal_callback_add’ f >>> > rom incompatible pointer type >>> > /usr/local/include/edje-1/Edje.h:550: note: expected ‘Edje_Signal_Cb’ >>> > but argument is of t >>> > ype ‘void (*)(void *, struct Evas_Object *, void *)’ >>> > >>> > Thanks. >>> > Daniel Juyung Seo (SeoZ) >>> > >>> > >>> > >>> > >>> > On Wed, Mar 23, 2011 at 7:15 AM, Gustavo Sverzut Barbieri >>> > <[email protected]> wrote: >>> > > On Wed, Mar 23, 2011 at 1:54 PM, cnook <[email protected]> wrote: >>> > >> Dear All, >>> > >> >>> > >> Thanks for your response. >>> > >> >>> > >> I have attached "elm_diskselector" patch using API and getting >>> default >>> > >> value from theme. >>> > >> But the default value from "elementary/themes/default.edc" is >>> applied only. >>> > >> If user wants new default value from his/her own theme file, it will >>> not be >>> > >> applied. >>> > >> >>> > >> >>> > >> Why I have changed like this.. because there is one case I worry >>> about >>> > > >>> > > >>> > > Do you have the screenshot? Nobody replied to my mail. >>> > > >>> > > >>> > > -- >>> > > Gustavo Sverzut Barbieri >>> > > http://profusion.mobi embedded systems >>> > > -------------------------------------- >>> > > MSN: [email protected] >>> > > Skype: gsbarbieri >>> > > Mobile: +55 (19) 9225-2202 >>> > > >>> > > >>> ------------------------------------------------------------------------------ >>> > > 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 >>> > > [email protected] >>> > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >>> > > >>> >>> >>> -- >>> ------------- Codito, ergo sum - "I code, therefore I am" -------------- >>> The Rasterman (Carsten Haitzler) [email protected] >>> >>> >> > ------------------------------------------------------------------------------ Create and publish websites with WebMatrix Use the most popular FREE web apps or write code yourself; WebMatrix provides all the features you need to develop and publish your website. http://p.sf.net/sfu/ms-webmatrix-sf _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
