Skip to content

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
@nx-cloud
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".

@frankieali
Copy link
Contributor

Any updates on this bugfix?

@jfraudeau
Copy link

Hi, is there anything that can be done to get this PR merged ? Would love to see it in the next release !

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

3 participants