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

Unified Diff: chrome/browser/resources/settings/settings_main/settings_main.js

Issue 2236723002: MD Settings: Hide the advanced page toggle control during searching. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 4 years, 4 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_main/settings_main.js
diff --git a/chrome/browser/resources/settings/settings_main/settings_main.js b/chrome/browser/resources/settings/settings_main/settings_main.js
index 676429b994c7533a5426cb7ced6b98673c6cadf0..408b68c55cad19cff860a9e067d7ee0d64d42732 100644
--- a/chrome/browser/resources/settings/settings_main/settings_main.js
+++ b/chrome/browser/resources/settings/settings_main/settings_main.js
@@ -3,6 +3,11 @@
// found in the LICENSE file.
/**
+ * @typedef {{about: boolean, basic: boolean, advanced: boolean}}
+ */
+var MainPageVisibility;
+
+/**
* @fileoverview
* 'settings-main' displays the selected settings page.
*/
@@ -37,8 +42,7 @@ Polymer({
/**
* Controls which main pages are displayed via dom-ifs.
- * @type {!{about: boolean, basic: boolean, advanced: boolean}}
- * @private
+ * @private {!MainPageVisibility}
*/
showPages_: {
type: Object,
@@ -47,6 +51,17 @@ Polymer({
},
},
+ /**
+ * The main pages that were displayed before search was initiated. When
+ * |null| it indicates that currently the page is displaying its normal
+ * contents, instead of displaying search results.
+ * @private {?MainPageVisibility}
+ */
+ previousShowPages_: {
+ type: Object,
+ value: null,
+ },
+
/** @private */
showNoResultsFound_: {
type: Boolean,
@@ -112,13 +127,12 @@ Polymer({
},
/**
- * @param {boolean} showBasicPage
- * @param {boolean} inSubpage
* @return {boolean}
* @private
*/
- showAdvancedToggle_: function(showBasicPage, inSubpage) {
- return showBasicPage && !inSubpage;
+ showAdvancedToggle_: function() {
+ var inSearchMode = !!this.previousShowPages_;
+ return this.showPages_.basic && !this.inSubpage_ && !inSearchMode;
},
/** @protected */
@@ -198,6 +212,12 @@ Polymer({
* @return {!Promise} A promise indicating that searching finished.
*/
searchContents: function(query) {
+ if (!this.previousShowPages_) {
+ // Store which pages are shown before search, so that they can be restored
+ // after the user clears the search results.
+ this.previousShowPages_ = this.showPages_;
+ }
+
this.ensureInDefaultSearchPage_();
this.toolbarSpinnerActive = true;
@@ -223,8 +243,15 @@ Polymer({
}
this.toolbarSpinnerActive = false;
+ var showingSearchResults = !request.isSame('');
this.showNoResultsFound_ =
- !request.isSame('') && !request.didFindMatches();
+ showingSearchResults && !request.didFindMatches();
+
+ if (!showingSearchResults) {
+ // Restore the pages that were shown before search was initiated.
+ this.showPages_ = assert(this.previousShowPages_);
+ this.previousShowPages_ = null;
+ }
}.bind(this));
}.bind(this), 0);
}.bind(this));

Powered by Google App Engine
This is Rietveld 408576698