? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
1 Feb 2021

Pull Request for Issue #32224 .

Summary of Changes

Makes sidebar-* countModules content-aware

Testing Instructions

  1. Enable only the Tag Similar module in either sidebar position
  2. Open an article with no similar tags

Before

Blank space in sidebar position

After

No blank space in sidebar position

Documentation Changes Required

avatar ciar4n ciar4n - open - 1 Feb 2021
avatar ciar4n ciar4n - change - 1 Feb 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Feb 2021
Category Front End Templates (site)
avatar ciar4n ciar4n - change - 1 Feb 2021
The description was changed
avatar ciar4n ciar4n - edited - 1 Feb 2021
avatar ciar4n ciar4n - change - 1 Feb 2021
The description was changed
avatar ciar4n ciar4n - edited - 1 Feb 2021
avatar ciar4n ciar4n - change - 1 Feb 2021
The description was changed
avatar ciar4n ciar4n - edited - 1 Feb 2021
avatar brianteeman
brianteeman - comment - 2 Feb 2021

Wouldnt you need this in almost every instance of module count. Its just really obvious with the sidebar

avatar ciar4n ciar4n - change - 2 Feb 2021
Title
[4.0] Make sidebar countModule content aware
[4.0] Set sidebar countModule content aware
avatar ciar4n ciar4n - edited - 2 Feb 2021
avatar ciar4n ciar4n - change - 2 Feb 2021
Labels Added: ?
avatar ciar4n
ciar4n - comment - 2 Feb 2021

Yes fair point. I've applied to all positions.

avatar brianteeman brianteeman - test_item - 2 Feb 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 2 Feb 2021

I have tested this item successfully on f5c1cfb


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

avatar chmst chmst - test_item - 2 Feb 2021 - Tested successfully
avatar chmst
chmst - comment - 2 Feb 2021

I have tested this item successfully on f5c1cfb


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

avatar chmst chmst - change - 2 Feb 2021
Status Pending Ready to Commit
avatar chmst
chmst - comment - 2 Feb 2021

RTC


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

avatar drmenzelit drmenzelit - close - 3 Feb 2021
avatar drmenzelit drmenzelit - merge - 3 Feb 2021
avatar drmenzelit drmenzelit - change - 3 Feb 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-02-03 07:28:50
Closed_By drmenzelit
Labels Added: ?
avatar drmenzelit
drmenzelit - comment - 3 Feb 2021

Thanks

Add a Comment

Login with GitHub to post a comment