MC-3870: Background does not conform to border radius for banner, image, contained row & video

danmooney2 · danmooney2 · commit 354ba40a05e6 · 2018-09-26T11:32:20.000-05:00
Apply inheritance of border-radius styles within content type scopes
diff --git a/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/banner/_default.less b/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/banner/_default.less
@@ -88,3 +88,7 @@
         cursor: default;
     }
 }
+
+.pagebuilder-banner-wrapper {
+    border-radius: inherit;
+}
diff --git a/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/image/_default.less b/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/image/_default.less
@@ -32,5 +32,9 @@
             padding: 15px 0;
             word-wrap: break-word;
         }
+
+        .preview-image {
+            border-radius: inherit;
+        }
     }
 }
diff --git a/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/video/_default.less b/app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/video/_default.less
@@ -29,6 +29,8 @@
             flex-grow: 1;
         }
         .pagebuilder-video-container {
+            border-radius: inherit;
+            overflow: hidden;
             padding-top: 56.25%;
             position: relative;
         }
diff --git a/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/banner/_default.less b/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/banner/_default.less
@@ -24,6 +24,7 @@
 }
 
 .pagebuilder-banner-wrapper {
+    border-radius: inherit;
     box-sizing: border-box;
     word-break: break-all;
 
diff --git a/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/image/_default.less b/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/image/_default.less
@@ -11,6 +11,10 @@ figure[data-role='image'] {
     box-sizing: border-box;
     width: 100%;
 
+    > [data-element='link'] {
+        border-radius: inherit;
+    }
+
     figcaption {
         word-wrap: break-word;
     }
diff --git a/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/video/_default.less b/app/code/Magento/PageBuilder/view/frontend/web/css/source/content-type/video/_default.less
@@ -17,6 +17,8 @@ div[data-role='video'] {
         height: 100%;
     }
     .pagebuilder-video-container {
+        border-radius: inherit;
+        overflow: hidden;
         padding-top: 56.25%;
         position: relative;
     }

Original file line number	Diff line number	Diff line change
`@@ -88,3 +88,7 @@`
`88`	`88`	`cursor: default;`
`89`	`89`	`}`
`90`	`90`	`}`
	`91`	`+`
	`92`	`+.pagebuilder-banner-wrapper {`
	`93`	`+ border-radius: inherit;`
	`94`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -32,5 +32,9 @@`
`32`	`32`	`padding: 15px 0;`
`33`	`33`	`word-wrap: break-word;`
`34`	`34`	`}`
	`35`	`+`
	`36`	`+ .preview-image {`
	`37`	`+ border-radius: inherit;`
	`38`	`+ }`
`35`	`39`	`}`
`36`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,8 @@`
`29`	`29`	`flex-grow: 1;`
`30`	`30`	`}`
`31`	`31`	`.pagebuilder-video-container {`
	`32`	`+ border-radius: inherit;`
	`33`	`+ overflow: hidden;`
`32`	`34`	`padding-top: 56.25%;`
`33`	`35`	`position: relative;`
`34`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`.pagebuilder-banner-wrapper {`
	`27`	`+ border-radius: inherit;`
`27`	`28`	`box-sizing: border-box;`
`28`	`29`	`word-break: break-all;`
`29`	`30`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,8 @@ div[data-role='video'] {`
`17`	`17`	`height: 100%;`
`18`	`18`	`}`
`19`	`19`	`.pagebuilder-video-container {`
	`20`	`+ border-radius: inherit;`
	`21`	`+ overflow: hidden;`
`20`	`22`	`padding-top: 56.25%;`
`21`	`23`	`position: relative;`
`22`	`24`	`}`