J4 Issue ?
avatar chmst
chmst
6 Jul 2018

Steps to reproduce the issue

You must have an istallation where the modules on cpanel are not edited yet.
The modules then have class "col-md-6" and are visible in two columns.

Klick on the edit button of a module, edit something, i.e. the number of items to be shown, save and close and go back to the cpanel.

Expected result

The Module has col-md-6 as before.

Actual result

The module has col-md-12, bootstrap-size is set to 0.

System information (as much as possible)

Additional comments

You cannot reproduce this when a module is edited once.

avatar chmst chmst - open - 6 Jul 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Jul 2018
avatar brianteeman
brianteeman - comment - 6 Jul 2018

This sounds like it's a result of the installation SQL default value not matched the default value in the modules. @SharkyKZ

avatar brianteeman brianteeman - change - 6 Jul 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 6 Jul 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 7 Jul 2018
Status New Discussion
avatar SharkyKZ
SharkyKZ - comment - 7 Jul 2018

bootstrap_size value is not present in SQL files but Atum's module chrome defaults to 6.

avatar chmst
chmst - comment - 7 Jul 2018

Let's wait for the new template.
If the param bootstrap_size is not set when the module is edited, bootstrap_size must remain null or '' and not be set to 0 as it is now.

avatar frogydiak
frogydiak - comment - 7 Jul 2018

Is there a branch or fork of the new backend template? Can it be tested, I want to help. Thanks!

avatar TobsBobs
TobsBobs - comment - 7 Jul 2018

@frogydiak
this is the pull request #21006 an this is the repository https://github.com/joomla/40-backend-template

avatar ghazal
ghazal - comment - 11 Jul 2018

I have been noticing this for a while.
The setting is in administrator/templates/atum/html/modules.php,
lines 56 and 97 (well and body):
$moduleClass = ($bootstrapSize) ? 'col-md-' . $bootstrapSize : 'col-md-12';
In my opinion it should be "col-md-6" but maybe we should wait for more advanced changes on Atum.

avatar wilsonge
wilsonge - comment - 19 Aug 2018

Is this still an issue?

avatar brianteeman
brianteeman - comment - 19 Aug 2018

yes

avatar wilsonge wilsonge - change - 19 Aug 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-08-19 15:41:04
Closed_By wilsonge
avatar wilsonge wilsonge - close - 19 Aug 2018
avatar wilsonge
wilsonge - comment - 19 Aug 2018

Fixed with 5defb00

Add a Comment

Login with GitHub to post a comment