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

Side by Side Diff: chrome/browser/resources/settings/settings_ui/settings_ui.js

Issue 2825203003: MD Settings: Remove subpage animation when landing directly on it. (Closed)
Patch Set: nits Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * 'settings-ui' implements the UI for the Settings page. 7 * 'settings-ui' implements the UI for the Settings page.
8 * 8 *
9 * Example: 9 * Example:
10 * 10 *
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 }, 151 },
152 downloads: { 152 downloads: {
153 googleDrive: false, 153 googleDrive: false,
154 }, 154 },
155 // </if> 155 // </if>
156 }; 156 };
157 } 157 }
158 158
159 this.showAndroidApps_ = loadTimeData.valueExists('androidAppsAllowed') && 159 this.showAndroidApps_ = loadTimeData.valueExists('androidAppsAllowed') &&
160 loadTimeData.getBoolean('androidAppsAllowed'); 160 loadTimeData.getBoolean('androidAppsAllowed');
161
162 this.addEventListener('show-container', function() {
163 this.$.container.style.visibility = 'visible';
164 }.bind(this));
165
166 this.addEventListener('hide-container', function() {
167 this.$.container.style.visibility = 'hidden';
168 }.bind(this));
161 }, 169 },
162 170
163 /** @private {?IntersectionObserver} */ 171 /** @private {?IntersectionObserver} */
164 intersectionObserver_: null, 172 intersectionObserver_: null,
165 173
166 /** @override */ 174 /** @override */
167 attached: function() { 175 attached: function() {
168 document.documentElement.classList.remove('loading'); 176 document.documentElement.classList.remove('loading');
169 177
170 setTimeout(function() { 178 setTimeout(function() {
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 this.$.container.removeAttribute('tabindex'); 286 this.$.container.removeAttribute('tabindex');
279 }.bind(this)); 287 }.bind(this));
280 }, 288 },
281 289
282 /** @private */ 290 /** @private */
283 directionDelegateChanged_: function() { 291 directionDelegateChanged_: function() {
284 this.$.drawer.align = this.directionDelegate.isRtl() ? 292 this.$.drawer.align = this.directionDelegate.isRtl() ?
285 'right' : 'left'; 293 'right' : 'left';
286 }, 294 },
287 }); 295 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698