#559: User sees no predefined gutters - Removed default margin and padding for Row

Author: hannahnida

app/code/Magento/PageBuilder/Test/Mftf/Data/AdvancedData.xml

@@ -460,10 +460,20 @@
         <data key="name">Margins and Padding</data>
         <data key="section">advanced</data>
         <data key="fieldName">margins_and_padding</data>
-        <data key="value">0 0 10</data>
+        <data key="value">0</data>
         <data key="marginTop">0</data>
         <data key="marginRight">0</data>
-        <data key="marginBottom">10</data>
+        <data key="marginBottom">0</data>
+        <data key="marginLeft">0</data>
+    </entity>
+    <entity name="PageBuilderMarginsPropertyRowStage" type="pagebuilder_margins_property">
+        <data key="name">Margins and Padding</data>
+        <data key="section">advanced</data>
+        <data key="fieldName">margins_and_padding</data>
+        <data key="value">0 0 1</data>
+        <data key="marginTop">0</data>
+        <data key="marginRight">0</data>
+        <data key="marginBottom">1</data>
         <data key="marginLeft">0</data>
     </entity>
     <entity name="PageBuilderMarginsPropertyColumnDefault" type="pagebuilder_margins_property">

app/code/Magento/PageBuilder/Test/Mftf/Test/AdminPageBuilderRowTest/RowValidateAdvancedConfigurationDefaultValuesTest.xml

@@ -34,6 +34,10 @@
         <actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage">
             <argument name="contentType" value="PageBuilderRowContentType"/>
         </actionGroup>
+        <!-- Add Min-height Value -->
+        <actionGroup ref="fillSlideOutPanelFieldGeneral" stepKey="enterMinimumHeightProperty">
+            <argument name="property" value="PageBuilderMinHeightProperty_300px"/>
+        </actionGroup>
         <!-- Validate Default Values -->
         <comment userInput="Validate Default Values" stepKey="CommentValidateDefaultValues"/>
         <actionGroup ref="seeOptionIsSelectedSlideOutProperty" stepKey="validateAlignment">
@@ -57,12 +61,12 @@
             <actualResult type="variable">getCssClasses</actualResult>
         </assertEmpty>
         <actionGroup ref="seeInMarginFieldsSlideOutPanel" stepKey="validateMargins">
-            <argument name="property" value="PageBuilderMarginsPropertyRowDefault"/>
+            <argument name="property" value="PageBuilderMarginsProperty_Empty"/>
         </actionGroup>
         <actionGroup ref="seeInPaddingFieldsSlideOutPanel" stepKey="validatePadding">
-            <argument name="property" value="PageBuilderPaddingProperty10"/>
+            <argument name="property" value="PageBuilderPaddingProperty_Empty"/>
         </actionGroup>
-        <actionGroup ref="closeEditPanelSettings" stepKey="closeEditPanelSettings"/>
+        <actionGroup ref="saveEditPanelSettings" stepKey="closeEditPanelSettings"/>
         <!-- Validate Stage -->
         <comment userInput="Validate Stage" stepKey="commentValidateStage"/>
         <actionGroup ref="ValidateAdvancedConfigurationWithBorderColorActionGroup" stepKey="validateStage">
@@ -72,12 +76,12 @@
             <argument name="borderColor" value="PageBuilderAdvancedBorderColorTransparent"/>
             <argument name="borderRadius" value="PageBuilderAdvancedBorderRadiusDefaultProperty"/>
             <argument name="margin" value="PageBuilderMarginsPropertyDefault"/>
-            <argument name="padding" value="PageBuilderPaddingProperty10"/>
+            <argument name="padding" value="PageBuilderPaddingPropertyDefault"/>
             <argument name="index" value="1"/>
         </actionGroup>
         <actionGroup ref="ValidateContentTypesStyleEqualsExpectedStyleActionGroup" stepKey="validateMarginStage">
             <argument name="selector" value="{{RowOnStage.affordance('1')}}"/>
