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

Unified Diff: chrome/browser/resources/settings/settings_page/settings_animated_pages.js

Issue 2841873004: MD Settings: Fix subpage navigation focus for bluetooth+internet (Closed)
Patch Set: . Created 3 years, 8 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/settings_page/settings_animated_pages.js
diff --git a/chrome/browser/resources/settings/settings_page/settings_animated_pages.js b/chrome/browser/resources/settings/settings_page/settings_animated_pages.js
index b50d4fc6d0680c27ed6700cc9be789ca19d15000..88cced6e7932b6a3fb4cb9751c07db16529bb0b1 100644
--- a/chrome/browser/resources/settings/settings_page/settings_animated_pages.js
+++ b/chrome/browser/resources/settings/settings_page/settings_animated_pages.js
@@ -47,8 +47,8 @@ Polymer({
/** @override */
created: function() {
// Observe the light DOM so we know when it's ready.
- this.lightDomObserver_ = Polymer.dom(this).observeNodes(
- this.lightDomChanged_.bind(this));
+ this.lightDomObserver_ =
+ Polymer.dom(this).observeNodes(this.lightDomChanged_.bind(this));
},
/** @override */
@@ -72,10 +72,12 @@ Polymer({
// Only handle iron-select events from neon-animatable elements and the
// given whitelist of settings-subpage instances.
if (!e.detail.item.matches(
- 'neon-animatable, ' +
- 'settings-subpage#site-settings, ' +
- 'settings-subpage[route-path=\"' +
- settings.Route.SITE_SETTINGS_COOKIES.path + '\"]')) {
+ 'neon-animatable, ' +
+ 'settings-subpage#site-settings, ' +
+ 'settings-subpage[route-path=\"' +
+ settings.Route.SITE_SETTINGS_COOKIES.path + '\"], ' +
+ 'settings-subpage[route-path=\"' +
+ settings.Route.INTERNET_NETWORKS.path + '\"]')) {
return;
}
@@ -124,8 +126,7 @@ Polymer({
if (!this.queuedRouteChange_)
return;
this.async(this.currentRouteChanged.bind(
- this,
- this.queuedRouteChange_.newRoute,
+ this, this.queuedRouteChange_.newRoute,
this.queuedRouteChange_.oldRoute));
},
@@ -203,8 +204,8 @@ Polymer({
return;
// Set the subpage's id for use by neon-animated-pages.
- var subpage = /** @type {{_content: DocumentFragment}} */(template)._content
- .querySelector('settings-subpage');
+ var subpage = /** @type {{_content: DocumentFragment}} */ (template)
+ ._content.querySelector('settings-subpage');
subpage.setAttribute('route-path', routePath);
// Carry over the 'no-search' attribute from the template to the stamped

Powered by Google App Engine
This is Rietveld 408576698