? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
2 Feb 2019

Pull Request for Issue # .

Summary of Changes

  • Removed the dictated jQuery.js and Bootstrap.js from the backend template.
  • Reason: almost ALL interactive components for Bootstrap are held by a PHP helper (HTMLHelper::_('bootstrap.something'...)) therefor there is zero benefit forcing these scripts in the index.php of the template. Also we've (as the Javascript team) put a huge amount of work to convert everything to vanilla javascript and quite frankly by forcing jQuery always makes all that work redundant.

Testing Instructions

Check that all pages in the backend still work

Expected result

No change

Actual result

Documentation Changes Required

No. jQuery and Bootstrap.js are still supported and can be loaded whenever they're needed but they're not forced anymore

avatar dgrammatiko dgrammatiko - open - 2 Feb 2019
avatar dgrammatiko dgrammatiko - change - 2 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Feb 2019
Category Administration com_cpanel com_media Modules Templates (admin)
avatar dgrammatiko dgrammatiko - change - 2 Feb 2019
Labels Added: ?
5e92152 2 Feb 2019 avatar dgrammatiko sort
avatar joomla-cms-bot joomla-cms-bot - change - 22 Feb 2019
Category Administration com_cpanel com_media Modules Templates (admin) Administration com_cpanel Modules Templates (admin)
avatar wilsonge wilsonge - change - 22 Feb 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-22 15:14:07
Closed_By wilsonge
avatar wilsonge wilsonge - close - 22 Feb 2019
avatar wilsonge wilsonge - merge - 22 Feb 2019
avatar wilsonge
wilsonge - comment - 22 Feb 2019

Just for the record IF we move to BS5 before stable (and therefore don't have to load jquery) I might reconsider this and revert back to loading bs5.js with the BACKEND template (not the frontend if/when we sort that out)

avatar dgrammatiko
dgrammatiko - comment - 22 Feb 2019

@wilsonge but the work on the custom elements was primarily to fix the a11y issues of Bootstrap. The vanilla js was a side effect. So falling back to BS is just blindly erase all the hard work done on the accessibility so far...

avatar dgrammatiko
dgrammatiko - comment - 8 Jan 2020

@Fedik did you just undid this PR with the changes in the assets manager?

avatar Fedik
Fedik - comment - 8 Jan 2020

what changes? :)
anything can be, there was many changes

avatar dgrammatiko
dgrammatiko - comment - 8 Jan 2020

You switched back to forcing BS js on the index.php

avatar Fedik
Fedik - comment - 8 Jan 2020

you mean changes in joomla.asset.json?
well, I did not forcing anything :neckbeard:
there was 1000 merge conflicts in that file, that can accidentally mix the changes

avatar dgrammatiko
dgrammatiko - comment - 8 Jan 2020

I know, but basically you reverted this PR

Add a Comment

Login with GitHub to post a comment