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

Support varying character widths #460

Open
fjakobs opened this issue Oct 11, 2011 · 29 comments
Open

Support varying character widths #460

fjakobs opened this issue Oct 11, 2011 · 29 comments

Comments

@fjakobs
Copy link
Contributor

fjakobs commented Oct 11, 2011

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"}

@fjakobs
Copy link
Contributor Author

fjakobs commented Oct 24, 2011

This includes non monospace fonts and asian characters

@sirupsen
Copy link

Any updates on this?

@fjakobs
Copy link
Contributor Author

fjakobs commented Dec 16, 2011

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.

https://github.com/ajaxorg/ace/tree/varchar

@sirupsen
Copy link

Will it be fixed to one line-height, or is it possible to allow multiple line-heights in the document?

@fjakobs
Copy link
Contributor Author

fjakobs commented Dec 16, 2011

variable line height is currently not planned.

@jevakallio
Copy link

Are there any plans to continue work on this feature in the (near-ish) future?

@nightwing
Copy link
Member

not in next 2-3 months.

@vegetableman
Copy link

Thanks @fjakobs , it worked beautifully with different font families.

@bvibber
Copy link

bvibber commented Sep 15, 2013

This has been biting us at Wikipedia -- we're using ace to do on-site editing of Lua, JavaScript etc which may include localized text in all sorts of languages.

I've updated @fjakobs 's older patch to run on current ace master: #1608

Appears to work for me in some ad-hoc testing.

@bvibber
Copy link

bvibber commented Sep 15, 2013

For handy testing, here's a (temporary) live copy built from the pull req: https://brionv.com/misc/demo/ace/build/kitchen-sink.html

@FahmiA
Copy link

FahmiA commented Feb 12, 2014

Any progress with this bug?

@georgephillips
Copy link

Any news on this feature?

@wohali
Copy link

wohali commented Sep 4, 2014

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.

ⅠⅡⅢⅣⅤⅰⅱⅲⅳⅴ
ᠠᠡᠢᠣᠤᠥᠦᠧᠨᠩ
༐༑༒༓༔༕༖༗༘༙
بةتثجحخد

Screenshot:
capture

@honzajavorek
Copy link

Hi, what's status of this issue? Can we be of any help?

@nightwing
Copy link
Member

@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.

@andrewnester andrewnester pinned this issue May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests