🎯 Changes

This pull request improves the row selection feature in the table core package by ensuring that selection and deselection operations respect theenableRowSelection configuration. It also adds comprehensive tests to verify this behavior for both selecting and deselecting all rows.

Bug Explanation:
When a row is selected and enableRowSelection is set to false, the row should remain selected because selection is supposed to be locked/disabled. However, when the user triggers Unselect All, the row becomes unselected even though selection is disabled.

Expected Behavior:
The row should remain selected when enableRowSelection = false, and the “Unselect All” action should not clear its selection state.

Actual Behavior:
The row is still unselected after “Unselect All,” meaning the selection state is not being preserved when selection is disabled.

Demo

=> In demo, the rowBarney should not be unselected

Testing Enhancements:

• Added new tests inRowSelection.test.ts to verify thattoggleAllRowsSelected correctly respects theenableRowSelection setting when selecting all rows, ensuring non-selectable rows remain unaffected.
• Added tests to confirm that when deselecting all rows, rows that are not allowed to be deselected (according toenableRowSelection) retain their selection state.

✅ Checklist

• I have followed the steps in theContributing guide.
• I have tested this code locally withpnpm test:pr.

🚀 Release Impact

• This change affects published code, and I have generated achangeset.
• This change is docs/CI/dev-only (no release).

Summary by CodeRabbit

• Bug Fixes

  • Fixed row selection to properly respect non-selectable rows when deselecting all rows, ensuring consistent behavior with select-all operations.

• Tests

  • Added test coverage for row selection toggle operations to verify proper handling of non-selectable rows during bulk selection changes.

_{✏️ Tip: You can customize this high-level summary in your review settings.}