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

Initial support for line heights calculations. #502

Closed
wants to merge 4 commits into from

Conversation

@jrfeenst
Copy link
Contributor

jrfeenst commented Jun 1, 2013

No description provided.

bors-servo and others added 3 commits May 29, 2013
See the explanation on the commit for more detail.

r? @metajack
…count but does align baselines based on the largest box.
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 0ff196d May 30, 2013

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

merging jrfeenst/servo/line-height = 0ff196d into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied May 30, 2013

jrfeenst/servo/line-height = 0ff196d merged ok, testing candidate = 1009e06

…es are expecting pixels, not points like the docs say. The glyph advance is also scaled slightly to match what is show in Firefox.
@jdm
Copy link
Member

jdm commented Jun 7, 2013

@jrfeenst This seems to need a rebase.

@jrfeenst
Copy link
Contributor Author

jrfeenst commented Jun 8, 2013

I'm still working on the font size issues. It seems fine on mac now, but the font metrics are all wrong on Linux. I'll reopen this once I've fixed everything and rebased.

@jrfeenst jrfeenst closed this Jun 8, 2013
glennw pushed a commit to glennw/servo that referenced this pull request Jan 16, 2017
Local rectangle clipping for transformed primitives

Finishes servo#427 implementation, now the text is cut evenly and not aligned to tile borders.
Also refactored `build_bounding_rect()` as well as a bit of `assign_prims_to_screen_tiles()`.

![textrun-comparison3](https://cloud.githubusercontent.com/assets/107301/19860234/5ce0c85a-9f5e-11e6-82f2-d72f19e4641a.png)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/502)
<!-- Reviewable:end -->
@jdm jdm mentioned this pull request Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.