Summary: | OpacNav menu hide user menu on opac-user.pl page | ||
---|---|---|---|
Product: | Koha | Reporter: | Frédéric Demians <f.demians> |
Component: | OPAC | Assignee: | Frédéric Demians <f.demians> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, christophe.croullebois, koha.sekjal, nengard |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Proposed patch
Proposed patch Revised patch [SIGNED-OFF] Bug 6825 OpacNav menu hide user menu on opac-user.pl page |
Description
Frédéric Demians
2011-08-31 10:03:09 UTC
Created attachment 5236 [details] [review] Proposed patch Created attachment 5238 [details] [review] Proposed patch taking that GBSD#4 ok fine I'm wondering if this patch is the kind of thing that all libraries will want - many people have worked their design around the navigation being as it is - I hate to say it but this might be a system preference type thing so that it doesn't mess things up for libraries who have been using Koha for a long time and have users used to the menus being where they are. Changes to navigation shouldn't be made lightly when they cause usability issues for the patrons. Nicole This is neither a bug nor an enhancement, but rather a matter of preference. For some libraries, the OpacNav content is best displayed above the opac-user menu (as it is now). For others, it's best below. The simplest solution for this is to include a system preference. A better, but much more difficult, solution would be to provide some kind of interface for users to be able to place content blocks on the various OPAC pages. While this bug is in itself sound, I think the change being made is not to the benefit of all libraries. Please reimplement with some kind of mechanism for the library to choose the order of this content. > This is neither a bug nor an enhancement, but rather a matter of > preference. For some libraries, the OpacNav content is best displayed > above the opac-user menu (as it is now). For others, it's best below. I've never seen a library using OpacNav and not complaining about patron pages account links being pushed down by OpacNav... but I agree that some libraries may have tweaked their OPAC design to cope with that. > The simplest solution for this is to include a system preference. A better, > but much more difficult, solution would be to provide some kind of interface > for users to be able to place content blocks on the various OPAC pages. What you think of an OpacNavBottom syspref which would be placed on all pages after OpacNav? on Patron pages, there will be: OpacNav Patron links OpacNavBottom OpacNavBottom syspref description could be: Show the following HTML on the left hand column of the main page and patron account on the OPAC, after OpacNav, and before patron account links if available. Works for me, but wouldn't it be easier to have a pref for OpacNacLocation and say - "show opacnav above/below patron links" (In reply to comment #8) > Works for me, but wouldn't it be easier to have a pref for > OpacNacLocation and say - "show opacnav above/below patron links" Your solution is better from an user point of view: easier to understand. My solution would allow to keep on Patron page few links above Patron links: Library Home page link for example. I can't decide. Please choose for me one solution or the other. Thanks. If you're willing to do the work then I choose your way :) It allows for more customization in the end. Nicole Created attachment 5395 [details] [review] Revised patch This patch add a new syspref OpacNavBottom which is placed on all pages after OpacNav. On Patron pages, specif patron links are placed between OpacNav and OpacNavBottom, like this: OpacNav Patron links OpacNavBottom To test: 1 Apply this patch 2 For RM, modify kohaversion.pl and updatedatabase.pl appropriately 3 Test that OPAC pages are displayed as before, OpacNav on the left 4 Test OPAC patron pages: OpacNav is as before displayed after patron links 5 Split OpacNav between itself and OpacNavBottom 6 Patron pages display patron links between OpacNav and OpacNavBottom Created attachment 5396 [details] [review] [SIGNED-OFF] Bug 6825 OpacNav menu hide user menu on opac-user.pl page This patch add a new syspref OpacNavBottom which is placed on all pages after OpacNav. On Patron pages, specif patron links are placed between OpacNav and OpacNavBottom, like this: OpacNav Patron links OpacNavBottom To test: 1 Apply this patch 2 For RM, modify kohaversion.pl and updatedatabase.pl appropriately 3 Test that OPAC pages are displayed as before, OpacNav on the left 4 Test OPAC patron pages: OpacNav is as before displayed after patron links 5 Split OpacNav between itself and OpacNavBottom 6 Patron pages display patron links between OpacNav and OpacNavBottom Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Adds a new system preference that gives libraries the choice of how they want their content to display in user pages. Very cleanly done, and doesn't introduce new strings. Marking as Passed QA. I agree with Ian, pushed Seems to work fine. Closing. |