I had initially started down that path but this is executed on an ajax success callback so it's not really "blindly" toggling (i.e. we know that the state changed if the success callback is executed).

I'm totally fine with adding an if/else statement on the state so we have something like

    if ( public ) {
      private_label.hide(400);
    } else {
      private_label.show(400);
    }

although I think effectively it's the same thing.