From 57973ac96b3b121bfcd70ad864aec0192f4e2b68 Mon Sep 17 00:00:00 2001 From: Ingo Weinhold Date: Fri, 20 Dec 2013 15:21:06 +0100 Subject: [PATCH] BRadioButton: Fix spurious Invoke() MouseUp(): Don't call SetValue() and Invoke() when the button was already the selected one. --- src/kits/interface/RadioButton.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/kits/interface/RadioButton.cpp b/src/kits/interface/RadioButton.cpp index 3e7659d0eb..89180954aa 100644 --- a/src/kits/interface/RadioButton.cpp +++ b/src/kits/interface/RadioButton.cpp @@ -420,8 +420,10 @@ BRadioButton::MouseUp(BPoint point) fOutlined = Bounds().Contains(point); if (fOutlined) { fOutlined = false; - SetValue(B_CONTROL_ON); - Invoke(); + if (Value() != B_CONTROL_ON) { + SetValue(B_CONTROL_ON); + Invoke(); + } } Invalidate();