Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Unified Diff: chrome/browser/resources/settings/device_page/pointers.html

Issue 2753353002: MD Settings: Pointers: Avoid setting mouse swap pref while mouse pressed (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/device_page/pointers.html
diff --git a/chrome/browser/resources/settings/device_page/pointers.html b/chrome/browser/resources/settings/device_page/pointers.html
index baa1d5140ea8a98c9d6ec220293a7957f5f44ac2..724c9a056a312b679deabb9750c9870009e58518 100644
--- a/chrome/browser/resources/settings/device_page/pointers.html
+++ b/chrome/browser/resources/settings/device_page/pointers.html
@@ -29,9 +29,14 @@
<!-- Subsection title only appears if both mouse and touchpad exist. -->
<h2 hidden$="[[!hasTouchpad]]">$i18n{mouseTitle}</h2>
<div class="settings-box first">
- <settings-toggle-button class="start"
+ <!-- Do not change the mouse button pref before the mouse is released.
+ See crbug.com/686949 -->
+ <settings-toggle-button id="mouseSwapButton" class="start"
pref="{{prefs.settings.mouse.primary_right}}"
- label="$i18n{mouseSwapButtons}">
+ label="$i18n{mouseSwapButtons}"
+ on-settings-boolean-control-change="onMouseSwapButtonsChange_"
+ on-down="onMouseSwapButtonsDown_" on-up="onMouseSwapButtonsUp_"
+ no-set-pref>
</settings-toggle-button>
</div>
<div class="settings-box">

Powered by Google App Engine
This is Rietveld 408576698