Skip to content

Commit 0b0b73d

Browse files
feat(TreeView): removed selectable modifier for penta (#10101)
1 parent 63695c1 commit 0b0b73d

File tree

2 files changed

+0
-33
lines changed

2 files changed

+0
-33
lines changed

packages/react-core/src/components/TreeView/TreeViewListItem.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,6 @@ const TreeViewListItemBase: React.FunctionComponent<TreeViewListItemProps> = ({
208208
<Component
209209
className={css(
210210
styles.treeViewNode,
211-
children && (isSelectable || hasCheckbox) && 'pf-m-selectable',
212211
(!children || isSelectable) &&
213212
activeItems &&
214213
activeItems.length > 0 &&

packages/react-core/src/components/TreeView/__tests__/TreeViewListItem.test.tsx

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -371,38 +371,6 @@ test(`Does not render ${styles.treeViewNode} element with additional classes by
371371
expect(treeViewNode).toHaveClass(styles.treeViewNode, { exact: true });
372372
});
373373

374-
test(`Renders ${styles.treeViewNode} element with pf-m-selectable class when hasCheckbox and children are passed`, () => {
375-
render(
376-
<TreeViewListItem hasCheckbox {...requiredProps}>
377-
Content
378-
</TreeViewListItem>
379-
);
380-
381-
const treeViewNode = screen.getByRole('treeitem').querySelector(`.${styles.treeViewNode}`);
382-
383-
expect(treeViewNode).toHaveClass('pf-m-selectable');
384-
});
385-
386-
test(`Renders ${styles.treeViewNode} element with pf-m-selectable class when isSelectable and children are passed`, () => {
387-
render(
388-
<TreeViewListItem isSelectable {...requiredProps}>
389-
Content
390-
</TreeViewListItem>
391-
);
392-
393-
const treeViewNode = screen.getByRole('treeitem').querySelector(`.${styles.treeViewNode}`);
394-
395-
expect(treeViewNode).toHaveClass('pf-m-selectable');
396-
});
397-
398-
test(`Does not render ${styles.treeViewNode} element with pf-m-selectable if children are not passed`, () => {
399-
render(<TreeViewListItem hasCheckbox isSelectable {...requiredProps} />);
400-
401-
const treeViewNode = screen.getByRole('treeitem').querySelector(`.${styles.treeViewNode}`);
402-
403-
expect(treeViewNode).not.toHaveClass('pf-m-selectable');
404-
});
405-
406374
test(`Renders ${styles.treeViewNode} element with ${styles.modifiers.current} class when isSelectable and activeItems are passed`, () => {
407375
render(
408376
<TreeViewListItem compareItems={() => true} isSelectable activeItems={[{ name: 'Active item' }]} {...requiredProps}>

0 commit comments

Comments
 (0)