Language Change NPM Resource Changed PR-6.0-dev Failure

User tests: Successful: Unsuccessful:

avatar drmenzelit
drmenzelit
18 Jun 2025

Pull Request for Issue # .

Alternative to PR #45539

Thank you @brianteeman for the ideas exchange.

Still a draft, because it can be improved and I'm not sure, if I should modify the database on updates (insert the new parameters in the template style).

Summary of Changes

Added new parameter into Cassiopeia to change colors and font sizes

Testing Instructions

Install Joomla (use the package from this PR) and select "Custom" on "Colour Theme" check the new parameters.

New fields will appear. Use the default values or change the colors / sizes as you like.

Actual result BEFORE applying this Pull Request

Cassiopeia without customization for colors and font sizes

Expected result AFTER applying this Pull Request

New parameters to customize Cassiopeia
grafik

(It would be nice, if we can change the sample data images too)
grafik

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 drmenzelit drmenzelit - open - 18 Jun 2025
avatar drmenzelit drmenzelit - change - 18 Jun 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jun 2025
Category Repository NPM Change SQL Installation Postgresql Language & Strings Front End Templates (site)
avatar drmenzelit drmenzelit - change - 18 Jun 2025
Labels Added: Language Change NPM Resource Changed PR-6.0-dev
avatar brianteeman
brianteeman - comment - 18 Jun 2025

Not exactly what I was thinking and now that I see it - it makes no sense to do it this way as the font sizes should not be related to the colour scheme.

My idea was to keep the two new fieldsets (as in the other proposal) but only to display them if you have set a flag in the templatedetails.xml

avatar N6REJ
N6REJ - comment - 18 Jun 2025

I think requiring users to touch the .xml is a bad idea. Thats a dev feature not user feature.

avatar brianteeman
brianteeman - comment - 19 Jun 2025

I think requiring users to touch the .xml is a bad idea. Thats a dev feature not user feature.

That's not what I said

avatar richard67
richard67 - comment - 19 Jun 2025

I'm not sure, if I should modify the database on updates (insert the new parameters in the template style).

@drmenzelit In past (before J5) we did not do that because we had to use risky string modifications for the params, but since 5.0 we can use json functions to add values, so we could do that with an update SQL script, e.g. "6.0.0-2025-06-19.sql". If you want I can make a PR for your branch with that.

avatar richard67
richard67 - comment - 19 Jun 2025

@drmenzelit What I don't understand yet is why we have the params in both the extensions table and the template styles table for the Cassiopeia template.
For Atum we have emty params in the extensions table and have only params in the template styles table.
Maybe in case of Cassiopeia it's a remainder from early development and can be cleaned up in the extensions table?

avatar Fedik
Fedik - comment - 19 Jun 2025

For front-end template:
We should not modify existing template under any circumstances, unless there a bug. It will break many exiting installations that uses this template.

There 2 options:

  • new style: but only when it is compatible with template file, and both old and new style will work without extra work from User side.
  • new template

For back-end template:
We can just override existing template parameters with new, and call it a "new". :neckbeard:

avatar Fedik
Fedik - comment - 19 Jun 2025

UPD.

We should not modify existing template

I meant the "template parameters"

avatar brianteeman
brianteeman - comment - 19 Jun 2025

We can add the new fieldsets to cassiopeia with the default option for them to be disabled and then only enabled by the user. This would not be changing any existing paramaters. There would no need for the db to be updated. Only when a user enables the fields and saves will any changes be written to the db.

image

avatar Fedik
Fedik - comment - 19 Jun 2025

That also can work, yes.

But for new installations it need to be enabled somehow :)

avatar drmenzelit
drmenzelit - comment - 19 Jun 2025

@richard67

If you want I can make a PR for your branch with that.

What I don't understand yet is why we have the params in both the extensions table and the template styles table for the Cassiopeia template.

When we finally find the best way to modify the template, I will be grateful if you can help with the DB part

@Fedik

We should not modify existing template under any circumstances, unless there a bug.

We have introduced changes in the template already (e.g. font schema).

avatar drmenzelit drmenzelit - change - 19 Jun 2025
The description was changed
avatar drmenzelit drmenzelit - edited - 19 Jun 2025

Add a Comment

Login with GitHub to post a comment