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

Fix: A display of unicode composing characters by CoreText renderer #680

Merged
merged 1 commit into from
May 3, 2018

Conversation

ichizok
Copy link
Member

@ichizok ichizok commented Apr 28, 2018

To reconstruct and use gui_macvim_draw_string for CoreText renderer.
This PR will fix #400 #604 vim-jp/issues#1114.

Current:

  • Cannot display correctly Unicode IVS and composing characters

macvim-coretextrenderer 1

Patched: almost fixed

  • NOTE: cannot display correctly sequences with virama (e.g. devanagari. the last 3 lines)
  • NOTE: 'macligatures' doesn't work. (always ligatures at using FiraCode font) This may be undesirable?

macvim-coretextrenderer 2

Reconstruct and use 'gui_macvim_draw_string' for CoreText renderer
@splhack splhack merged commit 1496167 into macvim-dev:master May 3, 2018
@ichizok ichizok deleted the fix/unicode-composing branch May 3, 2018 12:39
@amadeus
Copy link

amadeus commented May 3, 2018

Not totally sure, but I think this PR may have created a new bug:

example

@ichizok
Copy link
Member Author

ichizok commented May 6, 2018

@amadeus That seems to be caused by f53f990. Please confirm.

@splhack
Copy link
Contributor

splhack commented May 10, 2018

Should we revert f53f990?

@ichizok
Copy link
Member Author

ichizok commented May 10, 2018

I expect master...ichizok:fix/coretext-renderer can fix it (so need not revert). I will create PR soon.

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.

Combining accents not rendered correctly
3 participants