IDEA: nullable button radio groups

[David Healey]

@dannytaurus Yeah it makes sense, rather than having to apply it to individual buttons. I don't know if nullable is the most intuitive word for the behaviour.

[dannytaurus]

@David-Healey Good point. Any suggestions?

[David Healey]

@dannytaurus toggleable... lol don't use that. I can't think of anything better at the moment. Maybe someone else will chime in a with an idea.

[ustk]

@dannytaurus That's usually made on the scripting level so having this at a C++ level seems a good idea

setExclusive
allowDeselect (my preferred one)
setDeselectable
setExclusiveZeroAllowed

[David Healey]

@ustk said in IDEA: nullable button radio groups:

allowDeselect

Yeah this seems good

[dannytaurus]

@ustk @David-Healey Yep, allowDeselect is good, but not for the whole function name surely?

Content.allowDeselect() ?

I should refer to radio group, radio button, or similar, right?

[David Healey]

@dannytaurus setAllowRadioGroupDeselect()?

[ustk]

@David-Healey said in IDEA: nullable button radio groups:

@dannytaurus setAllowRadioGroupDeselect()?

yep much more better

[dannytaurus]

@ustk said in IDEA: nullable button radio groups:

@David-Healey said in IDEA: nullable button radio groups:

@dannytaurus setAllowRadioGroupDeselect()?

yep much more better

[dannytaurus]

@ustk @David-Healey Actually, maybe the 'Allow' is redundant, since we call it with true or false.

Maybe just setRadioGroupDeselect, like this:

Content.setRadioGroupDeselect(1, true); // turn it on
Content.setRadioGroupDeselect(1, false); // turn it off

Thoughts?

[David Healey]

@dannytaurus I think the allow is good because we are saying "allow buttons in this group to be deselected"

Whereas without it we are saying "set buttons in this group to be deselected" which has a different meaning.

Also there are other similarly named functions

[dannytaurus]

@David-Healey Nice. I'll update the name now

[dannytaurus]

@David-Healey @ustk Function name changed in PR.