From 91a7d51ec842817913a934d281390083352c4e76 Mon Sep 17 00:00:00 2001 From: Rebecca Alpert Date: Wed, 17 Jun 2026 15:12:36 -0400 Subject: [PATCH] fix(ArrowsAltVIcon): Replace ArrowsAltVIcon with RhUiArrowUpDownIcon Co-authored-by: Cursor --- .../src/components/Table/SortColumn.tsx | 4 +- .../__snapshots__/Table.test.tsx.snap | 320 ++++-------------- 2 files changed, 66 insertions(+), 258 deletions(-) diff --git a/packages/react-table/src/components/Table/SortColumn.tsx b/packages/react-table/src/components/Table/SortColumn.tsx index 4321c4923b7..325815b3874 100644 --- a/packages/react-table/src/components/Table/SortColumn.tsx +++ b/packages/react-table/src/components/Table/SortColumn.tsx @@ -1,7 +1,7 @@ import { useState } from 'react'; import LongArrowAltUpIcon from '@patternfly/react-icons/dist/esm/icons/long-arrow-alt-up-icon'; import LongArrowAltDownIcon from '@patternfly/react-icons/dist/esm/icons/long-arrow-alt-down-icon'; -import ArrowsAltVIcon from '@patternfly/react-icons/dist/esm/icons/arrows-alt-v-icon'; +import RhUiArrowUpDownIcon from '@patternfly/react-icons/dist/esm/icons/rh-ui-arrow-up-down-icon'; import { css } from '@patternfly/react-styles'; import styles from '@patternfly/react-styles/css/components/Table/table'; import { TableText } from './TableText'; @@ -45,7 +45,7 @@ export const SortColumn: React.FunctionComponent = ({ if (isSortedBy) { SortedByIcon = sortDirection === SortByDirection.asc ? LongArrowAltUpIcon : LongArrowAltDownIcon; } else { - SortedByIcon = ArrowsAltVIcon; + SortedByIcon = RhUiArrowUpDownIcon; } if (favoriteButtonProps) { diff --git a/packages/react-table/src/deprecated/components/Table/__tests__/__snapshots__/Table.test.tsx.snap b/packages/react-table/src/deprecated/components/Table/__tests__/__snapshots__/Table.test.tsx.snap index cb0bcd21b81..af53ff6d6c1 100644 --- a/packages/react-table/src/deprecated/components/Table/__tests__/__snapshots__/Table.test.tsx.snap +++ b/packages/react-table/src/deprecated/components/Table/__tests__/__snapshots__/Table.test.tsx.snap @@ -47,24 +47,12 @@ exports[`Table Actions table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -1007,24 +995,12 @@ exports[`Table Cell header table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -1590,24 +1566,12 @@ exports[`Table Collapsible nested table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -2363,24 +2327,12 @@ exports[`Table Collapsible table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -4415,24 +4367,12 @@ exports[`Table Selectable table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -5161,24 +5101,12 @@ exports[`Table Selectable table with Radio 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -6024,24 +5952,12 @@ exports[`Table Simple Actions table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -7078,24 +6994,12 @@ exports[`Table Table variants Breakpoint - 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -7618,24 +7522,12 @@ exports[`Table Table variants Breakpoint - grid 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -8158,24 +8050,12 @@ exports[`Table Table variants Breakpoint - grid-2xl 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -8698,24 +8578,12 @@ exports[`Table Table variants Breakpoint - grid-lg 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -9238,24 +9106,12 @@ exports[`Table Table variants Breakpoint - grid-md 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -9778,24 +9634,12 @@ exports[`Table Table variants Breakpoint - grid-xl 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -10318,24 +10162,12 @@ exports[`Table Table variants Size - compact 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -11441,24 +11273,12 @@ exports[`Table simple Row click table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - + @@ -11981,24 +11801,12 @@ exports[`Table simple Sortable table 1`] = ` fill="currentColor" height="1em" role="img" + viewBox="0 0 32 32" width="1em" > - - - - - - +