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

Avoid newlines between 2 tspan elements #114

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

wouterj
Copy link
Contributor

@wouterj wouterj commented Oct 8, 2024

This whitespace messes up word alignment when placed in between 2 tspan elements, causes misalignment when rendering more than 2 lines of tap text.

It is pretty much an edge-case, but I'm using a custom keymap as legend for the real keymap. And some of the legend labels have more than 2 words.

Before After
image image

This whitespace messes up word alignment when placed in between 2 tspan
elements. This causes misalignment when rendering more than 2 lines of
tap text.
@caksoylar caksoylar merged commit 9190216 into caksoylar:main Oct 8, 2024
1 check passed
@caksoylar
Copy link
Owner

This has been bugging me for sometime but I hadn't figured it out why it was happening, thank you!

@wouterj wouterj deleted the fix-space-multiline-tap branch October 8, 2024 20:15
@wouterj
Copy link
Contributor Author

wouterj commented Oct 8, 2024

Thanks for the quick merge (and great tool!)

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.

2 participants