[#12350] - Change `range` pagination to ?`strip` pagination.
- Closed
- 4 Jan 2018
- Medium
- Build: staging
- # 12350
- Diff
- bato3:patch-2
- Failure continuous-integration/drone the build failed Details
- Failure continuous-integration/travis-ci/pr The Travis CI build failed Details
- Pending JTracker/HumanTestResults Human Test Results: 0 Successful 1 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
Before solution gave me stupid results:
Documentation Changes Required
Before solution gave me stupid results:
page 1: 1 2 3 4 ... 6 7 8 9 10
page 10: 5 6 7 8 ... 10 11 12 13 14
page 14: ... 10 11 12 13 14 ... 16 17 18
page 16: 11 12 13 ... 15 16 17 18 19 ...
last page (20): 11 12 13 14 15 16 17 18 ... 20
AFTER:
page 1: 1 2 3 4 5 6 7 8 9 ...
page 6: 1 2 3 4 5 6 7 8 9 ...
page 7: ... 3 4 5 6 7 8 9 10 ...
page 189: ... 185 186 187 188 189 190 191 192 ...
page 190: ... 186 187 188 189 190 191 192 193 194
last page: ... 186 187 188 189 190 191 192 193 194
config:
Menu link: index.php?option=com_content&view=category&layout=blog&id=49
num_intro_articles: 20
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Front End Templates (site) |
| Labels |
Added:
?
Removed: ? |
||
I have tested this item
Notice: Undefined offset: 1 in /Joomla/templates/protostar/html/pagination.php on line 104
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12350.
I am closing this PR - it doesnt work and has not been touched for over a year. It an always be reopened if the merge conflicts and bugs are fixed
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-01-04 17:02:59 |
| Closed_By | ⇒ | brianteeman |
Do you realise that this file is an "override" for the specific protostar template so any changes made here will not apply to other templates - it also means that you can apply it to your own template without needing to be implemented in the core of joomla
I dont understand what is "stupid" about the results