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

refactor: shift select respects direction #207

Merged
merged 1 commit into from
Jul 16, 2021

Conversation

DomHynes
Copy link
Contributor

This PR checks the indexes when shift selecting replays, then flips the array if the user has shift selected a newer replay after an older one.

This means it's possible to queue up vods in chronological order with shift select.

@DomHynes
Copy link
Contributor Author

I just realised the problem this solves (chronological shift selecting) can be worked around by sorting by Least Recent, but this still updates the shift select behaviour to be more idiomatic.

Copy link
Member

@vinceau vinceau left a comment

Choose a reason for hiding this comment

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

LGTM

@vinceau vinceau merged commit 6d4d56d into project-slippi:main Jul 16, 2021
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