pkp/pkp-lib#10444 Add modalStyle to components that use the openDialo…

…g api
blesildaramirez · Oct 9, 2024 · 96f6396 · 96f6396
1 parent 286fd40
commit 96f6396
Show file tree

Hide file tree

Showing 11 changed files with 66 additions and 52 deletions.
diff --git a/src/components/Container/WorkflowPage.vue b/src/components/Container/WorkflowPage.vue
@@ -341,7 +341,7 @@ export default {
 				actions: [
 					{
 						label: this.t('common.yes'),
-						isWarnable: true,
+						isPrimary: true,
 						callback: (close) => {
 							close();
 							this.createVersion();
@@ -448,7 +448,7 @@ export default {
 							this.workingPublication.status === pkp.const.STATUS_SCHEDULED
 								? this.unscheduleLabel
 								: this.unpublishLabel,
-						isPrimary: true,
+						isWarnable: true,
 						callback: (close) => {
 							this.unpublish(this.workingPublication);
 							close();
@@ -459,6 +459,7 @@ export default {
 						callback: (close) => close(),
 					},
 				],
+				modalStyle: 'negative',
 			});
 		},
 

diff --git a/src/components/Form/fields/FieldOrcid.vue b/src/components/Form/fields/FieldOrcid.vue
@@ -190,6 +190,7 @@ export default {
 						},
 					},
 				],
+				modalStyle: 'primary',
 				close: () => {},
 			});
 		},
@@ -227,20 +228,20 @@ export default {
 				actions: [
 					{
 						label: this.t('common.yes'),
-						isPrimary: true,
+						isWarnable: true,
 						callback: async (close) => {
 							await this.deleteOrcid();
 							close();
 						},
 					},
 					{
 						label: this.t('common.no'),
-						isWarnable: true,
 						callback: (close) => {
 							close();
 						},
 					},
 				],
+				modalStyle: 'negative',
 				close: () => {},
 			});
 		},

diff --git a/src/components/ListPanel/doi/DoiListItem.vue b/src/components/ListPanel/doi/DoiListItem.vue
@@ -513,12 +513,12 @@ export default {
 				actions: [
 					{
 						label: this.t('common.ok'),
-						isPrimary: true,
 						callback: (close) => {
 							close();
 						},
 					},
 				],
+				modalStyle: 'negative',
 			});
 		},
 		openVersionModal() {
@@ -814,6 +814,7 @@ export default {
 						},
 					},
 				],
