Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(table-core): ensure isSubRowSelected returns false if no subRows are selectable #5790

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

remleduff
Copy link

This fixes #5173

Keep track whether we've seen any selectable rows and if not return false from isSubRowSelected.

@remleduff remleduff force-pushed the feat-rowselection-with-no-selectable branch from 6af6819 to c976103 Compare November 4, 2024 22:05
@remleduff remleduff changed the title Ensure isSubRowSelected returns false if no subRows are selctable Ensure isSubRowSelected returns false if no subRows are selectable Nov 4, 2024
@remleduff remleduff force-pushed the feat-rowselection-with-no-selectable branch from c976103 to e762532 Compare November 4, 2024 22:06
@remleduff remleduff force-pushed the feat-rowselection-with-no-selectable branch from e762532 to 2ae44a7 Compare November 4, 2024 22:07
@remleduff remleduff changed the title Ensure isSubRowSelected returns false if no subRows are selectable fix: ensure isSubRowSelected returns false if no subRows are selectable Nov 4, 2024
@remleduff remleduff changed the title fix: ensure isSubRowSelected returns false if no subRows are selectable fix(table-core): ensure isSubRowSelected returns false if no subRows are selectable Nov 4, 2024
@remleduff remleduff force-pushed the feat-rowselection-with-no-selectable branch from 6255ba8 to 220d8d3 Compare November 5, 2024 17:55
Copy link

nx-cloud bot commented Nov 7, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d87e9bc. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@remleduff
Copy link
Author

remleduff commented Nov 7, 2024

@KevinVandy Do I need to do anything for this? It seems like the automated fixes are correct, but when I click through to the workflow its status is "failed".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

row.getIsSomeSelected returning true for row with disabled subrows
1 participant