-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
basic-setup - expose extension config options #663
Comments
@x1unix I think the following method is simpler, won't increase complexity, and won't increase the size. <CodeMirror
value="console.log('hello world!');"
height="200px"
basicSetup={{
lineNumbers: false
}}
extensions={[
lineNumbers({
///...
})
]}
/> |
@jaywcjlove will including extension that already present in basic setup work? |
@x1unix You can give it a try.
react-codemirror/extensions/basic-setup/src/index.ts Lines 29 to 55 in e94e34c
react-codemirror/extensions/basic-setup/src/index.ts Lines 144 to 152 in e94e34c
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A lot of extensions used in
@uiw/codemirror-extensions-basic-setup
support configuration, for examplelineNumbers
extension.This is useful when some extra event handler needs to be defined for gutter or in any other customisation case.
Can you please update
BasicSetupOptions
to accept them?Thanks.
The text was updated successfully, but these errors were encountered: