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

TextRun creation must account for text render-ability by FontGroup fonts #177

Closed
burg opened this issue Nov 7, 2012 · 1 comment
Closed

TextRun creation must account for text render-ability by FontGroup fonts #177

burg opened this issue Nov 7, 2012 · 1 comment

Comments

@burg
Copy link

@burg burg commented Nov 7, 2012

If some text in a text run can't be rendered in the first FontGroup font (say, because of language or missing chars), then the next one must be used. This causes a text run split, because of the one Font per TextRun invariant.

The font group is not presently consulted. It should be built for each style context within an inline flow.

@burg
Copy link
Author

@burg burg commented Nov 12, 2012

The FontGroup::create_textrun method has been added, but it doesn't do anything complicated now. It just assigns the first Font instance in the FontGroup unconditionally.

ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
pcwalton added a commit to pcwalton/servo that referenced this issue Apr 8, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
pcwalton added a commit to pcwalton/servo that referenced this issue Apr 10, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
bors-servo pushed a commit that referenced this issue Apr 10, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Built on #5493.

Closes #177.
pcwalton added a commit to pcwalton/servo that referenced this issue Apr 10, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
bors-servo pushed a commit that referenced this issue Apr 14, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Built on #5493.

Closes #177.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5607)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Apr 22, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Built on #5493.

Closes #177.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5607)
<!-- Reviewable:end -->
pcwalton added a commit to pcwalton/servo that referenced this issue May 20, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
pcwalton added a commit to pcwalton/servo that referenced this issue May 22, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
pcwalton added a commit to pcwalton/servo that referenced this issue May 22, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
bors-servo pushed a commit that referenced this issue May 22, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Built on #5493.

Closes #177.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5607)
<!-- Reviewable:end -->
pcwalton added a commit to pcwalton/servo that referenced this issue May 23, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Closes servo#177.
bors-servo pushed a commit that referenced this issue May 23, 2015
This improves numerous pages, for example Wikipedia and Ars Technica.

Built on #5493.

Closes #177.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5607)
<!-- Reviewable:end -->
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
(This allows correct clipping for these cases when transforms are involved).

This is a partial fix for servo#177 (the other part relies on changes to layout).
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
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.

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