Skip to content

Add options argument to selectMulti for consistency with select#266

Merged
TrevorBurnham merged 1 commit intojameskerr:mainfrom
paulftw:patch-1
Apr 19, 2026
Merged

Add options argument to selectMulti for consistency with select#266
TrevorBurnham merged 1 commit intojameskerr:mainfrom
paulftw:patch-1

Conversation

@paulftw
Copy link
Copy Markdown
Contributor

@paulftw paulftw commented Aug 18, 2024

select(...) supports optional flag to disable focus change but selectMulti(...) doesn't. This change makes the two methods consistent.
It shouldn't break existing code.
I copy-pasted the align option without understanding it.

For reference: original select

select(...) supports optional flag to disable focus change but selectMulti(...) doesn't.
This change makes the two methods consistent.
It shouldn't break existing code.
I copy-pasted the align option without understanding it.
@TrevorBurnham TrevorBurnham merged commit 8e11b37 into jameskerr:main Apr 19, 2026
@TrevorBurnham TrevorBurnham mentioned this pull request Apr 20, 2026
3 tasks
TrevorBurnham added a commit that referenced this pull request Apr 20, 2026
- Document dndBackend prop (#326) and correct dndManager type in TreeProps
- Document opts arg on tree.select / tree.selectMulti (#266)
- Correct childrenAccessor return type to readonly T[] | null
- Add Utilities section documenting getTreeLinePrefix (#324)
TrevorBurnham added a commit that referenced this pull request Apr 20, 2026
- Document dndBackend prop (#326) and correct dndManager type in TreeProps
- Document opts arg on tree.select / tree.selectMulti (#266)
- Correct childrenAccessor return type to readonly T[] | null
- Add Utilities section documenting getTreeLinePrefix (#324)
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