User tests: Successful: Unsuccessful:
Testing instructions:
Main Category
First Subcategory
link1
link2
link3
Second Subcategory
link4
link5
link6
Third Subcategory
link7
link8
link9
All new settings will be ignored if Group By Subcategories
is set to No
and the default behaviour of the module hasn't changed.
Added requested fixes.
Milestone |
Added: 3.5.0 |
@manureggi Assuming this PR is still valid, can you fix the merge conflicts please?
@chrisdavenport The merge conflicts were only due to the fact that the mod_weblinks.xml file had been moved since I issued the pull request. I've now rebased onto the current master branch.
@manureggi Looks good. Perhaps add a showon="groupby:1" attribute to the Show Group Title, Group Ordering and Group Ordering Direction fields?
Since I haven't heard back from @manureggi I'm going to just merge this as it stands. I'll submit a new PR for the minor UI change that I proposed.
Thanks @manureggi for coding this feature and thanks to everyone else for testing and feedback.
Milestone |
Removed: 3.5.0 |
Milestone |
Added: 3.6.0 |
Milestone |
Added: 3.6.0 |
Milestone |
Removed: 3.5.0 |
Please add testing instructions