-            <argument name="expectedProperty" value="{{PageBuilderMarginsPropertyRowDefault.value}}"/>
+            <argument name="expectedProperty" value="{{PageBuilderMarginsPropertyRowStage.value}}"/>
             <argument name="cssProperty" value="margin"/>
         </actionGroup>
         <actionGroup ref="exitPageBuilderFullScreen" stepKey="exitPageBuilderFullScreen"/>
@@ -92,12 +96,12 @@
             <argument name="borderWidth" value="PageBuilderAdvancedBorderWidthProperty0"/>
             <argument name="borderRadius" value="PageBuilderAdvancedBorderRadiusDefaultProperty"/>
             <argument name="margin" value="PageBuilderMarginsPropertyDefault"/>
-            <argument name="padding" value="PageBuilderPaddingProperty10"/>
+            <argument name="padding" value="PageBuilderPaddingPropertyDefault"/>
             <argument name="index" value="1"/>
         </actionGroup>
         <actionGroup ref="ValidateContentTypesStyleEqualsExpectedStyleActionGroup" stepKey="validateMarginStageAfterSave">
             <argument name="selector" value="{{RowOnStage.affordance('1')}}"/>
-            <argument name="expectedProperty" value="{{PageBuilderMarginsPropertyRowDefault.value}}"/>
+            <argument name="expectedProperty" value="{{PageBuilderMarginsPropertyRowStage.value}}"/>
             <argument name="cssProperty" value="margin"/>
         </actionGroup>
         <actionGroup ref="openPageBuilderEditPanel" stepKey="openEditMenuOnStage2">
@@ -126,10 +130,10 @@
             <actualResult type="variable">getCssClasses2</actualResult>
         </assertEmpty>
         <actionGroup ref="seeInMarginFieldsSlideOutPanel" stepKey="validateMargins2">
-            <argument name="property" value="PageBuilderMarginsPropertyRowDefault"/>
+            <argument name="property" value="PageBuilderMarginsProperty_Empty"/>
         </actionGroup>
         <actionGroup ref="seeInPaddingFieldsSlideOutPanel" stepKey="validatePadding2">
-            <argument name="property" value="PageBuilderPaddingProperty10"/>
+            <argument name="property" value="PageBuilderPaddingProperty_Empty"/>
         </actionGroup>
         <actionGroup ref="navigateToStoreFront" stepKey="navigateToStoreFront">
             <argument name="contentType" value="PageBuilderRowContentType"/>
@@ -142,8 +146,8 @@
             <argument name="border" value="PageBuilderAdvancedBorderPropertyNone"/>
             <argument name="borderWidth" value="PageBuilderAdvancedBorderWidthProperty0"/>
             <argument name="borderRadius" value="PageBuilderAdvancedBorderRadiusDefaultProperty"/>
-            <argument name="margin" value="PageBuilderMarginsPropertyRowDefault"/>
-            <argument name="padding" value="PageBuilderPaddingProperty10"/>
+            <argument name="margin" value="PageBuilderMarginsPropertyDefault"/>
+            <argument name="padding" value="PageBuilderPaddingPropertyDefault"/>
             <argument name="index" value="1"/>
         </actionGroup>
     </test>

app/code/Magento/PageBuilder/view/adminhtml/ui_component/pagebuilder_row_form.xml

@@ -175,7 +175,7 @@
         <field name="margins_and_padding">
             <argument name="data" xsi:type="array">
                 <item name="config" xsi:type="array">
-                    <item name="default" xsi:type="string">{"margin":{"bottom":"10","left":"0","right":"0","top":"0"},"padding":{"bottom":"10","left":"10","right":"10","top":"10"}}</item>
+                    <item name="default" xsi:type="string">{"margin":{"bottom":null,"left":null,"right":null,"top":null},"padding":{"bottom":null,"left":null,"right":null,"top":null}}</item>
                 </item>
             </argument>
         </field>