Move "No Behaviors" to be the first option in the list of behaviors. (#50979)

* Move `No Behaviors` to be the first option

* Forgot the array spread

Author: michalczaplinski

packages/block-editor/src/hooks/behaviors.js

@@ -56,6 +56,23 @@ export const withBehaviors = createHigherOrderComponent( ( BlockEdit ) => {
 		// Block behaviors take precedence over theme behaviors.
 		const behaviors = merge( themeBehaviors, blockBehaviors || {} );
 
+		const noBehaviorsOption = {
+			value: '',
+			label: __( 'No behaviors' ),
+		};
+
+		const behaviorsOptions = Object.entries( settings )
+			.filter( ( [ , behaviorValue ] ) => behaviorValue ) // Filter out behaviors that are disabled.
+			.map( ( [ behaviorName ] ) => ( {
+				value: behaviorName,
+				label:
+					// Capitalize the first letter of the behavior name.
+					behaviorName[ 0 ].toUpperCase() +
+					behaviorName.slice( 1 ).toLowerCase(),
+			} ) );
+
+		const options = [ noBehaviorsOption, ...behaviorsOptions ];
+
 		return (
 			<>
 				<BlockEdit { ...props } />
@@ -65,19 +82,7 @@ export const withBehaviors = createHigherOrderComponent( ( BlockEdit ) => {
 						label={ __( 'Behaviors' ) }
 						// At the moment we are only supporting one behavior (Lightbox)
 						value={ behaviors?.lightbox ? 'lightbox' : '' }
-						options={ Object.entries( settings )
-							.filter( ( [ , behaviorValue ] ) => behaviorValue ) // Filter out behaviors that are disabled.
-							.map( ( [ behaviorName ] ) => ( {
-								value: behaviorName,
-								label:
-									// Capitalize the first letter of the behavior name.
-									behaviorName[ 0 ].toUpperCase() +
-									behaviorName.slice( 1 ).toLowerCase(),
-							} ) )
-							.concat( {
-								value: '',
-								label: __( 'No behaviors' ),
-							} ) }
+						options={ options }
 						onChange={ ( nextValue ) => {
 							// If the user selects something, it means that they want to
 							// change the default value (true) so we save it in the attributes.