Skip to content

Commit d3e63af

Browse files
committed
MC-3406: Explicit Background Attributes for Tab
- fix merge issues & stabilize tests
1 parent 86201e3 commit d3e63af

File tree

3 files changed

+11
-13
lines changed

3 files changed

+11
-13
lines changed

app/code/Magento/PageBuilder/Test/Mftf/Section/PageBuilderTabsSection.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -86,11 +86,6 @@
8686
<element name="tabHeaderItemActive" type="button" selector="(//div[@data-role='tabs']//descendant::*[@role='tablist'])/li[{{arg1}}][contains(@class, 'ui-state-active')]" parameterized="true"/>
8787
<element name="tabPanel" type="button" selector="(//div[@class='tabs-content']//div[@role='tabpanel'])[{{arg1}}]" parameterized="true"/>
8888
<element name="tabContent" type="button" selector="div.tabs-content div[data-role=tab-item]:nth-child({{arg1}})" parameterized="true"/>
89-
<element name="tabContentBackgroundColor" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style, 'background-color: {{arg2}}')]" parameterized="true"/>
90-
<element name="tabContentBackgroundAttachment" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style, 'background-attachment: {{arg2}}')]" parameterized="true"/>
91-
<element name="tabContentBackgroundPosition" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style, 'background-position: {{arg2}}')]" parameterized="true"/>
92-
<element name="tabContentBackgroundSize" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style, 'background-size: {{arg2}}')]" parameterized="true"/>
93-
<element name="tabContentBackgroundRepeat" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style, 'background-repeat: {{arg2}}')]" parameterized="true"/>
9489
<element name="container" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}]" parameterized="true"/>
9590
<element name="containerStyle" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][contains(@style,'{{arg2}}')]" parameterized="true"/>
9691
<element name="noMinHeight" type="button" selector="(//div[@class='tabs-content']//div[@data-role='tab-item'])[{{arg1}}][not(contains(@style,'min-height:'))]" parameterized="true"/>

app/code/Magento/PageBuilder/Test/Mftf/Test/AdminPageBuilderBlockTest.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -885,7 +885,7 @@
885885
<!--Check that tab was rendered on the stage-->
886886
<comment userInput="Check that tab was rendered on the stage" stepKey="commentCheckingThatTabAreRenderedOnTHeStage"/>
887887
<see selector="{{TabOnStorefront.tabHeader('1')}}" userInput="{{PageBuilderTabNameRandom1.value}}" stepKey="seeTab1TitleStorefront2"/>
888-
<seeElement selector="{{TabOnStorefront.tabContentBackgroundColor('1', PageBuilderTabBackgroundColorPink.rgb)}}" stepKey="seeTab1BackgroundColor2"/>
888+
<seeElement selector="{{TabOnStorefront.backgroundColor('1', PageBuilderTabBackgroundColorPink.rgb)}}" stepKey="seeTab1BackgroundColor2"/>
889889
<actionGroup ref="openOptionsMenu" stepKey="openEditMenuOnStage3">
890890
<argument name="contentType" value="PageBuilderBlockContentType"/>
891891
<argument name="contentTypeIndex" value="1"/>
@@ -896,7 +896,7 @@
896896
<comment userInput="Verify Block On Store front" stepKey="commentCheckingThatImageIsNotRenderedOnTheStoreFront"/>
897897
<amOnPage url="$$createCMSPage.identifier$$" stepKey="amOnPageTestPage"/>
898898
<see selector="{{TabOnStorefront.tabHeader('1')}}" userInput="{{PageBuilderTabNameRandom1.value}}" stepKey="seeTab1TitleStorefront"/>
899-
<seeElement selector="{{TabOnStorefront.tabContentBackgroundColor('1', PageBuilderTabBackgroundColorPink.rgb)}}" stepKey="seeTab1BackgroundColor"/>
899+
<seeElement selector="{{TabOnStorefront.backgroundColor('1', PageBuilderTabBackgroundColorPink.rgb)}}" stepKey="seeTab1BackgroundColor"/>
900900
<see userInput="$$createPreReqBlock.content$$" stepKey="seeBlockContentOnCmsStage1"/>
901901
</test>
902902
<test name="AddBlockWithEmptyImageToCmsPage">

app/code/Magento/PageBuilder/view/adminhtml/web/css/source/content-type/slider/_default.less

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,17 +3,20 @@
33
// * See COPYING.txt for license details.
44
// */
55

6+
@ccc: #ccc;
7+
@b3: #b3b3b3;
8+
69
//
710
// Default appearance styles
811
// _____________________________________________
912

1013
[data-role='slider'],
1114
.pagebuilder-content-type.pagebuilder-slider {
12-
border: none;
13-
transition: none;
1415
-moz-transition: none;
1516
-o-transition: none;
1617
-webkit-transition: none;
18+
border: none;
19+
transition: none;
1720

1821
.slider-container {
1922
.pagebuilder-system-border;
@@ -37,11 +40,11 @@
3740
}
3841

3942
&:hover {
40-
box-shadow: none;
41-
transition: none;
4243
-moz-transition: none;
4344
-o-transition: none;
4445
-webkit-transition: none;
46+
box-shadow: none;
47+
transition: none;
4548

4649
.slider-container {
4750
.pagebuilder-system-border-hover;
@@ -348,7 +351,7 @@
348351
width: 14px;
349352

350353
&:hover {
351-
border: 1px solid #b3b3b3;
354+
border: 1px solid @b3;
352355
}
353356

354357
&:active,
@@ -361,7 +364,7 @@
361364

362365
&.slick-active {
363366
button {
364-
background: #1c1918;
367+
background: @color-pagebuilder-darkest-gray;
365368
border: none;
366369
}
367370
}

0 commit comments

Comments
 (0)