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.

Sign up for GitHub

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

Jump to bottom

selection is data indexed, not table indexed #34

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

selection is data indexed, not table indexed #34

wants to merge 6 commits into from
+381 −41

Conversation

severo
Copy link
Contributor

@severo severo commented Feb 4, 2025 •
edited
Loading

fixes #32

Naive implementation for now.

  • try the demo to ensure it works as expected
  • add tests
  • optimize (let the user provide an index of the sorted rows?)

@severo severo marked this pull request as draft February 4, 2025 15:07
@severo severo force-pushed the selection-is-data-indexed branch 2 times, most recently from 31336e6 to 8ca343c Compare February 11, 2025 09:25
@severo severo force-pushed the selection-is-data-indexed branch from 8ca343c to 978a523 Compare February 11, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The selection should be sorting aware
1 participant
@severo