Skip to content

fix(text): add automatic glyph texture padding#302

Merged
han4wluc merged 4 commits into
mainfrom
add-d-l-overhang-vt
Jun 19, 2026
Merged

fix(text): add automatic glyph texture padding#302
han4wluc merged 4 commits into
mainfrom
add-d-l-overhang-vt

Conversation

@han4wluc

@han4wluc han4wluc commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add automatic text texture padding based on font size so overhanging glyphs are not clipped without caller-provided textStyle.padding
  • keep explicit padding and shadow padding supported by using the largest required texture reserve
  • update the Mr De Haviland D/L VT specs to verify the default user path without explicit padding
  • bump package version to 1.23.6

Testing

  • bunx vitest run spec/util/toPixiTextStyle.spec.js spec/elements/textHoverLayout.spec.js
  • bun run vt:generate
  • bun run vt:report (703/703, 0 mismatches)
  • bunx prettier --check src spec/util/toPixiTextStyle.spec.js spec/elements/textHoverLayout.spec.js vt/specs/textbasic/mr-de-haviland-correct-d-l-overhang.yaml vt/specs/textbasic/mr-de-haviland-leading-l-overhang.yaml package.json
  • bunx vitest run (83 files, 585 tests)
  • pre-push hook: bunx prettier --check src
  • pre-push hook: vitest run (83 files, 585 tests)

@han4wluc han4wluc changed the title test(vt): add d and l overhang reference fix(text): add automatic glyph texture padding Jun 19, 2026
@han4wluc han4wluc merged commit 1fed141 into main Jun 19, 2026
2 checks passed
@han4wluc han4wluc deleted the add-d-l-overhang-vt branch June 19, 2026 15:10
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.

1 participant