+				modalStyle: 'primary',
 			});
 		},
 		triggerDeposit() {

diff --git a/src/components/ListPanel/doi/DoiListPanel.vue b/src/components/ListPanel/doi/DoiListPanel.vue
@@ -375,13 +375,13 @@ export default {
 					actions: [
 						{
 							label: this.t('common.ok'),
-							isPrimary: true,
 							callback: (close) => {
 								this.failedDoiActions = [];
 								close();
 							},
 						},
 					],
+					modalStyle: 'negative',
 					bodyComponent: DoiFailedActionDialogBody,
 					bodyProps: {
 						failedDoiActions: this.failedDoiActions,

diff --git a/src/managers/FileManager/useFileManagerActions.js b/src/managers/FileManager/useFileManagerActions.js
@@ -111,15 +111,9 @@ export function useFileManagerActions() {
 		const {openDialog, openDialogNetworkError} = useModal();
 		openDialog({
 			actions: [
-				{
-					label: t('common.cancel'),
-					isWarnable: true,
-					callback: (close) => {
-						close();
-					},
-				},
 				{
 					label: t('common.ok'),
+					isWarnable: true,
 					callback: async (close) => {
 						const {url} = useLegacyGridUrl({
 							component: 'api.file.ManageFileApiHandler',
@@ -146,9 +140,16 @@ export function useFileManagerActions() {
 						}
 					},
 				},
+				{
+					label: t('common.cancel'),
+					callback: (close) => {
+						close();
+					},
+				},
 			],
 			title: t('common.delete'),
 			message: t('common.confirmDelete'),
+			modalStyle: 'negative',
 		});
 	}
 

diff --git a/src/managers/GalleyManager/useGalleyManagerActions.js b/src/managers/GalleyManager/useGalleyManagerActions.js
@@ -139,16 +139,9 @@ export function useGalleyManagerActions() {
 		const {openDialog, openDialogNetworkError} = useModal();
 		openDialog({
 			actions: [
-				{
-					label: t('common.cancel'),
-					isWarnable: true,
-					callback: (close) => {
-						close();
-						finishedCallback();
-					},
-				},
 				{
 					label: t('common.ok'),
+					isWarnable: true,
 					callback: async (close) => {
 						// http://localhost:7002/index.php/publicknowledge/$$$call$$$/grid/article-galleys/article-galley-grid/delete-galley
 						// ?submissionId=17&publicationId=22&representationId=9
@@ -177,9 +170,17 @@ export function useGalleyManagerActions() {
 						}
 					},
 				},
+				{
+					label: t('common.cancel'),
+					callback: (close) => {
+						close();
+						finishedCallback();
+					},
+				},
 			],
 			title: t('common.delete'),
 			message: t('common.confirmDelete'),
+			modalStyle: 'negative',
 		});
 	}
 

diff --git a/src/managers/ParticipantManager/useParticipantManagerActions.js b/src/managers/ParticipantManager/useParticipantManagerActions.js
@@ -40,15 +40,9 @@ export function useParticipantManagerActions() {
 
 		openDialog({
 			actions: [
-				{
-					label: t('common.cancel'),
-					isWarnable: true,
-					callback: (close) => {
-						close();
-					},
-				},
 				{
 					label: t('common.ok'),
+					isWarnable: true,
 					callback: async (close) => {
 						// http://localhost:7002/index.php/publicknowledge/
 						// $$$call$$$/grid/users/stage-participant/stage-participant-grid/delete-participant?submissionId=13&stageId=3&assignmentId=62
@@ -80,9 +74,16 @@ export function useParticipantManagerActions() {
 						}
 					},
 				},
+				{
+					label: t('common.cancel'),
+					callback: (close) => {
+						close();
+					},
+				},
 			],
 			title: t('editor.submission.removeStageParticipant'),
 			message: t('editor.submission.removeStageParticipant.description'),
+			modalStyle: 'negative',
 		});
 	}
 
@@ -124,13 +125,6 @@ export function useParticipantManagerActions() {
 
 		openDialog({
 			actions: [
-				{
-					label: t('common.cancel'),
-					isWarnable: true,
-					callback: (close) => {
-						close();
-					},
-				},
 				{
 					label: t('common.ok'),
 					callback: (close) => {
@@ -152,9 +146,17 @@ export function useParticipantManagerActions() {
 						redirectToPage();
 					},
 				},
+				{
+					label: t('common.cancel'),
+					isWarnable: true,
+					callback: (close) => {
+						close();
+					},
+				},
 			],
 			title: t('grid.action.logInAs'),
 			message: t('grid.user.confirmLogInAs'),
+			modalStyle: 'primary',
 		});
 	}
 

diff --git a/src/managers/ReviewerManager/useReviewerManagerActions.js b/src/managers/ReviewerManager/useReviewerManagerActions.js
@@ -430,13 +430,6 @@ export function useReviewerManagerActions() {
 		openDialog(
 			{
 				actions: [
-					{
-						label: t('common.cancel'),
-						isWarnable: true,
-						callback: (close) => {
-							close();
-						},
-					},
 					{
 						label: t('common.ok'),
 						callback: (close) => {
@@ -446,9 +439,17 @@ export function useReviewerManagerActions() {
 							redirectToPage();
 						},
 					},
+					{
+						label: t('common.cancel'),
+						isWarnable: true,
+						callback: (close) => {
+							close();
+						},
+					},
 				],
 				title: t('grid.action.logInAs'),
 				message: t('grid.user.confirmLogInAs'),
+				modalStyle: 'primary',
 			},
 			finishedCallback,
 		);
@@ -500,15 +501,9 @@ export function useReviewerManagerActions() {
 		openDialog(
 			{
 				actions: [
-					{
-						label: t('common.cancel'),
-						isWarnable: true,
-						callback: (close) => {
-							close();
-						},
-					},
 					{
 						label: t('common.ok'),
+						isWarnable: true,
 						callback: async (close) => {
 							// http://localhost:7002/index.php/publicknowledge/$$$call$$$/grid/users/reviewer/reviewer-grid/unconsider-review?submissionId=12&reviewAssignmentId=17&stageId=3
 
@@ -537,9 +532,16 @@ export function useReviewerManagerActions() {
 							}
 						},
 					},
+					{
+						label: t('common.cancel'),
+						callback: (close) => {
+							close();
+						},
+					},
 				],
 				title: t('editor.review.unconsiderReview'),
 				message: t('editor.review.unconsiderReviewText'),
+				modalStyle: 'negative',
 			},
 			finishedCallback,
 		);

diff --git a/src/pages/acceptInvitation/AcceptInvitationPageStore.js b/src/pages/acceptInvitation/AcceptInvitationPageStore.js
@@ -342,6 +342,7 @@ export const useAcceptInvitationPageStore = defineComponentStore(
 								},
 							},
 						],
+						modalStyle: 'success',
 					});
 				}
 			}

diff --git a/src/pages/userInvitation/UserInvitationPageStore.js b/src/pages/userInvitation/UserInvitationPageStore.js
@@ -261,6 +261,7 @@ export const useUserInvitationPageStore = defineComponentStore(
 								},
 							},
 						],
+						modalStyle: 'primary',
 					});
 				}
 			}

diff --git a/src/pages/workflow/composables/useWorkflowActions.js b/src/pages/workflow/composables/useWorkflowActions.js
@@ -202,7 +202,7 @@ export function useWorkflowActions({
 			actions: [
 				{
 					label: t('publication.unschedule'),
-					isPrimary: true,
+					isWarnable: true,
 					callback: async (close) => {
 						const {apiUrl: unschedulePublicationApiUrl} = useUrl(
 							`submissions/${submission.id}/publications/${selectedPublication.id}/unpublish`,
@@ -220,6 +220,7 @@ export function useWorkflowActions({
 					callback: (close) => close(),
 				},
 			],
+			modalStyle: 'negative',
 		});
 	}
 
@@ -234,7 +235,7 @@ export function useWorkflowActions({
 			actions: [
 				{
 					label: t('publication.unpublish'),
-					isPrimary: true,
+					isWarnable: true,
 					callback: async (close) => {
 						const {apiUrl: unschedulePublicationApiUrl} = useUrl(
 							`submissions/${submission.id}/publications/${selectedPublication.id}/unpublish`,
@@ -252,6 +253,7 @@ export function useWorkflowActions({
 					callback: (close) => close(),
 				},
 			],
+			modalStyle: 'negative',
 		});
 	}
 
@@ -265,7 +267,7 @@ export function useWorkflowActions({
 			actions: [
 				{
 					label: t('common.yes'),
-					isWarnable: true,
+					isPrimary: true,
 					callback: async (close) => {
 						close();
 
@@ -291,6 +293,7 @@ export function useWorkflowActions({
 					callback: (close) => close(),
 				},
 			],
+			modalStyle: 'primary',
 		});
 	}