Launchpad: Refactor launchpad save modal (#29134)

Automattic · Feb 27, 2023 · 752b1c9 · 752b1c9
1 parent 5bceb98
commit 752b1c9
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/projects/plugins/jetpack/changelog/update-launchpad-save-modal-selector-logic b/projects/plugins/jetpack/changelog/update-launchpad-save-modal-selector-logic
@@ -0,0 +1,4 @@
+Significance: minor
+Type: other
+
+Consolidates selector logic in the launchpad save modal
diff --git a/projects/plugins/jetpack/extensions/plugins/launchpad-save-modal/index.js b/projects/plugins/jetpack/extensions/plugins/launchpad-save-modal/index.js
@@ -12,15 +12,13 @@ export const name = 'launchpad-save-modal';
 
 export const settings = {
 	render: function LaunchpadSaveModal() {
-		const isSavingSite = useSelect(
-			select => select( editorStore ).isSavingNonPostEntityChanges(),
-			[]
-		);
-		const isSavingPost = useSelect( select => select( editorStore ).isSavingPost(), [] );
-		const isPublishingPost = useSelect( select => select( editorStore ).isPublishingPost(), [] );
-		const isCurrentPostPublished = useSelect(
-			select => select( editorStore ).isCurrentPostPublished(),
-			[]
+		const { isSavingSite, isSavingPost, isPublishingPost, isCurrentPostPublished } = useSelect(
+			selector => ( {
+				isSavingSite: selector( editorStore ).isSavingNonPostEntityChanges(),
+				isSavingPost: selector( editorStore ).isSavingPost(),
+				isPublishingPost: selector( editorStore ).isPublishingPost(),
+				isCurrentPostPublished: selector( editorStore ).isCurrentPostPublished(),
+			} )
 		);
 
 		const prevIsSavingSite = usePrevious( isSavingSite );