summaryrefslogtreecommitdiff
path: root/services/accessibility
diff options
context:
space:
mode:
author Gavin Williams <gavinwill@google.com> 2025-03-01 03:32:56 +0000
committer Gavin Williams <gavinwill@google.com> 2025-03-07 08:51:26 -0800
commit8f3a4ffd6d3b7e6a21c25b5f5f8380d9ab12d802 (patch)
treef6d9033f3a448affa346f5b0fa43bc7c8c70ef9e /services/accessibility
parent8d0a510b77fbef1a7414b086061f92b94c8cced9 (diff)
a11y: Resume autoclick when any button is clicked
Demo: http://b/398078456#comment2 Bug: b/398078456 Flag: com.android.server.accessibility.enable_autoclick_indicator Test: AutoclickTypePanelTest Change-Id: I8f43148dfbf078ab39c9246234a03fd5599348df
Diffstat (limited to 'services/accessibility')
-rw-r--r--services/accessibility/java/com/android/server/accessibility/autoclick/AutoclickTypePanel.java32
1 files changed, 26 insertions, 6 deletions
diff --git a/services/accessibility/java/com/android/server/accessibility/autoclick/AutoclickTypePanel.java b/services/accessibility/java/com/android/server/accessibility/autoclick/AutoclickTypePanel.java
index 90ddc43ae3ed..57fa77d73729 100644
--- a/services/accessibility/java/com/android/server/accessibility/autoclick/AutoclickTypePanel.java
+++ b/services/accessibility/java/com/android/server/accessibility/autoclick/AutoclickTypePanel.java
@@ -266,13 +266,21 @@ public class AutoclickTypePanel {
}
private void initializeButtonState() {
- mLeftClickButton.setOnClickListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_LEFT_CLICK));
- mRightClickButton.setOnClickListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_RIGHT_CLICK));
+ // Use `createButtonListener()` to append extra pause logic to each button's click.
+ mLeftClickButton.setOnClickListener(
+ wrapWithTogglePauseListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_LEFT_CLICK)));
+ mRightClickButton.setOnClickListener(
+ wrapWithTogglePauseListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_RIGHT_CLICK)));
mDoubleClickButton.setOnClickListener(
- v -> togglePanelExpansion(AUTOCLICK_TYPE_DOUBLE_CLICK));
- mScrollButton.setOnClickListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_SCROLL));
- mDragButton.setOnClickListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_DRAG));
- mPositionButton.setOnClickListener(v -> moveToNextCorner());
+ wrapWithTogglePauseListener(
+ v -> togglePanelExpansion(AUTOCLICK_TYPE_DOUBLE_CLICK)));
+ mScrollButton.setOnClickListener(
+ wrapWithTogglePauseListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_SCROLL)));
+ mDragButton.setOnClickListener(
+ wrapWithTogglePauseListener(v -> togglePanelExpansion(AUTOCLICK_TYPE_DRAG)));
+ mPositionButton.setOnClickListener(wrapWithTogglePauseListener(v -> moveToNextCorner()));
+
+ // The pause button calls `togglePause()` directly so it does not need extra logic.
mPauseButton.setOnClickListener(v -> togglePause());
// Initializes panel as collapsed state and only displays the left click button.
@@ -517,6 +525,18 @@ public class AutoclickTypePanel {
return true;
}
+ /* Appends a check of the pause state to the button's listener. */
+ private View.OnClickListener wrapWithTogglePauseListener(View.OnClickListener listener) {
+ return v -> {
+ listener.onClick(v);
+
+ // Resumes autoclick if the button is clicked while in a paused state.
+ if (mPaused) {
+ togglePause();
+ }
+ };
+ }
+
@VisibleForTesting
boolean getExpansionStateForTesting() {
return mExpanded;