User tests: Successful: Unsuccessful:
Pull Request for Issue # .
npm lint:js -- --fix
and committed the changes
nomodule
it's using the defer attribute, thus adding the listener for DOMContentLoaded
is pure overhead. At that point the DOM is already there fully parsed. This needs to be applied ON EVERY SCRIPTOn a fresh installation of Joomla (ie git clone
) do the composer install and then the npm install. Copy the media folder somewhere in your hardisk. (check the time needed)
Now clone this PR and do both the composer and npm installations. Once done do a folder compare of the current media folder against the folder that you saved in the previous step. You should have the same files apart from the build/media_source/com_content/js/articles-status.es6.js
which will have different content (expected).
Compare the times and do a brief check that both front end and back still work as expected
NO
@HLeithner have somebody merge this so I could do the changes in the v5.0. Thanks
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_media NPM Change JavaScript Repository |
Labels |
Added:
NPM Resource Changed
?
|
the change to image.vue is not correct. The span element is not self closing. Both parts are mandatory.
On your repo (4.2) try npm run lint:js -- --fix
, you will get the same. The HTML inside the .vue files is totally fine with this change and actually it's the recommendation, (it's the v- part that makes the difference)
btw here's the eslint rule and the explanation: https://eslint.vuejs.org/rules/html-self-closing.html
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-07-12 22:05:37 |
Closed_By | ⇒ | dgrammatiko |
Status | Closed | ⇒ | New |
Closed_Date | 2022-07-12 22:05:37 | ⇒ | |
Closed_By | dgrammatiko | ⇒ |
Status | New | ⇒ | Pending |
Category | Administration com_media NPM Change JavaScript Repository | ⇒ | JavaScript Repository NPM Change |
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-09-01 10:13:58 |
Closed_By | ⇒ | dgrammatiko | |
Labels |
Added:
PR-4.3-dev
Removed: ? |
the change to image.vue is not correct. The span element is not self closing. Both parts are mandatory.