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

fix: font rule #860

Merged
merged 154 commits into from Dec 21, 2021
Merged

fix: font rule #860

merged 154 commits into from Dec 21, 2021

Conversation

kellymears
Copy link
Member

@retlehs you are better at frontend than me. does this seem right to you?

Type of change

  • PATCH: bugfix

Dependencies added

  • none

Details

Fonts are currently incorrectly set to use the resolve-url loader.

This change loads fonts using the Webpack 5 asset type.

  • Fonts under 50kb are inlined
  • Fonts over 50kb are emitted as a separate asset and referenced with url()

ref #822

renovate-bot and others added 30 commits November 25, 2021 11:10
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
- adds sync filter: `'config.override'` (`build` namespace is reserved for compiler)
- modifies`bud.config` to pass built config to `bud.hook.on('config.override')` event.
	- this happens directly before `bud.build` verifies the configuration (strips `undefined` and `null` values)
- `bud.override` is just an alias for `bud.config`.
- fix: `hooks.filter` and `hooks.promised` will now tap the function passed to them if the supplied value is a `CallableFunction` type
Co-authored-by: Renovate Bot <bot@renovateapp.com>
@kellymears kellymears changed the base branch from main to next December 19, 2021 04:01
@kellymears kellymears self-assigned this Dec 19, 2021
Base automatically changed from next to main December 19, 2021 08:06
@QWp6t QWp6t changed the base branch from main to next December 19, 2021 08:29
@kellymears kellymears linked an issue Dec 21, 2021 that may be closed by this pull request
5 tasks
@kellymears kellymears added this to the 5.0.1 milestone Dec 21, 2021
@kellymears kellymears merged commit c65ce16 into next Dec 21, 2021
@kellymears kellymears deleted the fix-822-font-assets branch December 21, 2021 20:16
@kellymears kellymears mentioned this pull request Dec 22, 2021
3 tasks
@kellymears kellymears added this to next in Project releases Dec 30, 2021
@kellymears kellymears mentioned this pull request Jan 4, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Sage: url() in @font-face declaration doesn't work
3 participants