Skip to content

Commit 950a2ae

Browse files
committed
fix radio group not retaining selection if radiobutton already selected
1 parent 508253e commit 950a2ae

1 file changed

Lines changed: 4 additions & 11 deletions

File tree

AndroidBootstrap/src/main/java/com/beardedhen/androidbootstrap/BootstrapButton.java

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package com.beardedhen.androidbootstrap;
22

33
import android.content.Context;
4-
import android.content.DialogInterface;
54
import android.content.res.TypedArray;
65
import android.graphics.drawable.Drawable;
76
import android.os.Bundle;
@@ -10,7 +9,6 @@
109
import android.support.annotation.Nullable;
1110
import android.util.AttributeSet;
1211
import android.view.MotionEvent;
13-
import android.view.View;
1412
import android.view.ViewParent;
1513

1614
import com.beardedhen.androidbootstrap.api.attributes.BootstrapBrand;
@@ -221,17 +219,12 @@ public void setSelected(boolean selected) {
221219

222220
private boolean handleRadioEvent(@NonNull MotionEvent event) {
223221
if (event.getAction() == MotionEvent.ACTION_DOWN) {
224-
if (isSelected()) {
225-
setSelected(false);
226-
}
227-
else { // notify parent to deselect any peers
228-
setSelected(true);
222+
setSelected(true); // notify parent to deselect any peers
229223

230-
ViewParent parent = getParent();
224+
ViewParent parent = getParent();
231225

232-
if (parent instanceof BootstrapButtonGroup) {
233-
((BootstrapButtonGroup) parent).onRadioToggle(parentIndex);
234-
}
226+
if (parent instanceof BootstrapButtonGroup) {
227+
((BootstrapButtonGroup) parent).onRadioToggle(parentIndex);
235228
}
236229
return true;
237230
}

0 commit comments

Comments
 (0)