Skip to content

Exclude disabled fonts from builder integrations - #21

Open
Jan-CoreBunch wants to merge 1 commit into
CoreBunch:mainfrom
Jan-CoreBunch:fix/disabled-font-filtering
Open

Exclude disabled fonts from builder integrations#21
Jan-CoreBunch wants to merge 1 commit into
CoreBunch:mainfrom
Jan-CoreBunch:fix/disabled-font-filtering

Conversation

@Jan-CoreBunch

Copy link
Copy Markdown
Contributor

Summary

  • centralize filtering of enabled fonts from the selected preset
  • expose only enabled font families to the Oxygen Classic dropdown
  • omit disabled font CSS from Oxygen and Bricks inline styles
  • preserve disabled font configuration and uploaded files

Validation

  • PHPUnit: 32 tests, 59 assertions
  • project PHPCS checks pass for the new helper and regression tests
  • PHP syntax and diff checks pass
  • manually verified Oxygen Classic opens with all CF fonts disabled, no CF fonts appear in the dropdown, no WOFF2 requests are made, and uploaded files remain present

Follow-up

This branch is independently based on main. The enabled-to-disabled toggle cycle should receive a final end-to-end retest after #20 is merged.

Related to #19

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