Skip to content

/ servo

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

Jump to bottom

handle multi-glyph, ligature, clusters from harfbuzz shaping output #95

Closed
burg opened this issue Oct 5, 2012 · 2 comments
Closed

handle multi-glyph, ligature, clusters from harfbuzz shaping output #95

burg opened this issue Oct 5, 2012 · 2 comments
Labels
A-platform/shaping I-wrong

Comments

@burg
Copy link

@burg burg commented Oct 5, 2012

Our shaping code assumes one glyph per char, and doesn't keep track of ligatures, clusters, and so on. Basically, we need to port Gecko's gfxHarfBuzzShaper::SetGlyphsFromRun function, which does all of this.
@burg
Copy link
Author

@burg burg commented Nov 15, 2012

An english test case for some detailed glyph stuff (probably ligatures) is lipsum.html, using Calibri Bold Italic.
@burg
Copy link
Author

@burg burg commented Nov 19, 2012

Multi-glyph characters (and ligature continuation characters) are now handled properly. Followup issues have been created for multi-byte characters (Issue #215), computing character clusters (Issue #214), and using clusters during shaping output interpretation (Issue #216).
@burg burg closed this Nov 19, 2012
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
@jgraham
Merge pull request servo#95 from Ms2ger/createTBody
2690fca 
Add a test for createTBody.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-platform/shaping I-wrong
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@burg
You can’t perform that action at this time.