alenaksu / mdPickers

Material Design date/time pickers for Angular Material
MIT License
293 stars 216 forks source link

$mdDialog option skipHide #178

Open norbornen opened 7 years ago

norbornen commented 7 years ago

Hello!

You need replace option 'skipHide' to option 'multiple' in $mdDialog constructor.

ghost commented 7 years ago

I updated my version to use 'multiple' instead of 'skipHide' and it's not working.. using angular material 1.1.2...please check the attached codepen...we are in production now and it is causing trouble to lot of users please tell the fix to it

http://codepen.io/Prajwal_Jagadish/pen/ygZodj

LoganDupont commented 7 years ago

This should solve the problem for now

var oldShow = $mdDialog.show;
  $mdDialog.show = function (options) {
    if (options.hasOwnProperty("skipHide")) {
      options.multiple = options.skipHide;
    }
   return oldShow(options);
};
ghost commented 7 years ago

@LoganDupont can you show it in code pen attached http://codepen.io/Prajwal_Jagadish/pen/ygZodj

Thank you in advance

LoganDupont commented 7 years ago

@Prajwal8 Here is the codepen

ghost commented 7 years ago

Thanks a lot @LoganDupont. saved my day.

dpoetzsch commented 7 years ago

If you still need this you can use my fork for now https://github.com/dpoetzsch/md-pickers. It has merged pull request #189 which should fix this issue. For details on the fork see #192.