No Code Attached Yet
avatar Hackwar
Hackwar
27 Jan 2022

Try to compile the SCSS of the frontend template for example in media/templates/site/cassiopeia/scss/template.scss That is what we are shipping to our users with 4.1 and it does not compile, since it is a copy from the /build folder to the /media folder. However in the /build folder it is another level deeper than in the /media folder and thus for example the above file tries to include files by traveling one level to high in the filesystem.

avatar Hackwar Hackwar - open - 27 Jan 2022
avatar joomla-cms-bot joomla-cms-bot - change - 27 Jan 2022
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Jan 2022
avatar brianteeman
brianteeman - comment - 27 Jan 2022

How are you trying to compile it?

avatar Hackwar
Hackwar - comment - 27 Jan 2022

I've tried it with the code from #35403.

avatar richard67
richard67 - comment - 27 Jan 2022

@dgrammatiko Could you have a look on this issue? Thanks in advance.

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

The scss folder in the media/... is a deliverable and not the source thus they should NOT used (in conjunction with Joomla's own tools) for compiling the css files, for those you need to use the folder build/.... But anyways the PHP compiler should be abandoned as it's not a real replacement of the actual tools for compiling SCSS to CSS...

avatar Hackwar
Hackwar - comment - 27 Jan 2022

However we aren't talking about the PHP compiler, but about the SCSS we are going to ship with 4.1 and that SCSS right now is broken. You can not create an SCSS file which includes that template.scss and run npm to compile it. So please stay on topic.

avatar Hackwar
Hackwar - comment - 27 Jan 2022

If those files aren't meant to be compiled, then delete them now and don't deliver them with our release package.

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

You can, the source files are in the build folder...

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

If those files aren't meant to be compiled, then delete them now and don't deliver them with our release package.

Yes probably should be removed

avatar brianteeman
brianteeman - comment - 27 Jan 2022

You can, the source files are in the build folder...

That folder is not distributed

If those files aren't meant to be compiled, then delete them now and don't deliver them with our release package.

Yes probably should be removed

Irrespective of the phpcss thing having the scss for the templates is useful - assuming that the paths are fixed and they can be used.

@Hackwar raises a valid issue here

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

Irrespective of the phpcss thing having the scss for the templates is useful

Let me disagree here. Bootstrap 5.2 will have most of their components (I think their missing only the forms, but have;t checked lately) using CSS Variables. Meaning you will never-ever need to recompile Bootstrap... The whole SCSS precompiling is not needed anymore...
Here's what Bootstrap is doing regarding CSS Variables:

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

What I'm trying to say is JUST KILL SCSS It was a mistake

avatar brianteeman
brianteeman - comment - 27 Jan 2022

What I'm trying to say is JUST KILL SCSS It was a mistake

Maybe so but for at least the next 6 months we will be shipping it - so it should be usable

avatar dgrammatiko
dgrammatiko - comment - 27 Jan 2022

so it should be usable

Ok will try to do a pr this weekend

avatar dgrammatiko
dgrammatiko - comment - 28 Jan 2022

This can be closed

avatar richard67 richard67 - change - 28 Jan 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-01-28 10:33:36
Closed_By richard67
avatar richard67 richard67 - close - 28 Jan 2022
avatar richard67
richard67 - comment - 28 Jan 2022

Closing as having a pull request. Please test #36879 . Thanks in advance.

Add a Comment

Login with GitHub to post a comment