Summary: | Simplify MARC view choices in the OPAC | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, jwagner, m.de.rooy, nengard |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
MARC view with "view plain" link
After clicking the "view plain" link Plain view Proposed change Revised fix Revised for current HEAD Signed patch with marker fix Patch signed (2nd fix) |
Description
Owen Leonard
2010-08-09 15:55:01 UTC
Created attachment 2488 [details]
MARC view with "view plain" link
Note that there is now only one "MARC View" tab at the top. Clicking it brings you to this view. If XSLT is on, a "view plain" link will appear as it does in this screenshot.
Created attachment 2489 [details]
After clicking the "view plain" link
When the "view plain" link is clicked, JavaScript is used to pull the raw MARC data from a separate file. A "loading" indicator is displayed while that process completes.
Created attachment 2490 [details]
Plain view
The MARC view page after the raw MARC record has been loaded and displayed. Note the "view plain" link has turned into a "view labeled" link to return to the standard labeled view.
I would add 2 points: 1. Change the label to 'Librarian View' or 'Staff View' or something other than MARC (other OPACs have done this already) 2. I would rename make what you call 'plain view' the default and then have a link to 'labeled view' or if not come up with a different name for 'plain view' cause that really makes no sense :) > 1. Change the label to 'Librarian View' or 'Staff View' or something other than
> MARC (other OPACs have done this already)
I don't know... If the view is primarily for librarians, why not use a label that librarians understand?
Cause it's the OPAC :) it's for the public - not the librarians. But really - it's not a big deal to me, just something I've seen on other open source ILSes Created attachment 2491 [details] [review] Proposed change Here's patch for the work I've done so far if anyone is interested in testing. Created attachment 2818 [details] [review] Revised fix Revised patch includes CSS updates and missing image file Created attachment 2872 [details] [review] Revised for current HEAD Tested the patch. Works fine. I like the idea, but would like to make the distinction between view labeled and view plain more clear. I would suggest to stick to the current name Expanded (instead of the new Labeled). Additionally, you could perhaps mention both views on the same line: [View plain MARC] and [View expanded MARC]. Show by color or so which view is active, or by label: [Plain MARC view] and [Switch to expanded MARC view] (if plain is active, v.v.) This makes the user more aware of the active view and existence of both views. > Additionally, you could perhaps mention both views on the same line: [View
> plain MARC] and [View expanded MARC].
This is my motivation for keeping the links subtle: I think this is a feature of interest to a very limited subset of OPAC users. I'm assuming most users of the OPAC are not interested the details of the MARC record. Of those that are interested in the MARC record, only a few will be interested in seeing the plain unformatted view. I don't think the two versions of the MARC view deserve equal weight in the interface.
You are right that only a few opac users will be interested. Signing it off. Created attachment 2895 [details] [review] Signed patch with marker fix Removed a conflict marker line in opac.css from Owen's patch. Created attachment 2896 [details]
Patch signed (2nd fix)
Oops, forgot two new files in this patch..
Pushed to new/enh/bug_5106 Im going to do some testing on this tonight, and probably push a follow up patch for the opac-showmarc.pl bit of it. Just some tidy up. Id appreciate some others testing too, then we can merge it to master Pushed a follow up patch, and have tested, everything working as it should. Merged to master, please test and mark resolved Tested. Seems to work fine. One question (about tidying up code): Where is the call to opac-showmarc with view=card in the opac now? Is it still there? This bug is mentioned in: Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC http://lists.koha-community.org/pipermail/koha-patches/2010-December/013198.html Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC http://lists.koha-community.org/pipermail/koha-patches/2010-December/013199.html Bug 5106 Followup http://lists.koha-community.org/pipermail/koha-patches/2010-December/013200.html Bug 5106 Followup http://lists.koha-community.org/pipermail/koha-patches/2010-December/013201.html Bug 5106 Followup http://lists.koha-community.org/pipermail/koha-patches/2010-December/013202.html Bug 5106 Followup http://lists.koha-community.org/pipermail/koha-patches/2010-December/013203.html Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC http://lists.koha-community.org/pipermail/koha-patches/2010-December/013436.html Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC http://lists.koha-community.org/pipermail/koha-patches/2010-December/013437.html Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC (Conflict marker fix) http://lists.koha-community.org/pipermail/koha-patches/2010-December/013444.html Proposed fix for Bug 5106 - Simplify MARC view choices in the OPAC (Conflict marker fix 2nd try) http://lists.koha-community.org/pipermail/koha-patches/2010-December/013445.html |