Created attachment 19804 [details] holds warning The attached image is confusing. The radio buttons only apply if you click the 'yes' button. But if you click the 'no' button the radio buttons are ignored. The functionality is good! But it's confusing to the user because it makes it look like the radio buttons apply no matter what button you choose and both buttons end up messing with the hold so users don't want to choose anything.
How do you think this should work instead?
Maybe 3 buttons - no radio buttons. So : Revert waiting status & Check out Cancel hold & Check out No, don't check out
Still valid?
Yes.
Created attachment 48196 [details] Screenshot in current master > Revert waiting status & Check out > Cancel hold & Check out > No, don't check out Thos options would be more accurate too since the current interface suggests that 'Cancel hold' and 'Revert waiting status' are settings which apply to /all/ the buttons, as if you could select 'cancel hold' and click 'Don't check out' and have both happen. That's not the way it works.