PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar astridx
astridx
31 Oct 2023

If I'm not mistaking it's more straightforward to create a child template with this change.

avatar astridx astridx - open - 31 Oct 2023
avatar astridx astridx - change - 31 Oct 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Oct 2023
Category Front End Templates (site)
avatar astridx astridx - change - 31 Oct 2023
Title
Update index.php
Update index.php of Cassiopeia, so that it is easier to use a child of Cassiopeia
avatar astridx astridx - edited - 31 Oct 2023
avatar richard67
richard67 - comment - 1 Nov 2023

We should keep things consistent which are the same in index.php, error.php and maybe also component.php and offline.php. I am almost sure that the code changed here does not only exist in index.php but also in some of the other mentioned files.

avatar astridx astridx - change - 1 Nov 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-11-01 17:39:09
Closed_By astridx
Labels Added: PR-5.0-dev
avatar astridx
astridx - comment - 1 Nov 2023

This breaks the cascade, ie if the file x.css doesn't exist in the child and it exists in the parent then load it from there. I haven't tested but I think the code here should be

$wa->registerAndUseStyle($assetColorName, 'global/' . $paramsColorName . '.css');

(maybe it needs an attribute ['relative' => true] to work correctly, but it should be relative path so the cascade is not broken)

Oops, you're right. I hadn't considered that not all child templates overwrite all SCSS/CSS data.

avatar astridx astridx - close - 1 Nov 2023
avatar dgrammatiko
dgrammatiko - comment - 1 Nov 2023

@astridx this is not an invalid issue, the solution needs just some changes

Add a Comment

Login with GitHub to post a comment