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

Allow to use FreeTypeAbstraction fonts and render glyph by index #357

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Kolaru
Copy link

@Kolaru Kolaru commented Oct 3, 2022

These are small features that I am porting from CairoMakie so that they will be available to Luxor as well, to allow support for MathTeXEngine v0.5.

I do not know if I should add tests for those or what those tests should look like.

@cormullion
Copy link
Member

@lobingera How does this look to you?

@lobingera
Copy link
Contributor

Hmm, I lost track how we do things in julia today. Code looks reasonable and i think i have a local branch that does more or less the same, but i never saw libfreetext working correctly on all platforms.

Testing - just calling all functions - would be nice. Having a standard font available that loads in all platforms is a plus. Doing an example is the highest i can ask for.

@davibarreira
Copy link

This PR is greatly appreciated!

@ViralBShah
Copy link
Contributor

Please let me know if this should be merged? Also happy to give access to a couple of more folks to maintain the package.

@cormullion
Copy link
Member

Please let me know if this should be merged? Also happy to give access to a couple of more folks to maintain the package.

It would be great if Cairo.jl has occasional maintainers - it's pretty stable and unchanging generally.

I think Cairo.jl is still fairly central to the Julia ecosystem - probably needs releases and updates occasionally.

This PR probably needs some tests...

@ViralBShah
Copy link
Contributor

Good point. Maybe we should see if any of the folks who have made a couple of PRs to Cairo.jl are interested in stepping up for light maintenance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants