-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix line wrap for CJK characters #11296
base: main
Are you sure you want to change the base?
Conversation
b95e7c3
to
9113c99
Compare
The second commit is updated to match the exist test cases. https://github.com/zed-industries/zed/blob/v0.134.1-pre/crates/editor/src/editor_tests.rs#L1272 But in this way, the URL cannot be forcibly truncated, as shown below. Compared with my previous video, the new changes have a little issue compared to the previous video. However, at least most of the previous details will be retained (because I did not modify the previous test cases). |
49720a1
to
311c858
Compare
good job! |
Thanks for this PR! Really appreciate the attention to detail on other languages and I'd like to get it merged. |
Added more support: Latin-1 Supplement, Extended-A,B,C, Cyrillic Ref: |
Given the increase in ergonomics, and the coverage for the languages we're capable of supporting, I think it's good to try this one out. Thank you! |
Ahh, seems there's an issue with the macOS tests :) |
Test now passed |
Final test 2024-05-18.11.33.58.mov |
Hi @mikayla-maki, how about this PR |
Release Notes:
Demo
input1.mov
Fix issues: #4623 #11202
Render case