Skip to content

Conversation

YongThePsibor
Copy link
Contributor

As in bug #1041 and #1217 described. When filtered, we cannot just use draggingRow?.index for rangeExtractor, we should find the correct dragging row index in filtered rows.

Copy link

vercel bot commented Jan 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
material-react-table ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 19, 2025 2:27pm
material-react-table-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 19, 2025 2:27pm

@YongThePsibor
Copy link
Contributor Author

anybody help with the github actions. It says:

Error: The process '/usr/bin/git' failed with exit code 1

@YongThePsibor YongThePsibor marked this pull request as draft January 18, 2025 12:56
@YongThePsibor
Copy link
Contributor Author

@KevinVandy Could you please fix the github action error, I have no idea how. thx

@KevinVandy KevinVandy merged commit 4aefa15 into KevinVandy:v3 Feb 14, 2025
3 of 4 checks passed
jonathanl-dropit pushed a commit to dropit-dev/material-react-table that referenced this pull request Jun 22, 2025
…ing fixes (KevinVandy#1361)

* fix: KevinVandy#1041, KevinVandy#1217, wrong draggingRowIndex in virtual when filtered.

* docs: add docs of wyh should do a extra index of rangeExtractor

* chore: add vscode launch.json for debug storybook

* Delete .vscode/launch.json

* fix: draggingRowIndex should depend on realRows

---------

Co-authored-by: 章勇 <[email protected]>
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.

2 participants