-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@heygwangjin is attempting to deploy a commit to the Svelte Team on Vercel. A member of the Team first needs to authorize it. |
c1b7928
to
b12c5c7
Compare
b12c5c7
to
5f9a977
Compare
* Update codemirror/view package version to fix low caret contrast on mobile safari
5f9a977
to
cae2125
Compare
nice, thank you! |
Unfortunately we had to revert this because there's quite a nasty bug in the latest version — #974 |
@Rich-Harris 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. ![]() So, it seems like we need to keep an eye on this for a bit longer. |
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.
Before
After