From 6a43eebf60f6a83db280b891b89e8a0e464c96ff Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Thu, 19 Sep 2024 06:43:00 +0200
Subject: [PATCH] fix: remove container prop in NcModal and related components

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../BreakoutRoomsEditor/BreakoutRoomsEditor.vue           | 7 +------
 .../ConversationSettings/ConversationSettingsDialog.vue   | 7 +------
 src/components/GuestWelcomeWindow.vue                     | 7 +------
 .../LeftSidebar/CallPhoneDialog/CallPhoneDialog.vue       | 7 -------
 .../LeftSidebar/ConversationsList/Conversation.vue        | 7 +------
 src/components/LeftSidebar/InvitationHandler.vue          | 5 -----
 .../OpenConversationsList/OpenConversationsList.vue       | 5 -----
 src/components/MediaSettings/MediaSettings.vue            | 5 -----
 .../Message/MessageButtonsBar/MessageForwarder.vue        | 6 ------
 .../Message/MessageButtonsBar/MessageTranslateDialog.vue  | 5 -----
 .../MessagesGroup/Message/MessagePart/ReactionsList.vue   | 5 -----
 .../NewConversationDialog/NewConversationDialog.vue       | 6 ------
 src/components/NewMessage/NewMessagePollEditor.vue        | 5 -----
 src/components/PermissionsEditor/PermissionsEditor.vue    | 8 ++------
 src/components/PollViewer/PollViewer.vue                  | 5 -----
 .../RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue   | 5 -----
 src/components/RightSidebar/Participants/Participant.vue  | 7 +------
 .../RightSidebar/SharedItems/SharedItemsBrowser.vue       | 5 -----
 src/components/RoomSelector.vue                           | 6 ------
 src/components/SettingsDialog/SettingsDialog.vue          | 7 +------
 20 files changed, 8 insertions(+), 112 deletions(-)

diff --git a/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue b/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
index 0329a7f9240..085c9164dcc 100644
--- a/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
+++ b/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
@@ -4,8 +4,7 @@
 -->
 
 <template>
-	<NcModal :container="container"
-		:class="{'modal-mask__participants-step': isEditingParticipants}"
+	<NcModal :class="{'modal-mask__participants-step': isEditingParticipants}"
 		:label-id="dialogHeaderId"
 		v-on="$listeners">
 		<div class="breakout-rooms-editor"
@@ -130,10 +129,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		modalTitle() {
 			return this.isEditingParticipants
 				? t('spreed', 'Assign participants to rooms')
diff --git a/src/components/ConversationSettings/ConversationSettingsDialog.vue b/src/components/ConversationSettings/ConversationSettingsDialog.vue
index f978795a02f..07e1498d75a 100644
--- a/src/components/ConversationSettings/ConversationSettingsDialog.vue
+++ b/src/components/ConversationSettings/ConversationSettingsDialog.vue
@@ -8,8 +8,7 @@
 		:aria-label="t('spreed', 'Conversation settings')"
 		:name="t('spreed', 'Conversation settings')"
 		:open.sync="showSettings"
-		:show-navigation="true"
-		:container="container">
+		show-navigation>
 		<NcAppSettingsSection id="basic-info"
 			:name="t('spreed', 'Basic Info')">
 			<BasicInfo :conversation="conversation"
@@ -161,10 +160,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		canUserEnableSIP() {
 			return this.conversation.canEnableSIP
 		},
diff --git a/src/components/GuestWelcomeWindow.vue b/src/components/GuestWelcomeWindow.vue
index 3207e9c24bf..4c13d393937 100644
--- a/src/components/GuestWelcomeWindow.vue
+++ b/src/components/GuestWelcomeWindow.vue
@@ -4,8 +4,7 @@
 -->
 
 <template>
-	<NcModal :container="container"
-		:can-close="false"
+	<NcModal :can-close="false"
 		:close-on-click-outside="false"
 		:label-id="dialogHeaderId"
 		size="small">
@@ -100,10 +99,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		conversation() {
 			return this.$store.getters.conversation(this.token)
 		},
diff --git a/src/components/LeftSidebar/CallPhoneDialog/CallPhoneDialog.vue b/src/components/LeftSidebar/CallPhoneDialog/CallPhoneDialog.vue
index ac9adf11d39..46a353a4efa 100644
--- a/src/components/LeftSidebar/CallPhoneDialog/CallPhoneDialog.vue
+++ b/src/components/LeftSidebar/CallPhoneDialog/CallPhoneDialog.vue
@@ -7,7 +7,6 @@
 	<NcDialog :open="modal"
 		:name="t('spreed', 'Call a phone number')"
 		class="call-phone"
-		:container="container"
 		size="normal"
 		close-on-click-outside
 		@update:open="closeModal">
@@ -81,12 +80,6 @@ export default {
 		}
 	},
 
