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

12/19/24 Remove slot from Codemirror #1095

Merged
merged 10 commits into from
Dec 23, 2024
Merged

12/19/24 Remove slot from Codemirror #1095

merged 10 commits into from
Dec 23, 2024

Conversation

hardingjam
Copy link
Contributor

@hardingjam hardingjam commented Dec 19, 2024

Motivation

Remove the use of slot and test component for codeMirror, instead directly test the extendOrder call, and always show the CodeMirrorRainlang component.

image

Solution

Checks

By submitting this for review, I'm confirming I've done the following:

  • made this PR as small as possible
  • unit-tested any new functionality
  • linked any relevant issues or PRs
  • included screenshots (if this involves a front-end change)

@hardingjam hardingjam requested a review from hardyjosh December 19, 2024 11:45
@hardingjam hardingjam self-assigned this Dec 19, 2024
@hardyjosh hardyjosh enabled auto-merge December 20, 2024 10:54
@hardyjosh hardyjosh disabled auto-merge December 20, 2024 12:55
@hardingjam hardingjam enabled auto-merge December 23, 2024 09:48
@hardingjam hardingjam merged commit 90d5712 into main Dec 23, 2024
21 checks passed
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.

2 participants