feat (blocksy): overridden default breakpoints will now reflect in the settings placeholders

Author: [email protected]

src/dynamic-breakpoints.php

@@ -48,6 +48,9 @@ function __construct() {
 				// 11 Priority, do this last because changing style can affect inline css optimization.
 				add_filter( 'render_block', array( $this, 'adjust_block_styles' ), 11, 2 );
 			}
+
+			// Add the default breakpoints to the localized settings.
+			add_filter( 'stackable_localize_settings_script', array( $this, 'add_default_breakpoints' ) );
 		}
 
 		/**
@@ -78,6 +81,21 @@ public function get_dynamic_breakpoints() {
 			return $this->dynamic_breakpoints;
 		}
 
+		/**
+		 * Add the default breakpoints to the localized settings to be used for
+		 * the default placeholder in the settings.
+		 *
+		 * @param Array $args
+		 * @return Array
+		 */
+		public function add_default_breakpoints( $args ) {
+			$args['defaultBreakpoints'] = apply_filters( 'stackable_responsive_breakpoints', array(
+				'tablet' => '1024',
+				'mobile' => '768',
+			) );
+			return $args;
+		}
+
 		/**
 		 * Register the setting where the user's dynamic breakpoints will be
 		 * saved.

src/welcome/admin.js

@@ -28,6 +28,7 @@ import {
 	showProNoticesOption,
 	isPro,
 	v2disabledBlocks,
+	defaultBreakpoints,
 } from 'stackable'
 import classnames from 'classnames'
 import { importBlocks } from '~stackable/util/admin'
@@ -838,7 +839,7 @@ const Responsiveness = props => {
 										},
 									} )
 								} }
-								placeholder="1024"
+								placeholder={ defaultBreakpoints.tablet || '1024' }
 							> px</AdminTextSetting>
 							<AdminTextSetting
 								label={ __( 'Mobile Breakpoint', i18n ) }
@@ -853,7 +854,7 @@ const Responsiveness = props => {
 										},
 									} )
 								} }
-								placeholder="768"
+								placeholder={ defaultBreakpoints.mobile || '768' }
 							> px</AdminTextSetting>
 						</div>
 					}