Page navigation at the bottom of Shelf Browser screen

Bug #1763173 reported by Eva Cerninakova
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

It it would be useful if page navigation of Shelf Browser was at the bottom of the page, not just at the top, as it is now (similar to the search results interface). The bottom page navigation would be benefiting especially for mobile devices where it is now necessary to scroll up at the end of every page of the Shelf Browser.

The easy solution would be to add following code to /Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2 (before the line 59)

            <tr class='browse_border'>
                <td colspan='3'>&nbsp;</td>
            </tr>

                <tr>
                    <td>
                        <a class='classic_link lbl1'
                            href="[% mkurl('', {cnoffset => cnoffset - 1}) %]#cnbrowse">[% l("&lt;&lt; Previous Page") %]</a>
                    </td>
                    <td colspan='1' align='center'>&nbsp;</td>
                    <td>
                        <a class='classic_link lbl1'
                            href="[% mkurl('', {cnoffset => cnoffset + 1}) %]#cnbrowse">[% l("Next Page &gt;&gt;") %]</a>
                    </td>
                </tr>

The snapshot of Shelf Browser with added bottom page navigation is attached.
The "living" Shelf Browser with added bottom navigation is temporarily available in our test catalog (3.1.0), see http://eg-test.osvobozena-knihovna.cz/eg/opac/cnbrowse?cn=a;locg=1;cnoffset=1#cnbrowse

Revision history for this message
Eva Cerninakova (ece) wrote :
Revision history for this message
Eva Cerninakova (ece) wrote :
Eva Cerninakova (ece)
tags: added: usability
tags: added: opac
removed: tpac
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Wishlist
tags: removed: wishlist
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.next
milestone: 3.next → 3.13-beta
Revision history for this message
Stephanie Leary (stephanieleary) wrote (last edit ):

Looks good, signed off: user/sleary/lp1763173_bottom_nav_for_shelf_browser_signoff

tags: added: signedoff
Revision history for this message
Bill Erickson (berick) wrote :

Merge to main for 3.13. Thanks all.

Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
status: Confirmed → Fix Committed
Revision history for this message
Michele Morgan (mmorgan) wrote :

Bill beat me to it! Pushed one more commit with an alignment tweak as a followup.

Bill Erickson (berick)
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.