? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
6 Feb 2020

The child count of items is using a definition list. This is completely the wrong markup to use as a number is not the description of the term in this use.

This PR changes it to a span exactly the same as every other use of show_cat_items

avatar brianteeman brianteeman - open - 6 Feb 2020
avatar brianteeman brianteeman - change - 6 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Feb 2020
Category Front End com_newsfeeds
avatar Quy Quy - test_item - 6 Feb 2020 - Tested successfully
avatar Quy
Quy - comment - 6 Feb 2020

I have tested this item successfully on 6e97fd3


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

avatar richard67
richard67 - comment - 9 Feb 2020

@brianteeman I've tried to test but I did not get the questionable markup (before applying the PR of course). What am I doing wrong? Was checking the category view for newsfeeds category after having installed testing sample data and switched on all show this or that options in the component settings.

avatar brianteeman
brianteeman - comment - 9 Feb 2020

as it is in the child you need to make sure you add a subcategory and create a feed in that subcategory

avatar richard67
richard67 - comment - 9 Feb 2020

Thanks, will try soon.

avatar richard67 richard67 - test_item - 9 Feb 2020 - Tested successfully
avatar richard67
richard67 - comment - 9 Feb 2020

I have tested this item successfully on 6e97fd3


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

avatar richard67 richard67 - change - 9 Feb 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 9 Feb 2020

RTC


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

avatar richard67 richard67 - change - 9 Feb 2020
Labels Added: ? ?
avatar Quy Quy - change - 9 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-09 15:46:24
Closed_By Quy
Labels
avatar Quy Quy - close - 9 Feb 2020
avatar Quy Quy - merge - 9 Feb 2020
avatar brianteeman
brianteeman - comment - 9 Feb 2020

thx

Add a Comment

Login with GitHub to post a comment