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

update(dialog): remove .md-actions deprecation #9637

Merged
merged 1 commit into from
Oct 16, 2016

Conversation

devversion
Copy link
Member

  • Removes the .md-actions deprecation from pre- v1.0.0
  • Simplifies the method to search for a close button

Closes #9633

* Removes the `.md-actions` deprecation from pre- v1.0.0
* Simplifies the method to search for a close button

Closes angular#9633
@devversion devversion added the needs: review This PR is waiting on review from the team label Sep 18, 2016
@@ -80,7 +80,7 @@ md-dialog {
}
}

.md-actions, md-dialog-actions {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should leave the style classes for deprecated support (while we removing the warnings).
Otherwise this is a breaking change.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We had them since pre v1.0.0 that's why I intentionally removed them for now. The target is to remove the deprecation completely (two major versions~)

@ThomasBurleson ThomasBurleson added in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs and removed needs: review This PR is waiting on review from the team labels Sep 21, 2016
@ThomasBurleson ThomasBurleson added this to the 1.1.2 milestone Sep 21, 2016
@ThomasBurleson ThomasBurleson added needs: presubmit and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs labels Sep 23, 2016
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels Oct 16, 2016
@kara kara merged commit 2c286b4 into angular:master Oct 16, 2016
@devversion devversion deleted the update/dialog-md-actions-dep branch October 16, 2016 09:06
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.

dialog: deprecate .md-actions class
3 participants