? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
20 Jun 2018

Pull Request for Issue #15279 .

Summary of Changes

Adds HTML classes using the following standard to com-tags frontend views.

ExtensionType-Extension(-SubExtension)(-View) (Eg. mod-newsfeeds-category).

Note: If the default view then -View is omitted. If only one SubExtension exists or if the SubExtension matches the Extension then -SubExtension is omitted.

Child views and child elements within the views

BEM class naming is adopted..

ExtensionType-Extension__Element.

For B/C, old classes remain.

Testing Instructions

Code review.

Expected result

All works fine

Actual result

All works fine

Documentation Changes Required

Yes

avatar ciar4n ciar4n - open - 20 Jun 2018
avatar ciar4n ciar4n - change - 20 Jun 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Jun 2018
Category Front End com_newsfeeds
avatar ciar4n ciar4n - change - 20 Jun 2018
Title
HTML class naming standard [frontend][com-newsfeeds]
[4.0] HTML class naming standard [frontend][com-newsfeeds]
avatar ciar4n ciar4n - edited - 20 Jun 2018
avatar ciar4n ciar4n - change - 20 Jun 2018
Labels Added: ?
avatar Quy Quy - test_item - 21 Jun 2018 - Tested successfully
avatar Quy
Quy - comment - 21 Jun 2018

I have tested this item successfully on fb42d5e


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

avatar laoneo laoneo - change - 26 Jun 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-06-26 13:22:35
Closed_By laoneo
avatar laoneo laoneo - close - 26 Jun 2018
avatar laoneo laoneo - merge - 26 Jun 2018

Add a Comment

Login with GitHub to post a comment