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

Commit

Permalink
fix(dialog): fixed alert and confim md-transition-in class attachment
Browse files Browse the repository at this point in the history
Alert and confirm dialogs had in their template class attribute that was binded to `dialog.css`, in case the css property was undefined it caused the `md-transition-in` class to be applied but than deleted.

The solution was to change the usage to ng-class and let angular set the class.

fixes #4862. closes #5006.
  • Loading branch information
EladBezalel authored and ThomasBurleson committed Oct 6, 2015
1 parent 460e3ab commit 82160e1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/components/dialog/dialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ function MdDialogProvider($$interimElementProvider) {
function advancedDialogOptions($mdDialog, $mdTheming) {
return {
template: [
'<md-dialog md-theme="{{ dialog.theme }}" aria-label="{{ dialog.ariaLabel }}" class="{{ dialog.css }}">',
'<md-dialog md-theme="{{ dialog.theme }}" aria-label="{{ dialog.ariaLabel }}" ng-class="dialog.css">',
' <md-dialog-content role="document" tabIndex="-1">',
' <h2 class="md-title">{{ dialog.title }}</h2>',
' <div class="md-dialog-content-body" md-template="::dialog.mdContent"></div>',
Expand Down

0 comments on commit 82160e1

Please sign in to comment.