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.

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

Feature: shift to multiselect in QuickTag view #451

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

inversion
Copy link
Contributor

@inversion inversion commented Feb 26, 2025

Basic implementation to add a range of tracks. Does not implement nuanced UX of platforms e.g. https://stackoverflow.com/a/16530782

It simply gets the previous selection index, and adds anything to the selection between that index and the current click index.

It does not act in combination with ctrlKey click behavior, ctrlKey takes precedence.

See #450 for original ask.

20250225_223850.mp4

In the video we use shift to expand the selection three times, then deselect three tracks with ctrl click, then expand the selection again. I also tested expanding 'up' but didn't include this in the video.

@inversion
Copy link
Contributor Author

This was previously only supported using shift + up/down arrow, not shift and click.

@inversion inversion marked this pull request as ready for review February 28, 2025 06:33
Copy link
Owner

@Marekkon5 Marekkon5 left a comment

Choose a reason for hiding this comment

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

Tested locally, worked as in video, lgtm

@Marekkon5 Marekkon5 merged commit 16aaf28 into Marekkon5:master Feb 28, 2025
3 checks passed
@Marekkon5
Copy link
Owner

Thank you very much

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