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

Upgrade prawn-svg #2469

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Conversation

henry-hsieh
Copy link
Contributor

@henry-hsieh henry-hsieh commented Dec 17, 2023

Hello,

I'm using bytefield-svg which is heavily using the dominant-baseline of SVG attribute to make the texts center-aligned. However, prawn-svg cannot recognize the attribute. Therefore, I made a PR to support the feature, and it has been merged into main branch successfully. Moreover, the newest release 0.33.0 has included the feature. I hope the asciidoctor-pdf can upgrade it as well.

Thank you!

@mojavelinux
Copy link
Member

Three of the visual tests are failing. However, when I inspect the changes manually, I cannot perceive any difference, so likely it is some sort of rounding difference. Therefore, I'll update the expected output files and proceed.

adds support for the dominant-baseline SVG attribute to make text center-aligned
@henry-hsieh
Copy link
Contributor Author

OK. Thanks for your help!

@mojavelinux mojavelinux merged commit cd71092 into asciidoctor:main Dec 18, 2023
15 checks passed
mojavelinux pushed a commit that referenced this pull request Dec 18, 2023
adds support for the dominant-baseline SVG attribute to make text center-aligned
@henry-hsieh
Copy link
Contributor Author

Hi @mojavelinux , is it possible to upgrade the version of the release recently? I want to update the dependency of my project. Thank you!

@mojavelinux
Copy link
Member

I don't have any plans for an immediate release, but once I come back around to the project and make a few other updates, I'll push out a release.

@henry-hsieh
Copy link
Contributor Author

OK. I'll wait for the release. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants