From 2ac2f186f666b3829386d44ae6463e97de37cf43 Mon Sep 17 00:00:00 2001 From: Manuel Saelices Date: Thu, 3 Oct 2019 22:35:11 +0200 Subject: [PATCH 1/2] fix: force modal to be opened in the root NS component, which fixes #536 --- platform/nativescript/plugins/modal-plugin.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/nativescript/plugins/modal-plugin.js b/platform/nativescript/plugins/modal-plugin.js index 75a984fa..b3ad00da 100644 --- a/platform/nativescript/plugins/modal-plugin.js +++ b/platform/nativescript/plugins/modal-plugin.js @@ -85,7 +85,7 @@ export default { const modalPage = navEntryInstance.$mount().$el.nativeView updateDevtools() - this.$el.nativeView.showModal(modalPage, options) + this.$root.nativeView.showModal(modalPage, options) }) } } From ba22520a1a4b441ed69c04f2d23ca26df690bb87 Mon Sep 17 00:00:00 2001 From: Manuel Saelices Date: Thu, 3 Oct 2019 22:36:13 +0200 Subject: [PATCH 2/2] chore: no need to have the workaround code as the #536 issue is fixed --- samples/app/536.js | 49 ---------------------------------------------- 1 file changed, 49 deletions(-) diff --git a/samples/app/536.js b/samples/app/536.js index 784ae5f0..6ad318da 100644 --- a/samples/app/536.js +++ b/samples/app/536.js @@ -83,52 +83,3 @@ new Vue({ ` }).$start() - -// Working code. Please comment the previous Vue instance and -// uncomment the following commented code in order to workaround the issue -// The workaround is opening the modal in the root component -/* -const TabContentWorking = { - template: ` - -