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

font-rs? #325

Closed
leeoniya opened this issue Aug 3, 2016 · 4 comments
Closed

font-rs? #325

leeoniya opened this issue Aug 3, 2016 · 4 comments

Comments

@leeoniya
Copy link

@leeoniya leeoniya commented Aug 3, 2016

see servo/servo#12709

maybe i should have opened this here?

@glennw @pcwalton

@louisremi
Copy link

@louisremi louisremi commented Aug 12, 2016

As far as I understand, font-rs is a CPU based font renderer. @evanw has developped a GPU based glyph renderer, see https://medium.com/@evanwallace/easy-scalable-text-rendering-on-the-gpu-c3f4d782c5ac#.iuglb1e5v Which, as is, seems patent encumbered though : https://www.google.com/patents/US20070097123 :-/

@brandf
Copy link

@brandf brandf commented Aug 26, 2016

The loop/blinn technique from the patent is not the same as the stencil buffer technique that evanw describes in the medium article.

The stencil technique is similar to stencil shadows, which are used in many games, so I'd be surprised if it's encumbered.

@metajack
Copy link

@metajack metajack commented Aug 26, 2016

webrender already renders glyphs on the CPU, so font-rs may work well. I believe the plan is to make the font support a trait so that it can use whatever is provided. In Servo's case this is currently freetype or Core Text, and we were planning on adding DirectWrite. However, it could make sense to replace rasterization with font-rs if that has enough features.

cc @vvuk

@glennw
Copy link
Member

@glennw glennw commented Jun 29, 2017

We'll probably want to either stick with the default platform rasterizer, or use a GPU-specific solution.

@glennw glennw closed this Jun 29, 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.

None yet
5 participants
You can’t perform that action at this time.