CDPT-2264 Fix font assets, and debug logging typo. #788
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR includes 2 changes.
I've noticed that prior to the CPU spikes on FPM containers, in 5 cases there has been requests to font files that nginx could not find. This results in logs like:
The font files in question are not at
/dist/globals/fonts/moji-clarity.woff
but/dist/fonts/moji-clarity.woff
without globals in the name.I noticed that the path is correct in
https://intranet.justice.gov.uk/app/themes/clarity/dist/css/globals.css
but incorrct inhttps://intranet.justice.gov.uk/app/themes/clarity/dist/css/style.css
I also saw that
globals.css
andstyle.css
are both loaded on all frontend pages.The fix is to move the
@font-family
css statement tofonts.styl
, so that it is included only inglobals.css
and with the correct path.The 2nd fix is to debug logging, there was an error because I typed
document_serve
instead ofserve_document
. The former is a filter and the latter is an action. This has been addressed and the add_action parameters have also been tidied up.