Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Conversation

devversion
Copy link
Member

  • The md-is-locked-open class-selector wasn't valid, because it wasn't probably privatized
  • The disabled backdrop tests, weren't valid, because the the sidenav wasn't actually opening.

KUDOS to @EladBezalel for his unbeatable help!

Fixes #7537

* The md-is-locked-open class-selector wasn't valid, because it wasn't probably privatized
* The disabled backdrop tests, weren't valid, because the the sidenav wasn't actually opening.

Fixes angular#7537
@ErinCoughlan
Copy link
Contributor

LGTM. Nice catch.

@devversion devversion added the pr: merge ready This PR is ready for a caretaker to review label Mar 29, 2016
gmoothart pushed a commit to gmoothart/material that referenced this pull request Apr 5, 2016
* The md-is-locked-open class-selector wasn't valid, because it wasn't probably privatized
* The disabled backdrop tests, weren't valid, because the the sidenav wasn't actually opening.

Fixes angular#7537

Closes angular#7790
@devversion devversion deleted the fix/sidenav-backdrop-clickable branch April 19, 2016 19:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants