Create a new CSS file here:
administrator/templates/atum/css/vendor/choicesjs/choices.css
Inside this CSS file, add:
html { display: none }
Then go to your Article Manager
The page should be blank
The CSS file is not overridden
This works perfectly fine with JHtml::stylesheet
as it checks if the CSS file exists at template level first.
Labels |
Added:
?
|
That not a bug, that is a feature ;)
Well, it was planed like that. So /media/vendor should be overridden only via AssetManager
check comment #26077 (comment)
But that not hard to change, if it need.
Which directory do we override the .json file?
no no,
it not like that, no need override "file", but definition of asset.
Take a look https://docs.joomla.org/J4.x:Web_Assets
json
just a way to register a bunch of assets in the registry.
Example the asset of /media/vendor defined in media/vendor/joomla.asset.json
. To override one of its(vendor) assets on template level you can define your asset with the same name in your template joomla.asset.json
. Because the template json
will be loaded last, it will override previous assets (from vendor json) by its name, if there the same name.
Or programaticaly $AssetRegistry->add(new WebAssetItem('foobar', ['js' => ['new/path/to/file.js']]));
@wilsonge what do you think about the issue?
The main idea was that all scripts/styles that have defined Assets should be loaded and overridden through AssetManager. Since it is a new thing and /media/vendor
also "new" folder in j4, I thought it will be safe.
But as we see people can still use HTMLHelper to add the files from /media/vendor
that can lead to duplication when the file overridden on template via HTMLHelper.
From other side, if we allow override js/css files in /media/vendor
by AssetManager itself (by HTMLHelper while attaching), it will produce the same mistake, example when someone override asset via AssetManager, and another one use HTMLHelper to load old file.
That kind of this issue #25309
We can make some kind of requirement (or statement) that all assets should be managed through AssetManager only. But we cannot prevent to use HTMLHelper directly, because that also should work for compatibility.
@Fedik Using the following works:
{
"package": "choices.js",
"name": "choicesjs",
"type": "style",
"uri": "administrator/templates/xxxxxx/css/vendor/choicesjs/choices.min.css"
},
{
"package": "choices.js",
"name": "choicesjs",
"type": "preset",
"dependencies": [
"choicesjs#style",
"choicesjs#script"
]
}
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-11-14 12:10:12 |
Closed_By | ⇒ | C-Lodder |
@C-Lodder the uri can be simpler vendor/choicesjs/choices.min.css
A bit more explanation.
WebAsset use HTMLHelper in next way:
uri
absolute, it does not search override:HTMLHelper::_('script', 'blabla.js', ['relative' => false])
uri
not absolute, it use it as relative to /media, and search for overrides:HTMLHelper::_('script', 'blabla.js', ['relative' => true])
ah nice, thanks :)
@Fedik #25775 doesn't solve this either