Skip to content

Commit

Permalink
Settings WebUI: hide tap highlight while expanding advanced section
Browse files Browse the repository at this point in the history
Bug: 1061333
Change-Id: I33f1f46d4e7b64ee7ed77e7158ee4a3a77986037
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2159695
Auto-Submit: Esmael Elmoslimany <aee@chromium.org>
Commit-Queue: John Lee <johntlee@chromium.org>
Reviewed-by: John Lee <johntlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#761513}
  • Loading branch information
Esmael El-Moslimany authored and Commit Bot committed Apr 22, 2020
1 parent 4aa0f19 commit 70344df
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 6 deletions.
4 changes: 4 additions & 0 deletions chrome/browser/resources/settings/basic_page/basic_page.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@
overflow: hidden;
}

:host([advanced-toggling-in-progress_]) {
-webkit-tap-highlight-color: transparent;
}

#advancedToggle {
--ink-color: currentColor;
align-items: center;
Expand Down
16 changes: 10 additions & 6 deletions chrome/browser/resources/settings/basic_page/basic_page.js
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,16 @@ Polymer({

/** @private {!Route|undefined} */
currentRoute_: Object,

/**
* Used to avoid handling a new toggle while currently toggling.
* @private
*/
advancedTogglingInProgress_: {
type: Boolean,
value: false,
reflectToAttribute: true,
},
},

hostAttributes: {
Expand All @@ -140,12 +150,6 @@ Polymer({
'subpage-expand': 'onSubpageExpanded_',
},

/**
* Used to avoid handling a new toggle while currently toggling.
* @private {boolean}
*/
advancedTogglingInProgress_: false,

// <if expr="chromeos">
/** @private {boolean} */
osBannerShowMetricRecorded_: false,
Expand Down

0 comments on commit 70344df

Please sign in to comment.