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

Unified Diff: chrome/browser/resources/settings/people_page/sync_page.js

Issue 2512323002: MD Settings: In /syncSetup clicking links in checkbox label should not check/uncheck. (Closed)
Patch Set: add @private annotation to sync page function Created 4 years, 1 month 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/people_page/sync_page.js
diff --git a/chrome/browser/resources/settings/people_page/sync_page.js b/chrome/browser/resources/settings/people_page/sync_page.js
index 7ac0f0db0e0672dfd21c4b62c5b09e8765d56071..c80178ccafd8381d5f532a799c661b343a48e49d 100644
--- a/chrome/browser/resources/settings/people_page/sync_page.js
+++ b/chrome/browser/resources/settings/people_page/sync_page.js
@@ -420,6 +420,16 @@ Polymer({
return !emptyPassphrase && !mismatchedPassphrase;
},
+
+ /**
+ * @param {!Event} event
+ * @private
+ */
+ onLearnMoreTap_: function(event) {
+ // Stop the propagation of events, so that clicking on links inside
+ // checkbox labels won't change the checkbox value.
+ event.stopPropagation();
+ }
});
})();

Powered by Google App Engine
This is Rietveld 408576698