ionic-material-design-lite icon indicating copy to clipboard operation
ionic-material-design-lite copied to clipboard

Platform Override Not Working

Open DahliaWitt opened this issue 9 years ago • 7 comments

ios simulator screen shot sep 6 2015 11 11 30 pm

If you notice the header bar, platform overrides aren't working within the iOS simulator. It seems to work just fine on Safari when testing (and the Ionic Lab as well).

I have $ionicMaterialConfigProvider.enableForAllPlatforms(); in my .config function.

DahliaWitt avatar Sep 07 '15 03:09 DahliaWitt

The sidebar animation doesn't work, it just pops open non-smoothly. That's the biggest issue.

DahliaWitt avatar Sep 07 '15 03:09 DahliaWitt

The side bar changes in Android as a pop over, so currently the animation isn't complete, which is why you are experiencing this on iOS.

If you are able to test on a physical device that would be really helpful as the simulator has a few problems of its own when allocating memory to webviews which affects performance.

jjaybrown avatar Sep 07 '15 07:09 jjaybrown

Error still occurring in RC-1

DahliaWitt avatar Sep 16 '15 00:09 DahliaWitt

You've not provided any information I requested , so I've not been able to replicate this.

jjaybrown avatar Sep 16 '15 06:09 jjaybrown

My apologies, tested on a real device and the behavior remains the same. Steps to reproduce for me:

  1. Start new side menu from ionic start sidemenu
  2. Inject Ionic Material and IonIcons
  3. Add the default override to config.
  4. Run on iOS

Everything else is working but the side menu animation, it just appears open and close without a sliding animation.

DahliaWitt avatar Sep 17 '15 14:09 DahliaWitt

Some more information after testing around a bit:

  • It still occurs without the override in the config.
  • It will go away if you remove the CSS file, the JS file does not have an effect.

DahliaWitt avatar Sep 17 '15 14:09 DahliaWitt

Thanks for providing more information, I'll look into the animation. Maybe it's being over written in the of the mixins.

jjaybrown avatar Sep 18 '15 08:09 jjaybrown