ACP2E-3350: Disabled Categories are no longer have their names grayed out in the category tree

lakshmana49 · lakshmana49 · commit eeb88ccd3512 · 2024-10-21T20:48:38.000+05:30
diff --git a/app/code/Magento/Catalog/Block/Adminhtml/Category/Tree.php b/app/code/Magento/Catalog/Block/Adminhtml/Category/Tree.php
@@ -392,7 +392,7 @@ protected function _getNodeJson($node, $level = 0)
         $item['id'] = $node->getId();
         $item['store'] = (int)$this->getStore()->getId();
         $item['path'] = $node->getData('path');
-        $item['a_attr'] = ['class' => $node->getIsActive() ? 'active-category' : 'no-active-category'];
+        $item['a_attr'] = ['class' => $node->getIsActive() ? 'active-category' : 'not-active-category'];
         //$item['allowDrop'] = ($level<3) ? true : false;
         $allowMove = $this->_isCategoryMoveable($node);
         $item['allowDrop'] = $allowMove;
diff --git a/app/code/Magento/Catalog/view/adminhtml/layout/catalog_category_edit.xml b/app/code/Magento/Catalog/view/adminhtml/layout/catalog_category_edit.xml
@@ -9,7 +9,6 @@
     <update handle="styles"/>
     <head>
         <css src="jquery/uppy/dist/uppy.fileupload-ui.css"/>
-        <css src="Magento_Catalog::css/styles.css"/>
         <link src="Magento_Catalog::js/edit-tree.js"/>
     </head>
     <update handle="editor"/>
diff --git a/app/code/Magento/Catalog/view/adminhtml/web/css/styles.css b/app/code/Magento/Catalog/view/adminhtml/web/css/styles.css
diff --git a/app/design/adminhtml/Magento/backend/Magento_Catalog/web/css/source/_module.less b/app/design/adminhtml/Magento/backend/Magento_Catalog/web/css/source/_module.less
@@ -121,3 +121,13 @@
         }
     }
 }
+
+//
+//  Catalog Category Tree
+//  ---------------------------------------------
+
+.jstree-children {
+    .not-active-category {
+        color: #aaaaaa;
+    }
+}
diff --git a/dev/tests/integration/testsuite/Magento/Catalog/Block/Adminhtml/Category/TreeBlockTest.php b/dev/tests/integration/testsuite/Magento/Catalog/Block/Adminhtml/Category/TreeBlockTest.php
@@ -49,6 +49,6 @@ public function testDisabledCategoriesHtml()
         $this->assertCount(1, $categoryTreeArray);
         $this->assertArrayHasKey('a_attr', $categoryTreeArray[0]);
         $this->assertArrayHasKey('class', $categoryTreeArray[0]['a_attr']);
-        $this->assertStringContainsString('no-active-category', $categoryTreeArray[0]['a_attr']['class']);
+        $this->assertStringContainsString('not-active-category', $categoryTreeArray[0]['a_attr']['class']);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -121,3 +121,13 @@`
`121`	`121`	`}`
`122`	`122`	`}`
`123`	`123`	`}`
	`124`	`+`
	`125`	`+//`
	`126`	`+// Catalog Category Tree`
	`127`	`+// ---------------------------------------------`
	`128`	`+`
	`129`	`+.jstree-children {`
	`130`	`+ .not-active-category {`
	`131`	`+ color: #aaaaaa;`
	`132`	`+ }`
	`133`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,6 @@ public function testDisabledCategoriesHtml()`
`49`	`49`	`$this->assertCount(1, $categoryTreeArray);`
`50`	`50`	`$this->assertArrayHasKey('a_attr', $categoryTreeArray[0]);`
`51`	`51`	`$this->assertArrayHasKey('class', $categoryTreeArray[0]['a_attr']);`
`52`		`- $this->assertStringContainsString('no-active-category', $categoryTreeArray[0]['a_attr']['class']);`
	`52`	`+ $this->assertStringContainsString('not-active-category', $categoryTreeArray[0]['a_attr']['class']);`
`53`	`53`	`}`
`54`	`54`	`}`