Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Dont make sidebar mask diaplay:none by default.#5365

Merged
camelburrito merged 1 commit into
ampproject:masterfrom
camelburrito:prodfix
Oct 1, 2016
Merged

Dont make sidebar mask diaplay:none by default.#5365
camelburrito merged 1 commit into
ampproject:masterfrom
camelburrito:prodfix

Conversation

@camelburrito

Copy link
Copy Markdown
Contributor

Fixes
Can't dismiss sidebar with click on main page content #5362

Fixes
Can't dismiss sidebar with click on main page content ampproject#5362
@aghassemi

Copy link
Copy Markdown
Contributor

We should start using computedStyle instead of style in tests to catch cases like this. Maybe a good start would be should close sidebar on escape test in sidebar.

@camelburrito camelburrito merged commit 136281c into ampproject:master Oct 1, 2016
dreamofabear pushed a commit to dreamofabear/amphtml that referenced this pull request Oct 4, 2016
Fixes
Can't dismiss sidebar with click on main page content ampproject#5362
dreamofabear pushed a commit to dreamofabear/amphtml that referenced this pull request Oct 4, 2016
Fixes
Can't dismiss sidebar with click on main page content ampproject#5362
@camelburrito camelburrito deleted the prodfix branch October 4, 2016 04:59
erwinmombay pushed a commit that referenced this pull request Oct 6, 2016
Fixes
Can't dismiss sidebar with click on main page content #5362
dreamofabear pushed a commit to dreamofabear/amphtml that referenced this pull request Oct 12, 2016
Fixes
Can't dismiss sidebar with click on main page content ampproject#5362
mityaha pushed a commit to brightcove-archive/ooyala_amphtml that referenced this pull request Nov 30, 2016
Fixes
Can't dismiss sidebar with click on main page content ampproject#5362
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants