User tests: Successful: Unsuccessful:
Pull Request for Issue #46717 .
Adjust the code so that Fonts follow the logic of Modern/legacy templates
Follow the instruction of the issue #46717
Broken link is logged into the browser's console, no font preview
Works
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_templates |
I have tested this item ✅ successfully on 06fa1b2
For test 1:
Fonts located in /media/templates/site/cassiopeia/fonts
Font preview works correctly, no 404 errors in browser console
For test 2:
Fonts located in /templates/cassiopeia/fonts
404 error occurs (expected for modern templates)
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
bug
PR-5.4-dev
|
||
RTC
| Labels |
Added:
RTC
|
||
TL;DR The 2nd folder now works, but the 1st one no longer does. @dgrammatiko Could you please check? #46717 states The font is previewed in both tests.
❌ Final test before merge with JBT and used Ubuntu TTF font
mkdir media/templates/site/cassiopeia/fonts administrator/templates/atum/fonts media/templates/administrator/atum/fonts firsttemplates/cassiopeia/fonts dir, but not is the font only in media/templates/site/cassiopeia/fonts dir and 404 Not Found in JS console logadministrator/templates/atum/fonts, but not working if font is only in media/templates/administrator/atum/fonts foldermedia/templates/site/cassiopeia/fonts dir only is correctly rendered and no 404 Not Found anymoretemplates/cassiopeia/fonts dir is not rendered and 404 Not Foundmedia/templates/administrator/atum/fonts only is correctly rendered and no 404 Not Found anymoreadministrator/templates/atum/fonts dir only is not rendered and 404 Not FoundThat’s correct, the modern templates expect all the static assets to be served from the media folder (compatibility with the public folder and better security). In sort your test was correct and the results are the expected ones
As discussed yesterday in the maintainer meeting: Old templates load their assets from the template folder, new templates (inheritable) use the media folder, so the PR is fine and can be merged.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-01-29 05:08:45 |
| Closed_By | ⇒ | muhme |
Thank you @dgrammatiko for your contribution. Thank you @brianteeman and @krishnaGandhi11 for testing.
I have tested this item ✅ successfully on 06fa1b2
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.