? ? Success
Pull Request for # 7031

User tests: Successful: Unsuccessful:

avatar ghost
ghost
20 Oct 2015

This is a 1:1 copy of PR #7031 because of a merge conflict there.

See discussion and test results there please.

Copied instructions;

See feature request #6987
Menu item type Compact list of tagged items.
This PR adds a new option # Items to List like in menu item type Articles > Category List.
It removes old option Item Selection Options > Maximum Items because not needed anymore.

Test instructions:

  • Create a tag.
  • Apply it to several items (20, 30 ...) like articles, categories, contacts... It's easy with button Batch.
  • Create a menu item of type Compact list of tagged items and select above tag.
  • Save & Close.
  • Open in frontend but do not touch limit box yet!!!
  • Number of diplayed items = Default List Limit in Global Configuration of Joomla.
    (If not delete cookie [= kill session], clear browser cache and reload page).
    tags-limitbox
  • Apply patch.
  • Reload page in frontend. Nothing changed. Do not touch limit box yet!!!
  • Backend > open menu item.
  • Change new option List Layouts > # Items to List. E.g. to 5
    tags-limitbox2
  • Reload page in frontend.
    tags-limitbox3
avatar bertmert bertmert - open - 20 Oct 2015
avatar bertmert bertmert - change - 20 Oct 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2015
Labels Added: ? ?
avatar bertmert bertmert - change - 20 Oct 2015
Title
Patch 31
Compact list of tagged items. New option Items to List
avatar zero-24 zero-24 - change - 20 Oct 2015
Milestone Added:
Status Pending Ready to Commit
Rel_Number 0 7031
Relation Type Pull Request for
Labels
avatar zero-24 zero-24 - change - 20 Oct 2015
Category Front End
avatar zero-24
zero-24 - comment - 20 Oct 2015

Thanks RTC :smile:


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

avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2015
Labels Added: ?
avatar roland-d
roland-d - comment - 25 Oct 2015

Thanks everybody. Merged into 3.5-dev with commit db54caf

avatar roland-d roland-d - change - 25 Oct 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-10-25 17:09:21
Closed_By roland-d
avatar wilsonge wilsonge - change - 17 Jan 2016
Labels Removed: ?
avatar AndySDH
AndySDH - comment - 12 Oct 2020

So this was marked as RTC 5 years ago but never merged... Nice. @wilsonge @roland-d @zero-24 @ghost

avatar HLeithner
HLeithner - comment - 12 Oct 2020

@AndySDH maybe you should read the comment by roland

Thanks everybody. Merged into 3.5-dev with commit db54caf

avatar AndySDH
AndySDH - comment - 12 Oct 2020

It's clearly not merged because it's still not in the current version of Joomla lol

avatar HLeithner
HLeithner - comment - 12 Oct 2020

Then it got lost with 3.5-dev for what ever reason anyway can you redo the pr for 4.0?

avatar zero-24
zero-24 - comment - 12 Oct 2020

It's clearly not merged because it's still not in the current version of Joomla lol

Well the code is in 3.9.22:
https://github.com/joomla/joomla-cms/blob/3.9.22/components/com_tags/views/tag/tmpl/list.xml#L202
https://github.com/joomla/joomla-cms/blob/3.9.22/components/com_tags/models/tag.php#L221

(I have not tested whether that feature still works but atleast the code was not removed but correctly added and is still shipped with 3.9.x)

avatar AndySDH
AndySDH - comment - 12 Oct 2020

Oh I see. So there is a lot of confusion with this I guess.

It looks like this is only available in the "Compact" layout and not in "Normal" layout.

Also, it appears this is only available in the Menu Item settings but it's not in Global Configuration at all.

Weird.

avatar infograf768
infograf768 - comment - 12 Oct 2020

Please test 4.0 here #30998 and let know if something missing.

avatar AndySDH
AndySDH - comment - 12 Oct 2020

Please test 4.0 here #30998 and let know if something missing.

Doesn't look like that PR addresses or concerns what I mentioned above?

avatar infograf768
infograf768 - comment - 13 Oct 2020

It may as nothing was changed versus 3.x in the sense that the default number of tagged items for the Tagged Items menu item does not include a field to define a default list limit.
It could be added in another PR.

avatar AndySDH
AndySDH - comment - 13 Oct 2020

It may as nothing was changed versus 3.x in the sense that the default number of tagged items for the Tagged Items menu item does not include a field to define a default list limit.
It could be added in another PR.

Could definitely be good if you could make a PR for it. It should also be ported into Global Configuration, as it's missing from there.

PS: This setting ties in with the idea that Tag views should offer similar features (and appearance) to Category views in my opinion: #30998 (comment)

avatar infograf768
infograf768 - comment - 13 Oct 2020

articles category views are very different from tags as they concern the same type of items.

Add a Comment

Login with GitHub to post a comment