Overview

Perhaps it's premature to report these kinds of bugs since there is a lot changing right now, but will do anyway...

  1. Installed XB per these instructions.
  2. Look at logs
  3. See 404 for /assets/Inter-VariableFont_slnt_wght.ttf

Proposed resolution

Debug and fix ;)

User interface changes

I didn't see anything amiss in the UI... just the logs.

Issue fork experience_builder-3470552

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kristen pol created an issue. See original summary.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: Page not found for /assets/Inter-VariableFont_slnt_wght.ttf » Use system font stack rather than loading the custom "Inter" font
Priority: Normal » Critical
Issue tags: +front-end performance, +licensing

💯 — I've seen this happen too, but somehow currently cannot reproduce this.

We shouldn't be loading a custom font. We should use a font stack that uses system fonts.

Per https://github.com/FortAwesome/Font-Awesome/issues/1124, it looks like this is a d.o terms of service violation too, given that the Inter font license (https://github.com/rsms/inter/blob/master/LICENSE.txt) appears to not be GPL-compatible.

@lauriii, @jessebaker and I discussed this a week ago, but apparently @jessebaker didn't file an issue for it: https://www.drupal.org/project/issues/experience_builder?text=font+stack...

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented

It seems that Radix Themes is using system fonts by default: https://www.radix-ui.com/themes/docs/theme/typography#font-family. I wonder if we could just remove our overrides for the typography and it would work as expected?

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Issue tags: +Novice, +CSS

Per #3 🤞 🤠

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Parent issue: » #3450592: [META] Front-end Kanban issue tracker

omkar-pd made their first commit to this issue’s fork.

omkar-pd opened merge request !217

omkar-pd’s picture

omkar-pd commented
Status: Active » Needs review
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: Use system font stack rather than loading the custom "Inter" font » Follow-up for #3456084: Use system font stack rather than loading the custom "Inter" font
Assigned: Unassigned » hooroomoo
Status: Needs review » Reviewed & tested by the community
Related issues: +#3456084: Add initial implementation of primary menu

Thanks @omkar-pd!

This font was introduced by @hooroomoo in #3456084: Add initial implementation of primary menu. Assigning to them for final review to ensure we're not overlooking something.

hooroomoo made their first commit to this issue’s fork.

  • hooroomoo committed 7cb3623d on 0.x authored by omkar-pd 
    Issue #3470552 by omkar-pd, Wim Leers, kristen pol, lauriii: Follow-up...
hooroomoo’s picture

hooroomoo
they/them
Location nyc🗽
commented
Assigned: hooroomoo » Unassigned
Status: Reviewed & tested by the community » Fixed

Good to have those 404s gone now, thanks!

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented

assign credit to hooroomoo

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.