Well, the reason for the change was this:
https://forum.juce.com/t/redundant-buttonclicked-calls-due-to-button-callbackhelper-valuechanged/
If you connected a button to a Value, it would trigger a click event (and show the button being clicked) when something other than a click changed the Value, which doesn’t make sense.
But yes, good point, disabling all the notifications is too much - I’ve pushed a change now that won’t trigger the button click callback, but should still call the listeners.