J4 Issue ?
avatar ReLater
ReLater
16 May 2019

ToDo

  • After pr #24921 is merged check whick modules need line HTMLHelper::_('bootstrap.framework');
  • See #24921 (comment)
avatar ReLater ReLater - open - 16 May 2019
avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 16 May 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 16 May 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 16 May 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 16 May 2019
Status New Discussion
avatar richard67
richard67 - comment - 18 May 2019
$ find ./administrator/modules/ -type f -exec grep -l "HTMLHelper::_('bootstrap.framework');" {} \;
./administrator/modules/mod_popular/tmpl/default.php
./administrator/modules/mod_sampledata/tmpl/default.php
./administrator/modules/mod_feed/tmpl/default.php
./administrator/modules/mod_status/tmpl/default.php
./administrator/modules/mod_logged/tmpl/default.php
./administrator/modules/mod_multilangstatus/tmpl/default.php
./administrator/modules/mod_latestactions/tmpl/default.php
./administrator/modules/mod_latest/tmpl/default.php
$
$ find ./administrator/modules/ -type f -exec grep -l "dropdown" {} \;
./administrator/modules/mod_menu/tmpl/default_submenu.php
./administrator/modules/mod_status/tmpl/default.php
$

Don't know if that helps.

avatar ReLater
ReLater - comment - 18 May 2019

Finding is not the ""problem"" but to decide if Bootstrap is loaded also for functionalities of the HTML output of the modules.
I don't have the time at the moment to investigate that.

avatar richard67
richard67 - comment - 18 May 2019

So it would need to compare the HTML output produced by the modules listed in my first listing with the content of the same modules when the HTMLHelper::_('bootstrap.framework'); has been removed, and if output is the same, then the line is obsolete, right? Or does it need more to be investigated?

avatar ReLater
ReLater - comment - 18 May 2019

and if output is the same, then the line is obsolete, right?

Yes, if everything functions as before.

Even if I've learned Joomla 4 core doesn't need JQuery/Bootstrap-JS and everything in core will be replaced somehow by Vanilla scripts it looks like that this isn't the fact yet. Thus I'm in doubt if we can remove these lines.

avatar richard67
richard67 - comment - 18 May 2019

I will see if I can find time. But if you find time before me or others, let us known here so we don't do double work.

avatar richard67
richard67 - comment - 25 May 2019

@ReLater Meanwhile I am also in doubt if it is as easy as I assumed it to be. Do you know who could help us with that?

avatar ReLater
ReLater - comment - 25 May 2019

Yes, not easy to test. Time-consuming.

I use a plugin that removes bootstrap-*.js in backend. Then compare the output plus functionalities of the backend modules with and without active plugin. I also deactivate other backend modules as far as possible.

First result: mod_popular doesn't need bootstrap.

BUT then I've found that /templates/atum/layouts/chromes/body.php DOESN'T load bootstrap-*.js but uses it for the edit cog dropdowns of modules. Thus it should load BS if we follow the common practice.

25-05-_2019_15-42-10

Now I think it's better to place a pr first for that find in body.php and ask for comments there if it's correct to add HTMLHelper::_('bootstrap.framework'); where it's missing AND (more important) if it's worth the time to investigate further for other files; instead of waiting.

See my related comment: "Even if I've learned Joomla 4 core doesn't need JQuery/Bootstrap-JS and everything in core will be replaced somehow by Vanilla scripts it looks like that this isn't the fact yet."

avatar ReLater
ReLater - comment - 25 May 2019

Maybe @franz-wohlkoenig knows whom to ping?
I try @dgrammatiko , @wilsonge , @Bakual #24922 (comment)

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 25 May 2019

@ReLater sorry, have no idea.

avatar ReLater
ReLater - comment - 25 May 2019

From my point of view modules should load their dependencies also with other templates but only if needed.

The main question is: Will be backend core extensions BS and Jquery free or not?

There is NO point looking at this until the new template is completed

Yes for chrome body.php. That's an atum layout. But not mod_popular.

avatar ReLater ReLater - close - 25 May 2019
avatar ReLater ReLater - change - 25 May 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-05-25 19:33:17
Closed_By ReLater
avatar wilsonge
wilsonge - comment - 25 May 2019

The backend extensions will be jQuery (direct usages) free. However they won't be Bootstrap free from a CSS perspective. The aim is to make the backend not automatically load the javascript on all pages unless required.

Add a Comment

Login with GitHub to post a comment