RTC bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
25 Jan 2026

Pull Request for Issue #46717 .

Summary of Changes

Adjust the code so that Fonts follow the logic of Modern/legacy templates

Testing Instructions

Follow the instruction of the issue #46717

Actual result BEFORE applying this Pull Request

Broken link is logged into the browser's console, no font preview

Expected result AFTER applying this Pull Request

Works

Screenshot 2026-01-25 at 2 12 38 PM

Link to documentations

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

avatar dgrammatiko dgrammatiko - open - 25 Jan 2026
avatar dgrammatiko dgrammatiko - change - 25 Jan 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Jan 2026
Category Administration com_templates
avatar brianteeman brianteeman - test_item - 25 Jan 2026 - Tested successfully
avatar brianteeman
brianteeman - comment - 25 Jan 2026

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.

avatar krishnaGandhi11 krishnaGandhi11 - test_item - 25 Jan 2026 - Tested successfully
avatar krishnaGandhi11
krishnaGandhi11 - comment - 25 Jan 2026

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)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.

avatar richard67 richard67 - change - 26 Jan 2026
Status Pending Ready to Commit
Labels Added: bug PR-5.4-dev
avatar richard67
richard67 - comment - 26 Jan 2026

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.

avatar richard67 richard67 - change - 26 Jan 2026
Labels Added: RTC
avatar muhme
muhme - comment - 26 Jan 2026

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

  • Executing mkdir media/templates/site/cassiopeia/fonts administrator/templates/atum/fonts media/templates/administrator/atum/fonts first
  • Before PR, seen rendering is working in templates/cassiopeia/fonts dir, but not is the font only in media/templates/site/cassiopeia/fonts dir and 404 Not Found in JS console log
  • Same, working for administrator/templates/atum/fonts, but not working if font is only in media/templates/administrator/atum/fonts folder
  • Applied PR with Patch Tester
    • ✅ Font from media/templates/site/cassiopeia/fonts dir only is correctly rendered and no 404 Not Found anymore
    • ❌ Font from templates/cassiopeia/fonts dir is not rendered and 404 Not Found
    • ✅ Font from media/templates/administrator/atum/fonts only is correctly rendered and no 404 Not Found anymore
    • ❌ Font from administrator/templates/atum/fonts dir only is not rendered and 404 Not Found
avatar dgrammatiko
dgrammatiko - comment - 26 Jan 2026

That’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

avatar muhme
muhme - comment - 29 Jan 2026

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.

avatar muhme muhme - change - 29 Jan 2026
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
avatar muhme muhme - close - 29 Jan 2026
avatar muhme muhme - merge - 29 Jan 2026
avatar muhme
muhme - comment - 29 Jan 2026

Thank you @dgrammatiko for your contribution. Thank you @brianteeman and @krishnaGandhi11 for testing.

Add a Comment

Login with GitHub to post a comment