Jules, after updating an app immediately didnt passed tests, as we checked i tried to isolate the bug to the simplest form:
addAndMakeVisible (&someButton); someButton.setBounds(10, 7, 56, 56); someButton.addListener(this); someButton.setClickingTogglesState(true); someButton.setRadioGroupId(666, sendNotification); addAndMakeVisible (&otherButton); otherButton.setBounds(76, 7, 56, 56); otherButton.addListener(this); otherButton.setClickingTogglesState(true); otherButton.setRadioGroupId(666, sendNotification);
On Button Clicked:
if (activeButton == &someButton) { DBG("someButton clicked"); } else if (activeButton == &otherButton) { DBG("otherButton clicked"); }
And the dbg statement shows that the buttons got somehow inverted in no time, this was caused by commit:
[9d50560] Stopped buttons with auto-toggle sending duplicated events
Reverting commit modification makes app pass tests again:
void Button::internalClickCallback (const ModifierKeys& modifiers) { if (clickTogglesState) setToggleState (radioGroupId != 0 || ! lastToggleState, sendNotification); //else sendClickMessage (modifiers); }
This happens with Juce latest pull.