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

[Slider] Prevent default for Home and End keys #1076

Merged
merged 4 commits into from
Jan 13, 2022

Conversation

luisorbaiceta
Copy link
Contributor

@luisorbaiceta luisorbaiceta commented Jan 10, 2022

Closes #1054

Description

As titled

@jjenzz
Copy link
Contributor

jjenzz commented Jan 11, 2022

@luisorbaiceta there is a bit of a gotcha with our versioning 🙈

Can you run it again and deny the workspace please? We should probably update our contributing docs tbh. Sorry about that.

@luisorbaiceta
Copy link
Contributor Author

No problem! 👍

@luisorbaiceta
Copy link
Contributor Author

For some reason I don't seem to have access to CHROMATIC_PROJECT_TOKEN 🤷‍♂️ Is this expected?

@jjenzz
Copy link
Contributor

jjenzz commented Jan 13, 2022

@luisorbaiceta Yep, but to be honest I wouldn't expect your change to cause any visual regressions.

@andy-hook @benoitgrelard Shall we just merge this or think it's worth doing our usual PR branching dance still?

@luisorbaiceta
Copy link
Contributor Author

@luisorbaiceta Yep, but to be honest I wouldn't expect your change to cause any visual regressions.

Will this change in the future? I'm planning to spend time in the long term contributing to this project

@andy-hook
Copy link
Collaborator

Shall we just merge this or think it's worth doing our usual PR branching dance still?

@jjenzz I’d prefer if we stayed consistent tbh, inconvenient as it is 😅

I'm planning to spend time in the long term contributing to this project

@luisorbaiceta That's great!

Will this change in the future?

Do you mean access to secrets in code forked PRs?

@luisorbaiceta
Copy link
Contributor Author

luisorbaiceta commented Jan 13, 2022

Do you mean access to secrets in code forked PRs?

A way to check if changes have visual regressions. Chromatic suggests this:

image

Don't know if this is convenient though...

@jjenzz
Copy link
Contributor

jjenzz commented Jan 13, 2022

@luisorbaiceta I've just merged a PR which should fix all this. Would you mind pulling latest from main and pushing your branch again so we can see if it all works okay? 🙏

@luisorbaiceta
Copy link
Contributor Author

Sure! Done @jjenzz

@jjenzz jjenzz merged commit 792054a into radix-ui:main Jan 13, 2022
@jjenzz
Copy link
Contributor

jjenzz commented Jan 13, 2022

Thanks @luisorbaiceta , you should be able to see this in the next release candidate shortly 🎉

@luisorbaiceta luisorbaiceta deleted the fix/slider-prevent-default branch January 13, 2022 18:38
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.

Slider keyboard navigation not preventing default browser behaviour
3 participants