Problem with fontello resources with 1.4.3 version

[cekk]

I've updated my Plone 5.2.4 site to the new 5.2.5 and made the upgrade-step.

The site is published as "http://my-site.com/api" (with /api because it's Volto's backend).
All resources are loaded correctly (with /api/), except fontello ones: they don't have /api in the url and are loaded on a wrong context (that is the the Volto instance).

I think that could be related to recent changes made here.

Could be solved removing the "/"? Something like this:

@fontsPrefix: '++plone++static/fonts/';

[cekk]

@agitator could have the answer ;)

[agitator]

Moved the fontello fonts to their own bundle, to not include them inlined in every bundle.
Not sure how that should affect volto...?

[nzambello]

Volto is not affected, but a Plone site used as Volto backend is published under /api
Fontello resources are loaded as /...... so the browser is asking to Volto something that just doesn't exist.

[agitator]

I recall that I had to play around to make it work for dev and production... using / did solve it.
Not sure what your setup exactly looks like...should we discuss it on discord #classic-ui?
If those paths can't be resolved in your case, maybe inline that font into the css?

[agitator]

@cekk as we discussed ... best way would be to do a manual bundle... those font bundles won't change anyway

• don't use plone-compile for that
• create a little compile-script (probably just a one liner) to inline the fonts
• add some comments about the process into the readme

Only downside is that all types (ttf/eot/...) of that webfont are loaded then

cc @thet

[sneridagh]

@agitator so the upgrade step is broken? do the vanilla installation work?

[sneridagh]

@agitator at first I couldn't get a full grasp of the thing, even when we talked about it, but I think now I did:
Login at: https://6.demo.plone.org/api

(it's a 200 because Volto or whatever it's in the root answers)

Regardless that's a Volto in the root level, which it's not relevant in the issue (Volto simply does not care), this will break on any site that does not have an instance in the root level, thus, won't be able to deliver the resource properly.

Isn't it more sensible serve and get the assets relatively (like @cekk suggested?).

[erral]

This is because of this commit:

0228cd7

Here we are requesting the font to be loaded from the root of the site.

[agitator]

create a pull request without the "/" I'll happily will test it out ;-)

[erral]

Done, I just revert the previous commit #168

[thet]

Fixed via: #168

[erral]

After the revert of the fix we should reopen this to keep track of it.

[cekk]

@thet i'm confused.

#172 should be the pr that generated 1.4.4 version, right?
But if i try to download that from pypi, i see that all fontello compiled resources still have "/++plone++static" and not "++plone++static"

[thet]

Fixed by: #207