-
-
Notifications
You must be signed in to change notification settings - Fork 251
add vim mode setting to playground #1076
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
base: master
Are you sure you want to change the base?
Conversation
@rbertramResi is attempting to deploy a commit to the ReScript Association Team on Vercel. A member of the Team first needs to authorize it. |
8d7cf1b
to
1c7c4de
Compare
1c7c4de
to
4586f71
Compare
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.
Nice!
Maybe you could label it "Use Vim Keymap"?
src/Playground.res
Outdated
let (keyMap, setKeyMap) = React.useState(() => { | ||
Dom.Storage2.localStorage | ||
->Dom.Storage2.getItem("vimMode") | ||
->Belt.Option.map(CodeMirror.KeyMap.fromString) |
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.
just use Option
without the Belt
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.
updated 👍 And changed the label over the toggle to "Use Vim Keymap"
71f0f5d
to
3abae75
Compare
No description provided.