-	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-	},
-
 	expose: ['showModal'],
 
 	watch: {
diff --git a/src/components/LeftSidebar/ConversationsList/Conversation.vue b/src/components/LeftSidebar/ConversationsList/Conversation.vue
index 26e4af6a12c..a725986a719 100644
--- a/src/components/LeftSidebar/ConversationsList/Conversation.vue
+++ b/src/components/LeftSidebar/ConversationsList/Conversation.vue
@@ -98,8 +98,7 @@
 		<template v-if="isDialogOpen" #extra>
 			<NcDialog :open.sync="isDialogOpen"
 				:name="t('spreed','Delete conversation')"
-				:message="dialogMessage"
-				:container="container">
+				:message="dialogMessage">
 				<template #actions>
 					<NcButton type="tertiary" @click="isDialogOpen = false">
 						{{ t('spreed', 'No') }}
@@ -208,10 +207,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		canFavorite() {
 			return this.item.participantType !== PARTICIPANT.TYPE.USER_SELF_JOINED
 		},
diff --git a/src/components/LeftSidebar/InvitationHandler.vue b/src/components/LeftSidebar/InvitationHandler.vue
index 8cdbb68dd87..38d90566a24 100644
--- a/src/components/LeftSidebar/InvitationHandler.vue
+++ b/src/components/LeftSidebar/InvitationHandler.vue
@@ -6,7 +6,6 @@
 <template>
 	<NcDialog :open.sync="modal"
 		:name="t('spreed', 'Pending invitations')"
-		:container="container"
 		size="normal"
 		close-on-click-outside>
 		<div class="inbox">
@@ -115,10 +114,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		invitations() {
 			const invitations = {}
 			for (const id in this.federationStore.pendingShares) {
diff --git a/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue b/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
index f1a2eabc7a9..1aa7185594f 100644
--- a/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
+++ b/src/components/LeftSidebar/OpenConversationsList/OpenConversationsList.vue
@@ -5,7 +5,6 @@
 
 <template>
 	<RoomSelector v-if="modal"
-		:container="container"
 		list-open-conversations
 		show-postable-only
 		:dialog-title="dialogTitle"
@@ -39,10 +38,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		dialogTitle() {
 			return t('spreed', 'Join open conversations')
 		},
diff --git a/src/components/MediaSettings/MediaSettings.vue b/src/components/MediaSettings/MediaSettings.vue
index e9117b74f3e..659ae015791 100644
--- a/src/components/MediaSettings/MediaSettings.vue
+++ b/src/components/MediaSettings/MediaSettings.vue
@@ -5,7 +5,6 @@
 
 <template>
 	<NcModal v-if="modal"
-		:container="container"
 		:label-id="dialogHeaderId"
 		@close="closeModal">
 		<div class="media-settings">
@@ -331,10 +330,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		displayName() {
 			return this.$store.getters.getDisplayName()
 		},
diff --git a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageForwarder.vue b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageForwarder.vue
index d4fa2f7bb54..a7b361ce189 100644
--- a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageForwarder.vue
+++ b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageForwarder.vue
@@ -8,7 +8,6 @@
 		<!-- First step of the flow: selection of the room to which forward the
 		message to -->
 		<RoomSelector v-if="!showForwardedConfirmation"
-			:container="container"
 			show-postable-only
 			:dialog-title="dialogTitle"
 			:dialog-subtitle="dialogSubtitle"
@@ -20,7 +19,6 @@
 		message has been forwarded -->
 		<NcDialog v-else
 			:name="dialogTitle"
-			:container="container"
 			close-on-click-outside
 			@update:open="handleClose">
 			<NcEmptyContent :description="t('spreed', 'The message has been forwarded to {selectedConversationName}', { selectedConversationName })">
@@ -96,10 +94,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		dialogTitle() {
 			return t('spreed', 'Forward message')
 		},
diff --git a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageTranslateDialog.vue b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageTranslateDialog.vue
index a23275da93c..f642f3a9194 100644
--- a/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageTranslateDialog.vue
+++ b/src/components/MessagesList/MessagesGroup/Message/MessageButtonsBar/MessageTranslateDialog.vue
@@ -8,7 +8,6 @@
 		class="translate-dialog"
 		:name="t('spreed', 'Translate message')"
 		size="large"
-		:container="container"
 		close-on-click-outside
 		@update:open="$emit('close')">
 		<template v-if="isMounted" #default>
@@ -119,10 +118,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		userLanguage() {
 			return navigator.language.substring(0, 2)
 		},
diff --git a/src/components/MessagesList/MessagesGroup/Message/MessagePart/ReactionsList.vue b/src/components/MessagesList/MessagesGroup/Message/MessagePart/ReactionsList.vue
index c19bd441c0b..6ecfc0ce387 100644
--- a/src/components/MessagesList/MessagesGroup/Message/MessagePart/ReactionsList.vue
+++ b/src/components/MessagesList/MessagesGroup/Message/MessagePart/ReactionsList.vue
@@ -5,7 +5,6 @@
 
 <template>
 	<NcDialog :name="t('spreed', 'Reactions')"
-		:container="container"
 		close-on-click-outside
 		@update:open="closeModal">
 		<div class="reactions__modal">
@@ -104,10 +103,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		reactionsOverview() {
 			const mergedReactionsMap = {}
 			const modifiedDetailedReactions = {}
diff --git a/src/components/NewConversationDialog/NewConversationDialog.vue b/src/components/NewConversationDialog/NewConversationDialog.vue
index c9b89072651..3d9d880200e 100644
--- a/src/components/NewConversationDialog/NewConversationDialog.vue
+++ b/src/components/NewConversationDialog/NewConversationDialog.vue
@@ -9,7 +9,6 @@
 		<NcModal v-show="page !== 2"
 			class="new-group-conversation"
 			:close-on-click-outside="!isFilled"
-			:container="container"
 			:label-id="dialogHeaderPrepId"
 			@close="closeModal">
 			<h2 :id="dialogHeaderPrepId" class="new-group-conversation__header nc-dialog-alike-header">
@@ -69,7 +68,6 @@
 
 		<!-- Third page : this is the confirmation page-->
 		<NcModal v-if="page === 2"
-			:container="container"
 			:label-id="dialogHeaderResId"
 			@close="closeModal">
 			<NcEmptyContent>
@@ -192,10 +190,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		isPublic() {
 			return this.newConversation.type === CONVERSATION.TYPE.PUBLIC
 		},
diff --git a/src/components/NewMessage/NewMessagePollEditor.vue b/src/components/NewMessage/NewMessagePollEditor.vue
index de8c083313d..a65e22a10be 100644
--- a/src/components/NewMessage/NewMessagePollEditor.vue
+++ b/src/components/NewMessage/NewMessagePollEditor.vue
@@ -6,7 +6,6 @@
 <template>
 	<NcDialog :name="t('spreed', 'Create new poll')"
 		:close-on-click-outside="!isFilled"
-		:container="container"
 		v-on="$listeners"
 		@update:open="dismissEditor">
 		<!-- Poll Question -->
@@ -118,10 +117,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		isFilled() {
 			return !!this.pollQuestion || this.pollOptions.some(option => option)
 		},
diff --git a/src/components/PermissionsEditor/PermissionsEditor.vue b/src/components/PermissionsEditor/PermissionsEditor.vue
index 166064f763e..7001c3118a3 100644
--- a/src/components/PermissionsEditor/PermissionsEditor.vue
+++ b/src/components/PermissionsEditor/PermissionsEditor.vue
@@ -5,7 +5,7 @@
 
 <template>
 	<NcModal size="small"
-		:container="container"
+		:container="nestedContainer"
 		:label-id="dialogHeaderId"
 		v-on="$listeners">
 		<div class="wrapper">
@@ -101,7 +101,7 @@ export default {
 
 		nestedContainer: {
 			type: String,
-			default: null,
+			default: undefined,
 		},
 
 		/**
@@ -151,10 +151,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.nestedContainer ?? this.$store.getters.getMainContainerSelector()
-		},
-
 		modalTitle() {
 			if (this.displayName) {
 				return t('spreed', 'In this conversation <strong>{user}</strong> can:', {
diff --git a/src/components/PollViewer/PollViewer.vue b/src/components/PollViewer/PollViewer.vue
index e41b93c2618..5e978013c99 100644
--- a/src/components/PollViewer/PollViewer.vue
+++ b/src/components/PollViewer/PollViewer.vue
@@ -6,7 +6,6 @@
 <template>
 	<NcModal v-if="id"
 		size="small"
-		:container="container"
 		:label-id="dialogHeaderId"
 		@close="dismissModal">
 		<div v-if="poll" class="poll-modal">
@@ -161,10 +160,6 @@ export default {
 			return this.activePoll?.token
 		},
 
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		poll() {
 			return this.pollsStore.getPoll(this.token, this.id)
 		},
diff --git a/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue b/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue
index a767d03de9b..60509bae0f8 100644
--- a/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue
+++ b/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue
@@ -88,7 +88,6 @@
 
 		<!-- Participants editor -->
 		<NcModal v-if="showParticipantsEditor"
-			:container="container"
 			:label-id="dialogHeaderId"
 			@close="closeParticipantsEditor">
 			<div class="breakout-rooms-actions__editor">
@@ -216,10 +215,6 @@ export default {
 			return this.mainToken !== this.$store.getters.getToken()
 		},
 
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		manageBreakoutRoomsTitle() {
 			return t('spreed', 'Manage breakout rooms')
 		},
diff --git a/src/components/RightSidebar/Participants/Participant.vue b/src/components/RightSidebar/Participants/Participant.vue
index dc3e0cc30c5..32a55288d50 100644
--- a/src/components/RightSidebar/Participants/Participant.vue
+++ b/src/components/RightSidebar/Participants/Participant.vue
@@ -276,8 +276,7 @@
 			<!-- Confirmation required to remove participant -->
 			<NcDialog v-if="isRemoveDialogOpen"
 				:open.sync="isRemoveDialogOpen"
-				:name="removeParticipantLabel"
-				:container="container">
+				:name="removeParticipantLabel">
 				<p> {{ removeDialogMessage }} </p>
 				<template v-if="showBanOption">
 					<NcCheckboxRadioSwitch :checked.sync="isBanParticipant">
@@ -434,10 +433,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		participantNavigationId() {
 			return this.participant.actorType + '_' + this.participant.actorId
 		},
diff --git a/src/components/RightSidebar/SharedItems/SharedItemsBrowser.vue b/src/components/RightSidebar/SharedItems/SharedItemsBrowser.vue
index 31de45bc2ea..ae38bb6667f 100644
--- a/src/components/RightSidebar/SharedItems/SharedItemsBrowser.vue
+++ b/src/components/RightSidebar/SharedItems/SharedItemsBrowser.vue
@@ -5,7 +5,6 @@
 
 <template>
 	<NcModal size="large"
-		:container="container"
 		:label-id="dialogHeaderId"
 		v-on="$listeners">
 		<div class="shared-items-browser">
@@ -97,10 +96,6 @@ export default {
 		scroller() {
 			return this.$refs.scroller
 		},
-
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
 	},
 
 	watch: {
diff --git a/src/components/RoomSelector.vue b/src/components/RoomSelector.vue
index 0b88ac86026..aa1d4186e2c 100644
--- a/src/components/RoomSelector.vue
+++ b/src/components/RoomSelector.vue
@@ -5,7 +5,6 @@
 
 <template>
 	<NcDialog :name="dialogTitle"
-		:container="container"
 		close-on-click-outside
 		@update:open="close">
 		<template #default>
@@ -80,11 +79,6 @@ export default {
 	},
 
 	props: {
-		container: {
-			type: String,
-			default: undefined,
-		},
-
 		dialogTitle: {
 			type: String,
 			default: t('spreed', 'Link to a conversation'),
diff --git a/src/components/SettingsDialog/SettingsDialog.vue b/src/components/SettingsDialog/SettingsDialog.vue
index 387fb6c7974..b7c631d73e4 100644
--- a/src/components/SettingsDialog/SettingsDialog.vue
+++ b/src/components/SettingsDialog/SettingsDialog.vue
@@ -6,8 +6,7 @@
 <template>
 	<NcAppSettingsDialog :open.sync="showSettings"
 		:name="t('spreed', 'Talk settings')"
-		:show-navigation="true"
-		:container="container">
+		show-navigation>
 		<!-- Custom settings sections registered via OCA.Talk.Settings -->
 		<NcAppSettingsSection v-for="{ id, name, element } in customSettingsSections"
 			:id="id"
@@ -237,10 +236,6 @@ export default {
 	},
 
 	computed: {
-		container() {
-			return this.$store.getters.getMainContainerSelector()
-		},
-
 		playSounds() {
 			return this.$store.getters.playSounds
 		},