MC-3479: Remove Accordion

- remove unused classes and attributes and add comment to less file

Author: krissyhiserote

app/code/Magento/PageBuilder/Setup/DataConverter/Renderer/AccordionItem.php

@@ -47,8 +47,7 @@ public function render(array $itemData, array $additionalData = []) : string
         // data-role is not present on the accordion item as it's no longer it's own type
         $rootElementAttributes = [
             'data-collapsible' => 'true',
-            'class' => 'item title accordion-collapsible',
-            'data-open-on-load' => $eavData['open_on_load'] ? "1" : "0"
+            'class' => 'item title'
         ];
 
         if (isset($itemData['formData'])) {
@@ -63,7 +62,7 @@ public function render(array $itemData, array $additionalData = []) : string
             $rootElementHtml .= $attributeValue !== '' ? " $attributeName=\"$attributeValue\"" : '';
         }
         $rootElementHtml .= '><div class="switch" data-role="trigger"><span>' . $eavData['title'] . '</span></div></div>'
-        . '<div data-content="true" class="item content accordion-content">' . $eavData['textarea'] . '</div>';
+        . '<div data-content="true" class="item content">' . $eavData['textarea'] . '</div>';
 
         return $rootElementHtml;
     }

app/code/Magento/PageBuilder/view/frontend/web/css/source/_module.less

@@ -37,6 +37,10 @@
     margin-bottom: 20px;
 }
 
+//
+//  styles for legacy content types
+//  _____________________________________________
+
 .pagebuilder-accordion {
     .lib-data-accordion();
 }

dev/tests/integration/testsuite/Magento/PageBuilder/_files/accordions.html

@@ -1 +1 @@
-<div data-element="main" data-role="row" data-appearance="default"><div data-element="main" data-role="html" data-appearance="default"><div data-mage-init="{&quot;accordion&quot;:{&quot;active&quot;:[0],&quot;collapsibleElement&quot;:&quot;[data-collapsible=true]&quot;,&quot;content&quot;:&quot;[data-content=true]&quot;}}" class="pagebuilder-accordion one two"><div data-collapsible="true" class="item title accordion-collapsible" data-open-on-load="0"><div class="switch" data-role="trigger"><span>Accordion Title</span></div></div><div data-content="true" class="item content accordion-content"><p>Accordion Contents</p></div></div></div></div>
+<div data-element="main" data-role="row" data-appearance="default"><div data-element="main" data-role="html" data-appearance="default"><div data-mage-init="{&quot;accordion&quot;:{&quot;active&quot;:[0],&quot;collapsibleElement&quot;:&quot;[data-collapsible=true]&quot;,&quot;content&quot;:&quot;[data-content=true]&quot;}}" class="pagebuilder-accordion one two"><div data-collapsible="true" class="item title"><div class="switch" data-role="trigger"><span>Accordion Title</span></div></div><div data-content="true" class="item content"><p>Accordion Contents</p></div></div></div></div>

dev/tests/integration/testsuite/Magento/PageBuilder/_files/accordions_open_on_load.html

@@ -1 +1 @@
-<div data-element="main" data-role="row" data-appearance="default"><div data-element="main" data-role="html" data-appearance="default"><div data-mage-init="{&quot;accordion&quot;:{&quot;active&quot;:[1],&quot;collapsibleElement&quot;:&quot;[data-collapsible=true]&quot;,&quot;content&quot;:&quot;[data-content=true]&quot;}}" class="pagebuilder-accordion one two"><div data-collapsible="true" class="item title accordion-collapsible" data-open-on-load="0"><div class="switch" data-role="trigger"><span>Title 1</span></div></div><div data-content="true" class="item content accordion-content"><p>Content 1</p></div><div data-collapsible="true" class="item title accordion-collapsible" data-open-on-load="1"><div class="switch" data-role="trigger"><span>Title 2</span></div></div><div data-content="true" class="item content accordion-content"><p>Content 2</p></div></div></div></div>
+<div data-element="main" data-role="row" data-appearance="default"><div data-element="main" data-role="html" data-appearance="default"><div data-mage-init="{&quot;accordion&quot;:{&quot;active&quot;:[1],&quot;collapsibleElement&quot;:&quot;[data-collapsible=true]&quot;,&quot;content&quot;:&quot;[data-content=true]&quot;}}" class="pagebuilder-accordion one two"><div data-collapsible="true" class="item title"><div class="switch" data-role="trigger"><span>Title 1</span></div></div><div data-content="true" class="item content"><p>Content 1</p></div><div data-collapsible="true" class="item title"><div class="switch" data-role="trigger"><span>Title 2</span></div></div><div data-content="true" class="item content"><p>Content 2</p></div></div></div></div>