<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2013/5/10 David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im">On Wednesday 08 May 2013 17:26:17 David Gil Oliva wrote:






</div>
<div class="im"><br>
> Therefore, I think that it would be best to add a static method called<br>
> getItemFromListView. What do you think?<br>
<br>
</div>Option 1: adding getItemFromListView (the name looks a bit odd)<br>
Option 2: adding a getItem overload that takes an InputDialogOptions<br>
parameter, so that this will be ready for more options if any, in the future.<br></blockquote><div><br></div><div>There's something odd about the second one. I could write a getItem overload with a InputDialogOptions parameter, but the value of that parameter could be:<br>
<br>    enum InputDialogOption {<br>        NoButtons                    = 0x00000001,<br>        UseListViewForComboBoxItems  = 0x00000002,<br>        UsePlainTextEditForTextInput = 0x00000003<br>    };<br><br>The third one is for the new getMultiLineText method.<br>
<br></div><div></div><div>Actually, only the second one has something to do with the other static-getItem-with-listView method. Therefore it turns out the aim is not to have a listView in getItem, but to be able to use any existing option, even when they don't make any sense for the getItem method. I find it weird and counter-intuitive.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Option 3: porting coded to the multi-line call (constructor, setComboBoxItems,<br>
setOptions).<br></blockquote><div><br></div><div>Sorry that I insist, but with so many messages I don't want my question to get buried down. I don't understand this third option and I'd need a little explanation...<br>
<br></div><div>Thank you so much<br><br></div><div>David Gil<br><a href="http://www.hackingastrology.net">www.hackingastrology.net</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
In my opinion option 2 is best if not too ugly (not sure where to add the<br>
additional argument, depends how many default values there are etc.),<br>
otherwise option 3 (depending on how many calls to getItem we actually<br>
have...).<br>
<div class=""><div class="h5"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE, in particular KDE Frameworks 5<br>
<br>
</div></div></blockquote></div><br></div></div>