Ionic-framework: ionic 5: Modal over Modal (iOS13 Style): Border radius missing

[EinfachHans]

Bug Report

Ionic version:


[x] 5.x

Current behavior:

When i create a Modal from a Modal with presentingElement: this.modalCtrl.getTop(), the first Modal, that is in the Background, don't get a Class to have rounded corners. The inner Modal gets a Class that sets:

border-top-left-radius: 10px;
border-top-right-radius: 10px;
border-bottom-right-radius: 0;
border-bottom-left-radius: 0;

Also when i set the presenting Element to the IonRouterOutlet it works and it has rounded corners.

Expected behavior:

I think the behavior should be that the presenting Element always should have rounded corners. Even when it is an other Modal.

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 6.1.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.0.0
   @angular-devkit/build-angular : 0.803.25
   @angular-devkit/schematics    : 8.3.25
   @angular/cli                  : 8.3.25
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 9.0.0 ([email protected])
   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 53 other plugins)

Utility:

   cordova-res (update available: 0.9.0) : 0.8.1
   native-run                            : 0.3.0

System:

   Android SDK Tools : 25.2.3 (/Users/hanskrywalsky/Library/Android/sdk)
   ios-deploy        : 1.10.0
   ios-sim           : 8.0.2
   NodeJS            : v10.16.3 (/usr/local/bin/node)
   npm               : 6.13.7
   OS                : macOS Catalina
   Xcode             : Xcode 11.3.1 Build version 11C504

[liamdebeasi]

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue?

[EinfachHans]

Of course:
https://github.com/HansKrywaa/ionic-issue-20475

The Case is (i forget that) that the First Modal is a Full Screen Modal that don't have the presenting Element

[liamdebeasi]

Thanks for the follow up. Can you try the following dev build and let me know if it resolves the issue?

npm i @ionic/[email protected]

[EinfachHans]

@liamdebeasi yeah resolved! 💪🏼

[liamdebeasi]

Thanks for the issue. This has been resolved via https://github.com/ionic-team/ionic/pull/20476 and will be available in an upcoming release of Ionic Framework.

[ionitron-bot[bot]]

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.