-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Support varying character widths #460
Comments
This includes non monospace fonts and asian characters |
Any updates on this? |
I have it in an experimental branch. It doesn't support soft wrap yet and has some issues with tab characters but otherwise it works very well already. |
Will it be fixed to one line-height, or is it possible to allow multiple line-heights in the document? |
variable line height is currently not planned. |
Are there any plans to continue work on this feature in the (near-ish) future? |
not in next 2-3 months. |
Thanks @fjakobs , it worked beautifully with different font families. |
For handy testing, here's a (temporary) live copy built from the pull req: https://brionv.com/misc/demo/ace/build/kitchen-sink.html |
Any progress with this bug? |
Any news on this feature? |
We'd really love to see this merged in; it looks like potentially @Brion or @fjakobs 's branches have workarounds for variable width characters. I also believe #1797, #1153 and #1961 are duplicates of this ticket. Steps to recreate: paste this text into an ACE editor and cursor around, you'll see the cursor will land in the middle of the double-wide characters such as the roman numerals. ⅠⅡⅢⅣⅤⅰⅱⅲⅳⅴ |
Hi, what's status of this issue? Can we be of any help? |
@honzajavorek the latest version of somewhat working attempt to fix this issue is in #2002, it's still not ready to merge, and wrapping support may require major changes. |
CM2 does it by measuring the actual text and not a generic string.
https://github.com/marijnh/CodeMirror2/blob/master/lib/codemirror.js#L1209
@Gissues:{"order":41.6149068322984,"status":"backlog"}
The text was updated successfully, but these errors were encountered: