RTC NPM Resource Changed PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
13 Dec 2024

Summary of Changes

By using logical css properties and removing unused css we can remove the system-site-offline RTL css file. One less file to maintain!

Testing Instructions

Set site to offline in global configuration
Set site language to an RTL language such as persian
Rename the cassiopeia offline.php so that we are using the fallback system template's offline.php

As this involves removing a css file and rebuilding an scss file either use a prebuilt package or do a full npm ci

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

No visible change

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 brianteeman brianteeman - open - 13 Dec 2024
avatar brianteeman brianteeman - change - 13 Dec 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Dec 2024
Category Repository NPM Change Front End Templates (site)
avatar fgsw fgsw - test_item - 13 Dec 2024 - Tested successfully
avatar fgsw
fgsw - comment - 13 Dec 2024

I have tested this item ✅ successfully on 8d85c87

Untitled


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44615.
avatar chmst chmst - alter_testresult - 14 Dec 2024 - quy: Tested successfully
avatar alikon
alikon - comment - 16 Dec 2024

setting RTC cause it was successfully tested by @Quy too

avatar alikon alikon - change - 16 Dec 2024
Status Pending Ready to Commit
avatar alikon
alikon - comment - 16 Dec 2024

RTC


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

avatar rdeutz rdeutz - change - 7 Jan 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-01-07 09:30:21
Closed_By rdeutz
Labels Added: RTC NPM Resource Changed PR-5.3-dev
avatar rdeutz rdeutz - close - 7 Jan 2025
avatar rdeutz rdeutz - merge - 7 Jan 2025
avatar rdeutz
rdeutz - comment - 7 Jan 2025

Thanks

Add a Comment

Login with GitHub to post a comment