Skip to content

Update the default feature registry to include palt.#310

Merged
garretrieger merged 2 commits into
w3c:mainfrom
terkel:palt-default-feature
Jun 5, 2026
Merged

Update the default feature registry to include palt.#310
garretrieger merged 2 commits into
w3c:mainfrom
terkel:palt-default-feature

Conversation

@terkel
Copy link
Copy Markdown
Member

@terkel terkel commented Jun 1, 2026

Add palt (Proportional Alternate Widths) to the default feature registry by flipping its Is Default flag from 0 to 1.

palt is now included in HarfBuzz's default subset layout features (harfbuzz#6007, resolving harfbuzz#6006), and feature-registry.csv is derived from hb-subset-input.cc. This keeps the registry in sync with its source of truth, mirroring #306 which added halt/vhal. palt is one of the most commonly used optional features in Japanese typography, so subsetting silently dropping it is a real problem in practice.

I haven't regenerated Overview.html — it looks like the HTML rebuilds are usually done separately by maintainers (e.g. #309, #292). Happy to regenerate it in this PR if you'd prefer.

palt (Proportional Alternate Widths) is now included in harfbuzz's
default subset layout features (harfbuzz/harfbuzz#6007),
and feature-registry.csv is derived from hb-subset-input.cc. This mirrors
the earlier addition of halt and vhal in w3c#306.
@terkel terkel requested review from garretrieger June 1, 2026 04:33
@garretrieger
Copy link
Copy Markdown
Contributor

I haven't regenerated Overview.html — it looks like the HTML rebuilds are usually done separately by maintainers (e.g. #309, #292). Happy to regenerate it in this PR if you'd prefer.

Yes please regenerate Overview.html if possible. You'll just need to have an up to date copy of bikeshed installed.

…try.

Built with Bikeshed 7.0.12. The diff beyond the new palt registry row is
boilerplate regeneration from the newer Bikeshed (generator/revision
metadata, table-counter and print CSS, alphabetical class ordering).
@terkel
Copy link
Copy Markdown
Member Author

terkel commented Jun 4, 2026

@garretrieger Thanks! I rebuilt Overview.html with the latest Bikeshed (7.0.12), but it produced ~400 lines of unrelated churn (reflowed wrapping, alphabetized class attributes, new boilerplate CSS) on top of the palt change. How would you like to handle that — is the boilerplate refresh fine to land, or should I pin to an older Bikeshed to keep the diff to just the palt row?

Copy link
Copy Markdown
Contributor

@garretrieger garretrieger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

html changes from bikeshed update look good to me.

@garretrieger garretrieger merged commit 1bb61e0 into w3c:main Jun 5, 2026
1 check passed
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