?
avatar Chacapamac
Chacapamac
23 Dec 2018

Steps to reproduce the issue

Add a custom module class in the administration

Expected result

.basicclass .mycustomclass

Actual result (without adding the space

.basicclassmycustomclass

Is this a recurrent bug. You need to add a space to a custom module class to be sure this class appear as you have type ?
Is this by design?

avatar Chacapamac Chacapamac - open - 23 Dec 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Dec 2018
avatar mbabker
mbabker - comment - 23 Dec 2018

This was always the design but in 4.0 IIRC the space is put into the
layouts and not needed in the admin field anymore. IMO this is a step
backwards but it is what it is.

Without the space, you can override the class the suffix is added to if
need be, and add extra classes. Putting the space in the layout removes
the ability to override the class.

On Sun, Dec 23, 2018 at 10:38 AM Chacapamac notifications@github.com
wrote:

Steps to reproduce the issue

Add a custom module class in the administration
Expected result

.basicclass .mycustomclass
Actual result (without adding the space

.basicclassmycustomclass

Is this a recurrent bug. You need to add a space to a custom module class
to be sure this class appear as you have type ?
Is this by design?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#23325, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoQsA43_0LFn562Ls_emFEdOALm_Xks5u76OTgaJpZM4Zf4uK
.

--

  • Michael Please pardon any errors, this message was sent from my iPhone.
avatar Chacapamac
Chacapamac - comment - 9 Jan 2019

Understood. Make sens & practical to keep it that way.

If original temlplate defined class = originalclass

No space override the orignal class = originalclassmyclass

With space we add a class = originalclass + myclass

Great Thanks!

avatar Bakual Bakual - change - 9 Jan 2019
Status New Closed
Closed_Date 0000-00-00 00:00:00 2019-01-09 18:03:32
Closed_By Bakual
avatar Bakual Bakual - close - 9 Jan 2019
avatar Bakual
Bakual - comment - 9 Jan 2019

Closing as expected behavior since it's called a "suffix".

Add a Comment

Login with GitHub to post a comment