PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix button selected state

Browse Source 
Test: toggle between buttons with a11y service
Fixes: 156990411
Change-Id: I421c6d2e9762794bdf8a090db8a808678ffd8a1c
This commit is contained in:
Julia Reynolds
2020-05-19 14:36:32 -04:00
parent 663cc934d9
commit d1d32e7d22
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/com/android/settings/notification/app/ConversationPriorityPreference.java
View File

@@ -179,7 +179,11 @@ public class ConversationPriorityPreference extends Preference {
// a11y service won't always read the newly appearing text in the right order if the
// selection happens too soon (readback happens on a different thread as layout). post
// the selection to make that conflict less likely
parent.post(() -> mSilenceButton.setSelected(true));
parent.post(() -> {
mSilenceButton.setSelected(true);
mAlertButton.setSelected(false);
mPriorityButton.setSelected(false);
});
} else {
if (isPriority) {
alertSummary.setVisibility(GONE);
@@ -197,7 +201,11 @@ public class ConversationPriorityPreference extends Preference {
mAlertButton.setBackground(unselectedBackground);
mPriorityButton.setBackground(selectedBackground);
mSilenceButton.setBackground(unselectedBackground);
parent.post(() -> mPriorityButton.setSelected(true));
parent.post(() -> {
mSilenceButton.setSelected(false);
mAlertButton.setSelected(false);
mPriorityButton.setSelected(true);
});
} else {
alertSummary.setVisibility(VISIBLE);
alertIcon.setImageTintList(colorAccent);
@@ -214,7 +222,11 @@ public class ConversationPriorityPreference extends Preference {
mAlertButton.setBackground(selectedBackground);
mPriorityButton.setBackground(unselectedBackground);
mSilenceButton.setBackground(unselectedBackground);
parent.post(() -> mAlertButton.setSelected(true));
parent.post(() -> {
mSilenceButton.setSelected(false);
mAlertButton.setSelected(true);
mPriorityButton.setSelected(false);
});
}
}
}