Hm, no idea why Cypress need jQuery to run php 8.3 😄

Good work, what about renaming media_source to resources? Can the watch script also be executed for extensions only?

yes really nice work thanks fedir, I think media_source is ok, it's more clear that media_source will be media we know already, then resources to media.

what about renaming media_source to resources?

I think existing is good. But can rename to anything if we get consensus on new name 😄

Can the watch script also be executed for extensions only?

Yes, it watches only on specified extension (one or few):

npm run watch -- -n com_content

npm run watch -- -n templates/administrator/atum

# on few
npm run watch -- -n com_content,com_categories

Thanks Fedora 💚

Thanks Fedir 💚

This pull request has conflicts, please resolve those before we can evaluate the pull request.

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Would you mind updating the PR for me? I would really like to test it, and it would be easiest to compare if we were back up to date. :)

First testing with running npm ci - one after the other on the 6.1-dev branch and then with the PR and comparison of the output.

• installer.min.css - missing version on image
  
  it still works in installer.css
  Same with:
  client.css -> client.min.css

I will test it again thoroughly after the branch update.

I updated the PR.
And fixed "min" version. Thanks for testing.

hm, I think there still may be an old issue with removed copyright in CSS.
I totally forgot about that.

This pull request has conflicts, please resolve those before we can evaluate the pull request.

For removed copyright we have to edit comments from /** to /*! parcel-bundler/lightningcss#43 (comment)

Can probably do it automatically, will look on it later.

I thought today would be a good day to test, since the beta release is coming out and there probably shouldn't be so many merges, so that the test result is quite easy to evaluate.
I'll keep testing what I can and get back to you if I find anything.
For copyright issues, please let me know if you say I can test it. Until then, I will ignore this error.

media/vendor/altcha/css/altcha.css is minified, but there is no min.css.
However, min.css did not exist before either, as the CSS was delivered only unminified before.
Would be cool to have both.

For my understanding, should the media_source folder still be present in the build package?

concerns assets for media manager and workflow graph.

media/vendor/altcha/css/altcha.css is minified, but there is no min.css.

Yes, most of files from media/vendor/ comes from npm_modules, without modification.
altcha does not provide source for these file same for js file, there a few other vendors like that.
We can duplicate these file, but that does not really make sense.
I think it is fine as long as it works.

missing version in js but min.js is ok:

need to check

For my understanding, should the media_source folder still be present in the build package?

What do you mean?
com_media It just as any other asset, but with own builder.
and where from that screenshot?

Yes, most of files from media/vendor/ comes from npm_modules, without modification.
altcha does not provide source for these file same for js file, there a few other vendors like that.
We can duplicate these file, but that does not really make sense.
I think it is fine as long as it works.

With the new build script now we have a altcha.css but it is minified, before we had only a altcha.css but it was not minified. So now, with the new script, it is no longer just copied, as it was before. Perhaps then the file should be not .css but min.css? But in the end, it's not important to me at all. I just wanted to report it because I noticed it. :D

What do you mean?
com_media It just as any other asset, but with own builder.
and where from that screenshot?

The screenshot is a direct comparison of the media assets with the previous build script and then with the new one. These are the media assets that we will deliver in the media folder in the installable package.

• media/com_media/js/media-manager.js
  same in
• media/com_workflow/js/workflow-graph.js

Until now, we have not included the source files under build/media_source in the installable package. I think this will remain the case, so media_source will not be included in the final package. But then the path doesn't fit here, because it leads to nowhere. Basically, this is tricky because moving the files means that the files for workflow-graph and mediamanager are now no longer included. Or am I mistaken?

With the new build script now we have a altcha.css but it is minified, before we had only a altcha.css but it was not minified.

It maybe changed something on their side. The file is from node_modules and it is also minified
node_modules/altcha/dist_external/altcha.css on 6.1 branch. Just checked.

com_media

Now I see, it is some random Vue thing from its compiler, the file itself is not used.
All .vue files is compiled in to javascript. We can safely ignore that.

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@LadySolveig I fixed the stuff you found. The license comments and bootstrap version.

• @charset "UTF-8"; at the beginning of the .css and .min.css files is removed.

• // eslint-disable-next-line no-undef on first line is removed in file media/vendor/jquery/js/jquery-noconflict.js

• different order in css min.css - but seems ok to me, as far as I have seen it is now correct (alphabetically)

• all copyrights and all comments now seem to be preserved, also on min.css and also the comments within the doucment

• formatting seems to be different in css files (do we use here a different configuration???)

  • selectors split across multiple lines instead of one
  • different spacing2 instead of 4 / sometime reverse 4 instead of 2

• media/system/html/noxml.html different

  • -moz-radial-gradientand -webkit-radial-gradient seems missing - not sure what else
  

The .js files differ a lot, both in terms of formatting, but sometimes look completely different. :D

joomla.asset.json has a dfferent order, but we had this also before as far as I remember.

Thanks @Fedik ! 💚

joomla.asset.json has a dfferent order,

The order should be the same for every build now. Or it still different?
However it will be different from what we currently have in main branch.

@charset "UTF-8"; at the beginning of the .css and .min.css files is removed

Hm it should be there all the time, after last fix.
Any specific css for example?

// eslint-disable-next-line no-undef

That something useless now :)
jslint is fine

media/system/html/noxml.html

Before the PR css for the template files was not processed,
Now all CSS code uses same processing routine.

different order in

The compiler and its options is the same.
Well, should be fine :)

joomla.asset.json has a dfferent order,

The order should be the same for every build now. Or it still different? However it will be different from what we currently have in main branch.

That's cool :) You are right only differs from the previous we have in the branch 👍🏼 But if a ran it now multiple times it stays the same.

@charset "UTF-8"; at the beginning of the .css and .min.css files is removed

Hm it should be there all the time, after last fix. Any specific css for example?

Seems to be only the css and min.css files in the media/vendor folder,

Seems to be only the css and min.css files in the media/vendor folder,

Can be, the vendor files copied as they are. With few exceptions.

This pull request has conflicts, please resolve those before we can evaluate the pull request.

This pull request has been automatically rebased to 6.2-dev.

Hi Fedir - thank you for your great work. This PR is one the first we would like to bring into 6.2 - could you check for the conflicts and do clean manual rebase to 6.2? I have done some tests and it worked but got some discrepancies when trying to merge in my 6.2 branch. Thank you