Unfortunately it's not obvious what _toggle( value ) really does. Like "toggle but don't toggle"? I suggest setEnabled( enabled ). I got used to this in Java. I'm not sure if this is wanted here so feel free to ignore this suggestion.