Skip to content

Conversation

@sdidier-dev
Copy link
Contributor

@sdidier-dev sdidier-dev commented Jul 20, 2025

Hi guys! A tiny PR to:

  • add rowSelection in GRID_NESTED_FUNCTIONS, that allows nested functions for checkboxes and isRowSelectable (already in GRID_MAYBE_FUNCTIONS)
  • add checkboxes in GRID_MAYBE_FUNCTIONS, now nested in rowSelection
  • remove deprecated checkboxSelection and headerCheckboxSelection
  • add an example using checkboxes as function

…ckboxSelection` and `headerCheckboxSelection`, and example for selection checkboxes as function
@sdidier-dev
Copy link
Contributor Author

Should fix #379

@KoolADE85
Copy link
Contributor

Great catch @sdidier-dev ! This works well on my end too. Can you change the merge target to main, rather than v32? I'm going to release this off of main.

@sdidier-dev sdidier-dev changed the base branch from v32 to main July 21, 2025 19:03
@sdidier-dev
Copy link
Contributor Author

Here we go! 😁

…ility with deprecated API, update tests and propCategories.js
… into v32-add-props-in-propCategories

# Conflicts:
#	src/lib/fragments/AgGrid.react.js
Copy link
Collaborator

@BSd3v BSd3v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a changelog
💃

@KoolADE85
Copy link
Contributor

Add a changelog 💃

I'll merge this and add a changelog entry as part of the release!

@KoolADE85 KoolADE85 merged commit 3b7fa53 into plotly:main Jul 23, 2025
3 checks passed
@sdidier-dev sdidier-dev deleted the v32-add-props-in-propCategories branch July 31, 2025 14:55
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.

isRowSelectable is broken with v32.3.0rc0

3 participants