From cf69d9ca3f083ba204e2adf0deb8c1345dfee6f4 Mon Sep 17 00:00:00 2001
From: Eduardo Morales <emoral435@gmail.com>
Date: Wed, 13 Mar 2024 11:58:37 -0500
Subject: [PATCH 1/3] fix: migrate conflictpicker to NcDialog and remove
 incorrect semantic closing icon

Signed-off-by: Eduardo Morales <emoral435@gmail.com>
---
 lib/components/ConflictPicker.vue | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/lib/components/ConflictPicker.vue b/lib/components/ConflictPicker.vue
index 9006ca98..0300e51f 100644
--- a/lib/components/ConflictPicker.vue
+++ b/lib/components/ConflictPicker.vue
@@ -1,7 +1,8 @@
 <template>
-	<NcModal class="conflict-picker"
+	<NcDialog class="conflict-picker"
 		data-cy-conflict-picker
 		:close-on-click-outside="false"
+		:canClose="false"
 		:show="opened"
 		size="large"
 		@close="onCancel">
@@ -67,7 +68,7 @@
 				{{ t('Continue') }}
 			</NcButton>
 		</div>
-	</NcModal>
+	</NcDialog>
 </template>
 
 <script lang="ts">
@@ -81,7 +82,7 @@ import Vue from 'vue'
 
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/dist/Components/NcCheckboxRadioSwitch.js'
-import NcModal from '@nextcloud/vue/dist/Components/NcModal.js'
+import NcDialog from '@nextcloud/vue/dist/Components/NcDialog.js'
 import ArrowRight from 'vue-material-design-icons/ArrowRight.vue'
 import Close from 'vue-material-design-icons/Close.vue'
 
@@ -99,7 +100,7 @@ export default Vue.extend({
 		Close,
 		NcButton,
 		NcCheckboxRadioSwitch,
-		NcModal,
+		NcDialog,
 		NodesPicker,
 	},
 
@@ -372,7 +373,7 @@ export default Vue.extend({
 		position: sticky;
 		z-index: 10;
 		top: 0;
-		padding: var(--margin);
+		padding: 0 var(--margin);
 		padding-bottom: 0;
 	}
 

From 1a7891103fe3cb68793935edf6d0597b19a24d5f Mon Sep 17 00:00:00 2001
From: Eduardo Morales <emoral435@gmail.com>
Date: Thu, 14 Mar 2024 08:10:31 -0500
Subject: [PATCH 2/3] fix: added name to ConflictPicker dialog

Signed-off-by: Eduardo Morales <emoral435@gmail.com>
---
 lib/components/ConflictPicker.vue | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/components/ConflictPicker.vue b/lib/components/ConflictPicker.vue
index 0300e51f..21e83e11 100644
--- a/lib/components/ConflictPicker.vue
+++ b/lib/components/ConflictPicker.vue
@@ -2,14 +2,13 @@
 	<NcDialog class="conflict-picker"
 		data-cy-conflict-picker
 		:close-on-click-outside="false"
-		:canClose="false"
+		:can-close="false"
 		:show="opened"
+		:name="name"
 		size="large"
 		@close="onCancel">
 		<!-- Header -->
 		<div class="conflict-picker__header">
-			<h2 class="conflict-picker__title" v-text="name" />
-
 			<!-- Description -->
 			<p id="conflict-picker-description" class="conflict-picker__description">
 				{{ t('Which files do you want to keep?') }}<br>

From 87da200371b89a20c03e2d899748c469505f7569 Mon Sep 17 00:00:00 2001
From: Eduardo Morales <emoral435@gmail.com>
Date: Fri, 15 Mar 2024 08:29:56 -0500
Subject: [PATCH 3/3] fix(cypress): updated cypress tests to new ConflictPicker
 dialog

Signed-off-by: Eduardo Morales <emoral435@gmail.com>
---
 cypress/components/ConflictPicker.cy.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cypress/components/ConflictPicker.cy.ts b/cypress/components/ConflictPicker.cy.ts
index aa18816c..bda30f12 100644
--- a/cypress/components/ConflictPicker.cy.ts
+++ b/cypress/components/ConflictPicker.cy.ts
@@ -36,11 +36,11 @@ describe('ConflictPicker rendering', { testIsolation: true }, () => {
 		cy.get('[data-cy-conflict-picker-fieldset="all"]').should('exist')
 		cy.get('[data-cy-conflict-picker-fieldset="image.jpg"]').should('exist')
 
-		cy.get('[data-cy-conflict-picker-skip]').should('be.visible')
+		cy.get('[data-cy-conflict-picker-skip]').scrollIntoView().should('be.visible')
 		cy.get('[data-cy-conflict-picker-submit]').should('be.visible')
 
 		// Force close and cancel
-		cy.get('[data-cy-conflict-picker] .modal-container__close').click({ force: true })
+		cy.get('[data-cy-conflict-picker-skip]').click({ force: true })
 	})
 })