Skip to content

fix: Update package version of codemirror/view to fix carret low contrast #947

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
Dec 9, 2024

Conversation

heygwangjin
Copy link
Contributor

@heygwangjin heygwangjin commented Dec 6, 2024

To fix #925 issue, I reported this bug to codemirror and they fixed it. (codemirror/dev#1487)

So I updated package version of codemirror/view and we can see white caret in dark mode on mobile safari.

Changed the version of codemirror in below packages.

  • /packages/editor
  • /packages/repl
  • /packages/site-kit

Before

before

After

after

Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
svelte-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 6, 2024 3:59pm

Copy link

vercel bot commented Dec 6, 2024

@heygwangjin is attempting to deploy a commit to the Svelte Team on Vercel.

A member of the Team first needs to authorize it.

* Update codemirror/view package version to fix low caret contrast on mobile safari
@Rich-Harris Rich-Harris merged commit 337799e into sveltejs:main Dec 9, 2024
5 checks passed
@Rich-Harris
Copy link
Member

nice, thank you!

@Rich-Harris
Copy link
Member

Unfortunately we had to revert this because there's quite a nasty bug in the latest version — #974

@heygwangjin
Copy link
Contributor Author

@Rich-Harris
I’m sorry, I didn’t know there was a bug in Chrome.

I think this bug has also been reported by someone else (https://discuss.codemirror.net/t/cursor-position-issue-on-the-end-of-the-line/8866/5)

The maintainer has fixed it in @codemirror/view version 6.35.2 (codemirror/view@2aa5d7f)

However, after updating the package to version 6.35.2 and testing it locally, the Svelte compiler doesn’t seem to work properly.

Screenshot 2024-12-11 at 6 03 57 AM

So, it seems like we need to keep an eye on this for a bit longer.

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.

REPL caret is low contrast
2 participants