The MARC view in the OPAC is useful to librarians, but probably less to to non-librarian patrons. Offering two ways to view a MARC record (when XSLT is on) seems like overkill, an unnecessary complication to the interface. Users who click the link that pops up the modal MARC window will find that their back button doesn't work until they click "close" on that window. I propose the following: 1. Eliminate the "Extended MARC View" tab 2. Point the "MARC View" tab to opac-MARCDetail.pl as it is when XSLT is off 3. Offer a "view plain" link on opac-MARCDetail.pl. Clicking this link would replace the standard labeled MARC view with an unformatted view similar to the one displayed in the pop-up modal MARC view. 4. When viewing the "plain view," clicking the "view labeled" link will return you to the standard labeled MARC view. This solution accomplishes the following: 1. Simplifies the interface 2. Eliminates the back button problem with the pop-up MARC view 3. Gives librarians alternatives for viewing MARC, including a view which makes cut and paste of the raw record easy.
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