Sorry for the wrong documentation. and I've attached patch file again that includes removed documentation in header file. I'll modify documentation more and send it later.
Guidetext is to set the text literally.. If user set a guide text through elm_multibuttonentry_guide_text_set(), Guidetext shows up when Multibuttonentry which is not added button is unfocused. instead of showing entry to provide guide message. such as, if I set the guidetext as " Tap to add recipient" in Multibuttonentry, User could recognize how do I act to insert button in Multibuttonentry when it's unfocused. But, This function is optional, If user don't want, doesn't need to be necessarily. Please review it's APIs and fouctionalities again. I'd appreciate any comments on this. Thank you, Goun ------- Original Message ------- Sender : Iv?n Briano (Sachiel)<[email protected]> Date : Nov 23, 2011 21:29 (GMT+09:00) Title : Re: [E-devel] [Patch] Add new elementary widget 'Multibuttonentry' 2011/11/23 GoUn Lee : > > Dear Developers, > > I added a new widget called “Multibuttonentry" which could be used as contact > button in App > This widget allows a user to insert a text button. > text button is inserted by pressing the "Enter key" > If there is no space in the current row, the new button is entered in the > next row. If the button is clicked, it will become focused. > And focused button can be removed by pressing the "backspace" key. > when items are added over 1 lines, if Multibuttonentry lost focus, it becase > shrink mode (made it as 1 line) > > Please review it's APIs and functionalities. > I didn't review the code and I don't have time for that now, but a quick look at the patch already tells me that documentation is poor. + /** + * Set the guide text + * + * @param obj The multibuttonentry object + * @param label The guide text string + * + * @ingroup Multibuttonentry + */ + EAPI void elm_multibuttonentry_guide_text_set(Evas_Object *obj, const char *guidetext) EINA_ARG_NONNULL(1); I can see in the function name that this function sets the guide text. But what is the guide text? What does it do? Why do I want to set it? For new widgets, having clear documentation also helps with the review. It makes it clear what the intention of each function, and the whole widget, is, so that we can read the code and match it to that intention, finding possible problems in things that without knowing any better, would look just fine. > > Thank you, > Goun > ------------------------------------------------------------------------------ > All the data continuously generated in your IT infrastructure > contains a definitive record of customers, application performance, > security threats, fraudulent activity, and more. Splunk takes this > data and makes sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-novd2d > _______________________________________________ > enlightenment-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-novd2d _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
elc_multibuttonentry.patch.txt
Description: Binary data
------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-novd2d
_______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
