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

fix(dialog): add extra classes to identify buttons #9463

Merged
merged 1 commit into from
Aug 31, 2016

Conversation

crisbeto
Copy link
Member

Adds classes to help identify the confirmation and close buttons in a dialog. This helps with writing tests that involve the dialog, because it doesn't depend on DOM order.

CC @ErinCoughlan

Adds classes to help identify the confirmation and close buttons in a dialog. This helps with writing tests that involve the dialog, because it doesn't depend on DOM order.
@crisbeto crisbeto added needs: review This PR is waiting on review from the team - Easy fix labels Aug 30, 2016
@ErinCoughlan
Copy link
Contributor

LGTM

@gmoothart
Copy link
Contributor

thanks for this!

@topherfangio
Copy link
Contributor

LGTM 👍

@ThomasBurleson ThomasBurleson added g3: reported The issue was reported by an internal or external product team. and removed needs: review This PR is waiting on review from the team labels Aug 30, 2016
@ThomasBurleson ThomasBurleson added this to the 1.1.1 milestone Aug 30, 2016
@ThomasBurleson ThomasBurleson added the pr: merge ready This PR is ready for a caretaker to review label Aug 30, 2016
@hansl hansl merged commit b11441c into angular:master Aug 31, 2016
@fuchao2012
Copy link

1024

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
g3: reported The issue was reported by an internal or external product team. 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.

7 participants