Advanced toggle button doesn't react to first click at chrome://md-settings/advanced |
|||
Issue description1. chrome://md-settings/advanced 2. click Advanced toggle button Expected: Advanced closes. Actual: Advanced stays open and page scrolls down. This is, weirdly, a different root cause from all the other Advanced toggle bugs I've opened.
,
Jul 22 2016
,
Jul 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5b7b3f37f385179dac9a1ae4407a20fb2a28647a commit 5b7b3f37f385179dac9a1ae4407a20fb2a28647a Author: michaelpg <michaelpg@chromium.org> Date: Fri Jul 22 19:55:09 2016 MD Settings: refactor some main page properties, fix Advanced toggle Fixes the Advanced toggle when navigating directly to Advanced. Make currentRouteChanged_ less confusing. Wrap non-About stuff in a dom-if. Removes a bunch of redundant properties. Part of https://codereview.chromium.org/2106013002/ BUG= 630010 , 589681 R=dbeam@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2160713002 Cr-Commit-Position: refs/heads/master@{#407236} [modify] https://crrev.com/5b7b3f37f385179dac9a1ae4407a20fb2a28647a/chrome/browser/resources/settings/settings_main/settings_main.html [modify] https://crrev.com/5b7b3f37f385179dac9a1ae4407a20fb2a28647a/chrome/browser/resources/settings/settings_main/settings_main.js
,
Aug 16 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by michae...@chromium.org
, Jul 22 2016