? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
8 Aug 2015

In articles with a TOC and pagebreaks, the 2nd or later page may not be displayed as active because the active class is not applied to the <li> element as demonstrated in Bootstrap's documentation.

Testing Instructions

Create an article that uses page breaks; this snippet can be pasted into an editor in HTML mode to accomplish the test:

<p>Page 1</p>
<hr title="Second Page" alt="second-page" class="system-pagebreak" />
<p>Page 2</p>

Pre-test, when you navigate pages, the 2nd page will never highlight as active. Post patch, it does.

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar mbabker mbabker - open - 8 Aug 2015
avatar mbabker mbabker - change - 8 Aug 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Aug 2015
Labels Added: ?
avatar MAT978
MAT978 - comment - 8 Aug 2015

success

able to reproduce then #7663 works as described and 2nd or later page displayed as active

question: when "All pages" is selected, then shouldn't all the pages displayed as active?

Thanks


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

avatar MAT978 MAT978 - test_item - 8 Aug 2015 - Tested successfully
avatar mbabker
mbabker - comment - 8 Aug 2015

It should. That's fix now too.

avatar zero-24 zero-24 - change - 9 Aug 2015
Category Front End UI/UX
avatar zero-24 zero-24 - change - 9 Aug 2015
Easy No Yes
avatar watchfulli-dev
watchfulli-dev - comment - 11 Aug 2015

@test it works, after applying the patch when navigating to second page it gets highlighted


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

avatar watchfulli-dev watchfulli-dev - test_item - 11 Aug 2015 - Tested successfully
avatar zero-24 zero-24 - test_item - 11 Aug 2015 - Tested successfully
avatar zero-24 zero-24 - change - 11 Aug 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 11 Aug 2015

Tested also the last changes successfull. Thanks :smile: I'm RTC'ing now.


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

avatar joomla-cms-bot joomla-cms-bot - change - 11 Aug 2015
Labels Added: ?
avatar Kubik-Rubik Kubik-Rubik - change - 12 Aug 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-08-12 14:10:41
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - close - 12 Aug 2015
avatar joomla-cms-bot joomla-cms-bot - close - 12 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - reference | f99cb75 - 12 Aug 15
avatar Kubik-Rubik Kubik-Rubik - merge - 12 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - close - 12 Aug 2015
avatar joomla-cms-bot joomla-cms-bot - change - 12 Aug 2015
Labels Removed: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 12 Aug 2015

Thank you @mbabker! Merged.

avatar Kubik-Rubik Kubik-Rubik - change - 12 Aug 2015
Milestone Added:
avatar mbabker mbabker - head_ref_deleted - 12 Aug 2015
avatar dinhtiendung
dinhtiendung - comment - 4 Oct 2015

Hi there. Can you modify the second patch for me? My page break component worked as your first one, that is highlighting active links. However, when I want to highlight all of them when choosing "All pages", then the second patch is not working for me. Thanks.


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

Add a Comment

Login with GitHub to post a comment