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

Add keyboard event to navigate the race-chart #34

Merged
merged 2 commits into from
Mar 21, 2023

Conversation

THeK3nger
Copy link
Contributor

I added a keyboard listener for the arrow key when the race-chart's range input is focused. In this way, it will be possible to navigate the chart by using the left and right keys.

Adds a keyboard listener for the arrow key when the range input is
focused in the race-chart. In this way, it would be possible to navigate
the chart by using the left and right keys.
@felhag
Copy link
Owner

felhag commented Mar 20, 2023

Hey, thanks for your contribution :) Seems like a nice little feature. Did you try to change onclick to onchange? I'd say that would work as well.

@THeK3nger
Copy link
Contributor Author

Haha! That's my lack of experience with front-end development in action. 😅 I updated the PR using onchange, and it works.

@felhag felhag merged commit 5a4169a into felhag:master Mar 21, 2023
@felhag
Copy link
Owner

felhag commented Mar 21, 2023

Cheers, I will include it in the next release.

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