? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
7 Jul 2022

Pull Request for Issue #38238 .

Summary of Changes

As title says - seems that this was forgotten.

Testing Instructions

Add a page class to menu item for a single article.

class="com-content-article item-page my-class

Actual result BEFORE applying this Pull Request

class="com-content-article item-pagemyclass

Expected result AFTER applying this Pull Request

class="com-content-article item-page myclass

Documentation Changes Required

avatar chmst chmst - open - 7 Jul 2022
avatar chmst chmst - change - 7 Jul 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jul 2022
Category Front End com_content
avatar brianteeman
brianteeman - comment - 7 Jul 2022

pretty sure that this one was kept for b/c - there will be a discussion on it somewhere in one of the many template repositories

avatar chmst
chmst - comment - 7 Jul 2022

We could make it ugly like this class="com-content-article item-pagemyclass myclass
But as it is now, we use this pageclass_sfx in an inconsistent way. I have already made an extra override in a site for that.

@brianteeman your opinion? It is ugly - no matter what we do here ;)

avatar brianteeman
brianteeman - comment - 7 Jul 2022

The ugly option obviously won't work

avatar brianteeman
brianteeman - comment - 7 Jul 2022

To be clear I don't have a problem with adding a space. It's just my memory saying it wasn't forgotten and there was a reason.

avatar Quy
Quy - comment - 7 Jul 2022
avatar chmst
chmst - comment - 7 Jul 2022

Thank you, @Quy - so we can only use overides here.

avatar chmst chmst - close - 7 Jul 2022
avatar chmst chmst - change - 7 Jul 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-07-07 10:32:30
Closed_By chmst
Labels Added: ?
avatar brianteeman
brianteeman - comment - 7 Jul 2022

lol - so much for my memory

Add a Comment

Login with GitHub to post a comment