Menu
▾
▴
#500 DS: Change options dialog to use popup widget for zoom level
Would be nice to have radio buttons in the GUI. They would have to be grouped, so that when one is selected, the others are deselected.
The DS ports would benefit from that; it currently fakes radio buttons in its options dialog by using checkboxes and lots of code which "manually" deselects the other checkboxes in the same group (the groups being hard coded, naturally...). See also dsoptions.cpp, e.g. DSOptionsDialog::handleCommand() in there.
I can't see a clear advantadge of radio buttons over the popup widget. Both allow selection of 1 option from a predefined set, and the popup widget has the advantadge of requiring less screen space when not active (which I think would be a big plus for the DS' small resolution).
You are probably right. Neil, what do you say?
Yeah, the code which fakes radio buttons is pretty dirty. If I remember correctly though, at the time many widgets didn't allow me to create them without defining their parameters in the theme definition. Or, at least, it looked that way. I didn'r understand all the theme stuff so I did it the way that seemed simple.
Radio buttons might be nice, or I could switch to a drop down list. I'm not bothered either way and obviously there's an advantage to keeping the number of widgets to a minimum.
I think switching to a popup / dropdown menu would be best then. This would be less confusing to the use than the current "checkboxes that behave like radio buttons", and it would simplify your code, too.
Thus, as I suppose, the arguments are valid, and we'd better stick to popup widgets?
Instead of closing this, I am turning this into a request to switch the NDS port away frm abusing checkboxes, to using a popup :). Specifically, instead of having three checkboxes for 100%, 150% and 200%, there should be a single popup.