-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add multiple : true to the $mdDialog ? #100
Comments
I'm having the same issue. Ever since the 1.1.2 release of angular material, the color picker inside a $mdDialog will close the dialog https://github.com/angular/material/blob/master/CHANGELOG.md |
Hello, I was having the same issue. So i dig inside dist/mdColorPicker.js. The plugin is using skipHide to implement the multi dialog functionality but skipHide is removed from angular material. They added the multiple option, so go inside dist/mdColorPicker.js search for skipHide and replace it with multiple and that's it. I hope a next version @brianpkelley will implement that new option instead of skipHide. |
+1 |
Is this fixed in the latest version |
+1 |
I set default options for .config(function($provide) {
// Decorate the $mdDialog service using $provide.decorator
$provide.decorator("$mdDialog", function ($delegate) {
// Get a handle of the show method
var methodHandle = $delegate.show;
function decorateDialogShow () {
var args = angular.extend({}, arguments[0], { multiple: true })
return methodHandle(args);
}
$delegate.show = decorateDialogShow;
return $delegate;
});
}); |
Hello, thank you for your plugin.
I would like to use it inside a $mdDialog, is it possible to add the multiple option for the $mdDialog ?
Thanks
The text was updated successfully, but these errors were encountered: