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

Build a font group by doing private matching according to CSS spec #178

Closed
burg opened this issue Nov 7, 2012 · 2 comments
Closed

Build a font group by doing private matching according to CSS spec #178

burg opened this issue Nov 7, 2012 · 2 comments

Comments

@burg
Copy link

@burg burg commented Nov 7, 2012

FontContext::create_font_group needs to consult the Font list, available families, and so on to map a FontStyle to concrete Font instances.

Tasks

  • (A) Implement FontList and FontListHandle, which are the generic and platform-specific machinery for populating a list of font families.
  • (B) Implement population of FontFamily and FontEntry.
  • (C) Implement basic font matching for 1-face and 4-face font families.
  • (D) Hook up the matching API to create_font_group.
  • (E) Cache FontGroup instances.
@burg
Copy link
Author

@burg burg commented Nov 7, 2012

It seems pretty clear that FontContext should do the heavy lifting here, since it could cache FontGroup instances, and almost all of the data needed to do matching is owned by FontContext.

@burg
Copy link
Author

@burg burg commented Nov 12, 2012

Initial work done. Followup bugs: #194, #193, #192, #190, #189, #188, #186.

@burg burg closed this Nov 12, 2012
burg pushed a commit that referenced this issue Nov 12, 2012
add FontGroup construction and interaction with TextRun.

Closes #178.
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
Added basic set of HTML Imports test.
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
(Clip rect is not in the same space as item rect during batch building).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

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