-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[fields] Do not apply digit editing when pressing Space
(@flaviendelangle)
#13516
Conversation
Signed-off-by: Flavien DELANGLE <flaviendelangle@gmail.com>
Signed-off-by: Flavien DELANGLE <flaviendelangle@gmail.com>
Deploy preview: https://deploy-preview-13516--material-ui-x.netlify.app/ |
version "1.0.30001576" | ||
resolved "https://registry.yarnpkg.com/caniuse-lite/-/caniuse-lite-1.0.30001576.tgz#893be772cf8ee6056d6c1e2d07df365b9ec0a5c4" | ||
integrity sha512-ff5BdakGe2P3SQsMsiqmt1Lc8221NR1VzHj5jXN5vBny9A6fpze94HiVV/n7XRosOlsShJcvMv5mdnpjOGCEgg== | ||
version "1.0.30001587" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've manually installed a newer version that passed the build.
Installing the latest package would also fail the build. 🙈
This is not directly related to this PR, but it's just a side-effect of the caniuse-lite
.
Building on the v6.x
would have resulted in the same error and the need to bump the package.
Asked for a review from @flaviendelangle to have a final look and the "honour" to merge with with overriding checks (vale fails for some unknown reason 🤷 ) |
Cherry-pick of #13510