alchemist icon indicating copy to clipboard operation
alchemist copied to clipboard

fix: loading fonts from other packages

Open krispypen opened this issue 1 year ago โ€ข 1 comments

Description

Font rendering did not work when fonts are from another package. We are using a separate package for our styleguide including assets, textstyles and fonts... This change makes it possible.

Type of Change

  • [ ] โœจ New feature (non-breaking change which adds functionality)
  • [x] ๐Ÿ› ๏ธ Bug fix (non-breaking change which fixes an issue)
  • [ ] โŒ Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] ๐Ÿงน Code refactor
  • [ ] โœ… Build configuration change
  • [ ] ๐Ÿ“ Documentation
  • [ ] ๐Ÿ—‘๏ธ Chore

krispypen avatar Mar 01 '24 10:03 krispypen

This pull request has been automatically closed because it has not been updated in the last month. :sleepy:

If you still need this change, you can reopen it.

Thanks for helping keep our house in order!

bmt-github-policybot avatar Apr 16 '24 09:04 bmt-github-policybot

Thank you!

TimmyChannel avatar Aug 02 '24 07:08 TimmyChannel

@krispypen mind pushing an empty commit to your branch? We updated CI configuration on our end which seems to have reset the statuses for build and pana and AFAIK Github UI has no way of kicking these off separately/manually. I'll re-approve upon success and we can get this merged.

btrautmann avatar Aug 07 '24 18:08 btrautmann

hey @btrautmann I pushed an empty commit, not sure if it's actually triggering something now

krispypen avatar Aug 08 '24 12:08 krispypen

hey @btrautmann I pushed an empty commit, not sure if it's actually triggering something now

Looks like it worked, we just have one failing test

btrautmann avatar Aug 08 '24 13:08 btrautmann

Hi everyone, can we get this in, pls?

pedromassango avatar Sep 08 '24 21:09 pedromassango

Hi everyone, can we get this in, pls?

@krispypen just needs to patch/address the one failing test

btrautmann avatar Sep 09 '24 14:09 btrautmann

oh, having a look sorry ๐Ÿฅน

krispypen avatar Sep 09 '24 14:09 krispypen

I did the change and locally it works, but it looks like someone needs to approve before the test runs.

krispypen avatar Sep 09 '24 14:09 krispypen