?
avatar joo7
joo7
6 May 2017

Steps to reproduce the issue

Create a "Custom" module, on tab "Advanced" set "Module Tag" to "article" or else... save

Expected result

ok (tag is "article")

Actual result

not ok (tag is always "div", no matter what is selected)

System information (as much as possible)

tested local and on demo-site

Additional comments

Feature request: Sometimes it would be great to select "none"
(to insert scribts on special sites...)

avatar joo7 joo7 - open - 6 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 6 May 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 May 2017
avatar brianteeman
brianteeman - comment - 6 May 2017

This will be dependant on the module chrome being used for the relevant position in your template.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15859.

avatar joo7
joo7 - comment - 6 May 2017

tested on demo-site (protostar) position-1

local position in index.php: <jdoc:include type="modules" name="script" style="none" />

avatar brianteeman
brianteeman - comment - 6 May 2017

As is said it is dependant on the module chrome. In your test you are using "none" and if you look at the code you will see that means the tag is not being used (as you have discovered ) https://github.com/joomla/joomla-cms/blob/staging/templates/system/html/modules.php

avatar joo7
joo7 - comment - 6 May 2017

in modules.php comment on line 13: none (output raw module content)
but output is with div-tags:

module content

no surrounding div-tags should be created, right? Should be raw?

avatar joo7
joo7 - comment - 6 May 2017

sorry, forgot github-code-tags...
output is with div-tags:

<div>
module content
</div>

avatar brianteeman
brianteeman - comment - 6 May 2017

That is because the div is coming from the module itself.

This really should be discussed on the forum as its not a bug

avatar brianteeman brianteeman - change - 6 May 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-05-06 22:56:05
Closed_By brianteeman
avatar brianteeman brianteeman - close - 6 May 2017

Add a Comment

Login with GitHub to post a comment