Skip to content

Turn on wordwrap on code editor #2291

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

Merged
merged 1 commit into from
Mar 19, 2025
Merged

Turn on wordwrap on code editor #2291

merged 1 commit into from
Mar 19, 2025

Conversation

johnsimons
Copy link
Member

This fixes issue with overflow in dialogs

This fixes issue with overflow in dialogs
@johnsimons johnsimons requested a review from PhilBastian March 18, 2025 23:51
@johnsimons johnsimons self-assigned this Mar 18, 2025
@johnsimons johnsimons added the Type: Improvement Type: Improvement label Mar 18, 2025
@johnsimons johnsimons enabled auto-merge March 18, 2025 23:51
@johnsimons johnsimons added this to the vNext milestone Mar 18, 2025
@@ -53,7 +53,7 @@ const extensions = computed(() => {
<div v-if="props.showCopyToClipboard" class="toolbar">
<CopyToClipboard :value="code" />
</div>
<CodeMirror v-model="code" :extensions="extensions" :basic="props.showGutter" :minimal="!props.showGutter" :readonly="props.readOnly" :gutter="!props.readOnly"></CodeMirror>
<CodeMirror v-model="code" :extensions="extensions" :basic="props.showGutter" :minimal="!props.showGutter" :readonly="props.readOnly" :gutter="!props.readOnly" :wrap="true"></CodeMirror>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be put on all usages of CodeMirror, so that in future if people copy it from a page other than this one they get this same behaviour?

@johnsimons johnsimons merged commit 2d600f2 into master Mar 19, 2025
5 checks passed
@johnsimons johnsimons deleted the john/wordwrap branch March 19, 2025 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Improvement Type: Improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants