Skip to content

Commit

Permalink
STSMACOM-871: Remove unnecessary aria-rowindex in ItemView and `I…
Browse files Browse the repository at this point in the history
…temEdit` components (#1533)

* STSMACOM-871: Remove unnecessary aria-rowindex in ItemView and ItemEdit components

* STSMACOM-871: Revert aria-rowindex in ItemView and ItemEdit components

* STSMACOM-871: Replace aria-rowindex with data-row-inner

(cherry picked from commit 52a627b)
  • Loading branch information
OleksandrHladchenko1 authored and zburke committed Oct 31, 2024
1 parent 1f8018f commit 595c3e6
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Change history for stripes-smart-components

## 9.2.2

* Remove unnecessary `aria-rowindex` in `ItemView` and `ItemEdit` components. Fixes STSMACOM-871.

## [9.2.1](https://github.com/folio-org/stripes-smart-components/tree/v9.2.1) (2024-10-30)
[Full Changelog](https://github.com/folio-org/stripes-smart-components/compare/v9.2.0...v9.2.1)

Expand Down
2 changes: 1 addition & 1 deletion lib/EditableList/ItemEdit.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ const ItemEdit = ({
{ [css.isOdd]: !(rowIndex % 2) },
rowClass,
)}
aria-rowindex={rowIndex + 2}
data-row-inner={rowIndex + 2}
>
{fields}
</div>
Expand Down
2 changes: 1 addition & 1 deletion lib/EditableList/ItemView.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const ItemView = ({ cells, rowClass, rowIndex }) => (
{ [css.isOdd]: !(rowIndex % 2) },
rowClass,
)}
aria-rowindex={rowIndex + 2}
data-row-inner={rowIndex + 2}
>
{cells}
</div>
Expand Down

0 comments on commit 595c3e6

Please sign in to comment.