Skip to content
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

bug: IonMenu's backdrop not displayed on iOS after upgrade from 4.8.1 to 4.9.0/4.9.1 #19387

Closed
maxtacco opened this issue Sep 18, 2019 · 8 comments
Labels
package: core @ionic/core package type: bug a confirmed bug report

Comments

@maxtacco
Copy link

Bug Report

Ionic version:

[x] 4.x

Current behavior:
Ionic menu has no backdrop when opened on iOS since version 4.9.0

Expected behavior:
Ionic menu should have backdrop

Other information:
Android is fine. Works fine using 4.8.1

Ionic info:

Ionic:

   Ionic CLI                     : 5.2.4 (/Users/[user]/.nvm/versions/node/v10.15.3/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.9.0
   @angular-devkit/build-angular : 0.13.9
   @angular-devkit/schematics    : 7.3.9
   @angular/cli                  : 7.3.9
   @ionic/angular-toolkit        : 1.5.1

Cordova:

   Cordova CLI       : 9.0.0 ([email protected])
   Cordova Platforms : ios 5.0.1
   Cordova Plugins   : no whitelisted plugins (3 plugins total)

Utility:

   cordova-res : 0.6.0
   native-run  : 0.2.8
@ionitron-bot ionitron-bot bot added the triage label Sep 18, 2019
@maxtacco
Copy link
Author

maxtacco commented Sep 18, 2019

It looks like both "overlay" and "push" menu types work fine on iOS (tested in Chrome using Dev tools). But, using "reveal" produces the issue. On Android, only "overlay" works fine, both "reveal" and "push" don't add the backdrop.

@tricki
Copy link

tricki commented Sep 19, 2019

I have a similar problem where nothing in the menu is clickable on iOS because the ion-content's .transition-cover overlays it.

Plunker: https://next.plnkr.co/edit/83Qz2MFr8B8lD4cc?preview

This demo forces iOS mode. Just open the menu and inspect any menu item which will wrongly select the following element:
image

Ionic:

   Ionic CLI : 5.2.4 (C:\Users\thomas\AppData\Roaming\npm\node_modules\ionic)

Cordova:

   Cordova CLI       : 9.0.0 ([email protected])
   Cordova Platforms : android 8.0.0
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 3.1.2, (and 14 other plugins)

Utility:

   cordova-res : 0.6.0
   native-run  : not installed

System:

   Android SDK Tools : 26.1.1 (D:\ProgramFiles\Android\sdk)
   NodeJS            : v10.15.0 (C:\Program Files\nodejs\node.exe)
   npm               : 6.11.3
   OS                : Windows 10

@VIMHaos
Copy link

VIMHaos commented Sep 24, 2019

Same problem on iOS after upgrade from 4.8.1 to 4.9.0/4.9.1

@liamdebeasi
Copy link
Contributor

@tricki Can you open a separate issue for this?

I can reproduce the original issue. We will look into it and will post here when we have more to share. Thanks!

@liamdebeasi liamdebeasi added package: core @ionic/core package type: bug a confirmed bug report labels Sep 26, 2019
@ionitron-bot ionitron-bot bot removed the triage label Sep 26, 2019
@liamdebeasi
Copy link
Contributor

Thanks! This issue has been fixed and will be available in the next release of Ionic.

@maxtacco
Copy link
Author

@liamdebeasi Thank you for a quick turnaround!

@calvinckho
Copy link

this is to confirm that @ionic/core 4.10.0 have fixed it.

@ionitron-bot
Copy link

ionitron-bot bot commented Oct 28, 2019

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Oct 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

No branches or pull requests

5 participants