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

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

Issue 2825203003: MD Settings: Remove subpage animation when landing directly on it. (Closed)
Patch Set: fix compiler error Created 3 years, 7 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 * @typedef {{about: boolean, settings: boolean}} 6 * @typedef {{about: boolean, settings: boolean}}
7 */ 7 */
8 var MainPageVisibility; 8 var MainPageVisibility;
9 9
10 /** 10 /**
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 type: Boolean, 68 type: Boolean,
69 value: false, 69 value: false,
70 notify: true, 70 notify: true,
71 }, 71 },
72 72
73 /** 73 /**
74 * Dictionary defining page visibility. Controlled by settings-ui. 74 * Dictionary defining page visibility. Controlled by settings-ui.
75 * @type {!PageVisibility|undefined} 75 * @type {!PageVisibility|undefined}
76 */ 76 */
77 pageVisibility: Object, 77 pageVisibility: Object,
78
79 shouldHideContainer: {
80 type: Boolean,
81 notify: true,
82 }
78 }, 83 },
79 84
80 /** @override */ 85 /** @override */
81 attached: function() { 86 attached: function() {
82 this.listen(this, 'freeze-scroll', 'onFreezeScroll_'); 87 this.listen(this, 'freeze-scroll', 'onFreezeScroll_');
83 this.listen(this, 'lazy-loaded', 'onLazyLoaded_'); 88 this.listen(this, 'lazy-loaded', 'onLazyLoaded_');
84 }, 89 },
85 90
86 /** @private */ 91 /** @private */
87 onLazyLoaded_: function() { 92 onLazyLoaded_: function() {
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
276 text: this.showNoResultsFound_ ? 281 text: this.showNoResultsFound_ ?
277 loadTimeData.getString('searchNoResults') : 282 loadTimeData.getString('searchNoResults') :
278 loadTimeData.getStringF('searchResults', query) 283 loadTimeData.getStringF('searchResults', query)
279 }); 284 });
280 } 285 }
281 }.bind(this)); 286 }.bind(this));
282 }.bind(this), 0); 287 }.bind(this), 0);
283 }.bind(this)); 288 }.bind(this));
284 }, 289 },
285 }); 290 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698