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

Convert NativeFontHandle to a wrapper and implement serde traits for it #896

Closed

Conversation

@jbg
Copy link

jbg commented Feb 17, 2017

See servo/servo#15607

Other related pull requests:


This change is Reviewable

@nox nox changed the title Convert NativeFontHandle to a wrapper and implement serde traits for it (Do not merge) Convert NativeFontHandle to a wrapper and implement serde traits for it Feb 17, 2017
@nox nox changed the title (Do not merge) Convert NativeFontHandle to a wrapper and implement serde traits for it Convert NativeFontHandle to a wrapper and implement serde traits for it Feb 18, 2017
@nox
Copy link
Member

nox commented Feb 18, 2017

Why do we need to create the font at all from the Deserialize trait? Can't we explicitly just receives and sends font names instead of font handles?

@glennw
Copy link
Member

glennw commented Feb 19, 2017

cc @jdm

@nox nox self-assigned this Mar 3, 2017
@nox
Copy link
Member

nox commented Mar 4, 2017

Actually, this is ok.

@jbg Could you rebase on master?

@glennw
Copy link
Member

glennw commented Mar 27, 2017

Closing for now due to conflicts and lack of activity. Feel free to re-open when updated.

@glennw glennw closed this Mar 27, 2017
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.