On Thu, 2006-03-30 at 16:23 +0200, Juergen Spitzmueller wrote: > I think a good starting point, especially if your new on this area, is to use > an existing dialog/controller, copy and adapt that. I think vspace could be a > prototype in this case.
A couple of issues which it would be nice to rectify in new controller code: * Please supply all strings (translated) for the pulldown lists (by the way, in GTK we call them comboboxes, I think Qt people do too). Having these duplicated in the frontends is a bad thing, and also leads to "magic numbers". * Don't make me do a switch{} on which row is selected in a combobox, and pass a certain value for certain values. Just let me pass an integer, given that I've kept the options in the same order that they were supplied to me. Equally, when I'm doing update(), just give me an integer, don't make me do a switch{} there too. * Please don't depend on the length being zero or not passed - just ignore it if "Custom" is not chosen, and please supply a well defined length even if "Custom" is not chosen. I would like to always read out of the length entry on apply(), and always write into it on update(), without having to do any conditionals about it. John