[Bug 6945] New: Add a link to show a MARC preview in the normal view
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Bug #: 6945 Summary: Add a link to show a MARC preview in the normal view Classification: Unclassified Change sponsored?: Sponsored Product: Koha Version: master Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 Component: Cataloging AssignedTo: gmcharlt@gmail.com ReportedBy: pelletiermaxime@gmail.com QAContact: koha-bugs@lists.koha-community.org CC: ian.walls@bywatersolutions.com Created attachment 5651 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5651 Screenshot showing the new link MARC Preview Adds a link MARC Preview to show the MARC in a greybox in the Normal view of catalogue/detail.pl. Sponsored by our client http://www.ccsr.qc.ca/ -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #1 from Maxime Pelletier <pelletiermaxime@gmail.com> 2011-09-29 21:05:04 UTC --- Created attachment 5652 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5652 patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Status|NEW |ASSIGNED CC| |nengard@gmail.com Patch Status|--- |Needs Signoff AssignedTo|gmcharlt@gmail.com |pelletiermaxime@gmail.com -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5652|0 |1 is obsolete| | --- Comment #2 from Nicole C. Engard <nengard@gmail.com> 2011-09-30 12:06:27 UTC --- Created attachment 5658 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5658 [SIGNED-OFF] Add a link to show a MARC preview in the normal view of the detail http://bugs.koha-community.org/show_bug.cgi?id=6945 Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Needs Signoff |Signed Off --- Comment #3 from Nicole C. Engard <nengard@gmail.com> 2011-09-30 12:07:00 UTC --- I have signed off on this simple patch, but you might consider editing the XSLT stylesheets with this link as well. Nicole -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> 2011-09-30 13:11:16 UTC --- Created attachment 5660 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5660 Follow-up fix This follow-up (to be applied after the original patch) adds the same preview link to the section of the template which is displayed when XSLT is on. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5660|0 |1 is obsolete| | --- Comment #5 from Nicole C. Engard <nengard@gmail.com> 2011-09-30 13:44:00 UTC --- Created attachment 5661 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5661 [SIGNED-OFF] Follow-up fix for Bug 6945, Add a link to show a MARC preview in the normal view Adding the same MARC preview link to the section of the template which is displayed when XSLT is on. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Maxime Pelletier <pelletiermaxime@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent |P5 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Maxime Pelletier <pelletiermaxime@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Signed Off |Passed QA --- Comment #6 from Ian Walls <ian.walls@bywatersolutions.com> 2011-10-18 15:08:01 UTC --- This feature is not going to be universally desirable to all libraries; I'd prefer a unique ID in one of the HTML elements so that JQuery can hide the feature at the library's discretion. This is not reason enough to block QA, however. Marking as Passed, with a request for followup. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #7 from Frédérick Capovilla <fcapovilla@live.ca> 2011-10-21 20:12:46 UTC --- Created attachment 6026 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=6026 Follow-up : Add an id on the "Marc Preview" <span> New follow-up patch adding an id on the <span> of the MARC Preview. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #8 from Owen Leonard <oleonard@myacpl.org> 2011-10-24 17:15:34 UTC --- *** Bug 2134 has been marked as a duplicate of this bug. *** -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |paul.poulain@biblibre.com Patch Status|Passed QA |Failed QA --- Comment #9 from Paul Poulain <paul.poulain@biblibre.com> 2011-11-06 10:29:53 UTC --- For consistency reasons i'll reject this patch. Here is what I get on my screen : ==================================== Massacre (The) éô ï By 50 Cent * Additional Authors: by 50 Cent * Published by: Shady/Aftermath , 2006 * Description: 1 d.c. * MARC Preview: Show =================================== (that is what is also in the screenshot submitted by Maxime) The "MARC Preview" is not related to the biblio content, so the link should not be here. It should be either on the action button (on the right of "place hold" for example) or on the left tab. You can also do exactly the same thing just using XSLT, and I think it's the best way to do it: just activate XSLT and add a XSLT link to the MARC preview after the description. Thus, no need to add code to Koha -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #10 from Paul Poulain <paul.poulain@biblibre.com> 2011-11-06 10:31:54 UTC --- unless you're very good arguments, I think this bug must be RESO/INVALID. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #11 from Nicole C. Engard <nengard@gmail.com> 2011-11-06 13:06:00 UTC --- I have a very good reason. Professional catalogers (like me) hate the way Koha shows MARC records. It's cluttered, shows way more info than I need and makes my life difficult. In the OPAC there is a way to see the plain MARC record and patrons could care less - but in the staff client there isn't an option to see this which seems just wrong. I don't care where the link is moved to, but there must be a link to see the real MARC record in the staff client from the bib record instead of just the labeled and slightly differently labeled views. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #12 from Paul Poulain <paul.poulain@biblibre.com> 2011-11-06 14:39:54 UTC --- OK, seen comment 11, I won't try to go against your needs, but I'm still thinking: * it should not be in the biblio * if one really want it in the biblio, then just add it into the XSLT and activate XSLT So an improved/updated patch is still welcomed (and I won't close this bug until we've found a consensus/solution) -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #13 from Nicole C. Engard <nengard@gmail.com> 2011-11-06 15:08:37 UTC --- I'm not sure I understand what you mean about it not being in the biblio. Do you mean it shouldn't be mixed in with the info from the bib (like the title, author, subjects, etc)? If so, I'm okay with that - there is a MARC tab on the left and a labeled MARC tab over there too. So I'm okay with adding another tab for MARC - not sure what to call it, because right now the MARC and the Labeled MARC tabs both show Labeled MARC just slightly different .. hopefully Maxime has some ideas. Nicole -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |oleonard@myacpl.org --- Comment #14 from Paul Poulain <paul.poulain@biblibre.com> 2011-11-06 15:39:37 UTC --- (In reply to comment #13)
Do you mean it shouldn't be mixed in with the info from the bib (like the title, author, subjects, etc)? yes, exactly.
If so, I'm okay with that Perfect
- there is a MARC tab on the left and a labeled MARC tab over there too. So I'm okay with adding another tab for MARC - not sure what to call it, because right now the MARC and the Labeled MARC tabs both show Labeled MARC just slightly different .. hopefully Maxime has some ideas. Another option would be to put it in a button on the right of "Print" and "Place hold". I'm not sure which is the best option: it would be more logical to have it on the left, where we already have "Normal" "MARC", "labelled MARC", "ISBD" and "Items". But that would add a 6th view, and put "Hold" and "Subscriptions" (if applicable) very low.
Adding Owen to this discussion, Owen, if you have a suggestion, it will be welcomed ! A small idea : remove the "Holds" tab, that points exactly to the same location that "Place hold" on the top-right. I remember some libraries being a little bit confused to have exactly the same thing in 2 places. Maybe it's time to remove the tab, and use the place for the "marc preview" ! -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #15 from Owen Leonard <oleonard@myacpl.org> 2011-11-06 18:10:45 UTC --- (In reply to comment #14) .
Another option would be to put it in a button on the right of "Print" and "Place hold"
Buttons are for actions, tabs are for views. That's why we have both a button and a tab for holds. A person might want to view holds or they might want to add a hold. That said, I don't have any problem with this patch as-is. If we figure out a way to do it better later we can fix it. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Maxime Pelletier <pelletiermaxime@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #16 from Maxime Pelletier <pelletiermaxime@gmail.com> --- Is my patch still a problem after the comments by oleonard and nengard ? I think the id in the followup patch makes it easy to hide if it's not wanted. Let me know if this needs any more work. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jcamins@cpbibliography.com --- Comment #17 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Could you clarify the status of this bug? If this were rebased, and the follow-up signed off, would that be sufficient, or are there additional changes required? -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #18 from Owen Leonard <oleonard@myacpl.org> --- I'll add my sign-off once this is rebased. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5661|0 |1 is obsolete| | -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #6026|0 |1 is obsolete| | -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #19 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 11754 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11754&action=edit Follow-up: Add preview to XSLT view, add unique id to containing element in both views Adding the same MARC preview link to the section of the template which is displayed when XSLT is on. This revision also adds a unique id to the surrounding element in the normal and XSLT versions. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent (DO NOT USE) |P5 - low Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5658|0 |1 is obsolete| | --- Comment #20 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 11781 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11781&action=edit [SIGNED-OFF] Add a link to show a MARC preview in the normal view of the detail http://bugs.koha-community.org/show_bug.cgi?id=6945 Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11754|0 |1 is obsolete| | --- Comment #21 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 11782 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11782&action=edit [SIGNED-OFF] Bug 6945 [FOLLOW-UP] Add a link to show a MARC preview in the normal view Adding the same MARC preview link to the section of the template which is displayed when XSLT is on. This revision also adds a unique id to the surrounding element in the normal and XSLT versions. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |mtj@kohaaloha.com QA Contact|koha-bugs@lists.koha-commun |mtj@kohaaloha.com |ity.org | --- Comment #22 from Mason James <mtj@kohaaloha.com> ---
Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
patches looks good, passing QA… $ koha-qa.pl -c 2 testing 2 commit(s) (applied to commit b592d34) * 86b0a36 Bug 6945 [FOLLOW-UP] Add a link to show a MARC preview in the nor koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt * 1227251 Add a link to show a MARC preview in the normal view of the detai koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt * koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt OK -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply --- Comment #23 from Paul Poulain <paul.poulain@biblibre.com> --- Trying to apply patch & follow-up, I get a git problem with the 2nd patch: Applying: Bug 6945 [FOLLOW-UP] Add a link to show a MARC preview in the normal view fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Passed QA --- Comment #24 from Owen Leonard <oleonard@myacpl.org> --- Paul could you try again? I used git bz to apply these patches today and while the second one required an automatic 3-way merge it did finish successfully. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #25 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 12695 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12695&action=edit Bug 6945 counter patch adding link in the cat-toolbar -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #26 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #24)
Paul could you try again? I used git bz to apply these patches today and while the second one required an automatic 3-way merge it did finish successfully.
I tried again, and it does not work again. I've submitted a counter-patch to explain what I would prefer: the patch add a YUI button to the toolbar. It works except that the page is not displayed in a small window, but in the main one. I couldn't find how to have the rel="gb_page_center[600,500]" -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|master |rel_3_12 --- Comment #27 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Changing target version so this bug report will show up in the list of 3.12-targeted features and doesn't get lost in the shuffle during feature freeze. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #28 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Owen, Could you please rebase *both* patches and reattach them? That should resolve the "sha1 information is lacking or useless" message that I get. Also, what is your thought on Paul's counter-patch? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11781|0 |1 is obsolete| | --- Comment #29 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 13256 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13256&action=edit [SIGNED-OFF] Add a link to show a MARC preview in the normal view of the detail Add a link to show a MARC preview in the normal view of the detail http://bugs.koha-community.org/show_bug.cgi?id=6945 Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11782|0 |1 is obsolete| | --- Comment #30 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 13257 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13257&action=edit [SIGNED-OFF] Bug 6945 [FOLLOW-UP] Add a link to show a MARC preview in the normal view Adding the same MARC preview link to the section of the template which is displayed when XSLT is on. This revision also adds a unique id to the surrounding element in the normal and XSLT versions. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off --- Comment #31 from Owen Leonard <oleonard@myacpl.org> --- I feel pretty strongly that putting the link in the toolbar is not the right solution. This MARC preview link doesn't fit with views in the left-hand sidebar menu because it doesn't take you to a new page. It doesn't fit with the actions in the toolbar because it doesn't "do" something (like edit the record or put a hold on the record). I can't give a good argument for putting the link where it is in these patches other than to say that it's better than the other options. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #32 from Nicole C. Engard <nengard@gmail.com> --- I agree with Owen on this. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12695|0 |1 is obsolete| | --- Comment #33 from Paul Poulain <paul.poulain@biblibre.com> --- Comment on attachment 12695 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12695 Bug 6945 counter patch adding link in the cat-toolbar counter-patch does not please owen & nicole, removing it by obsoleting -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #13256|0 |1 is obsolete| | Attachment #13257|0 |1 is obsolete| | --- Comment #34 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 13780 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13780&action=edit Add a link to show a MARC preview in the normal view of the detail http://bugs.koha-community.org/show_bug.cgi?id=6945 Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 --- Comment #35 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 13781 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13781&action=edit Bug 6945 [FOLLOW-UP] Add a link to show a MARC preview in the normal view Adding the same MARC preview link to the section of the template which is displayed when XSLT is on. This revision also adds a unique id to the surrounding element in the normal and XSLT versions. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Signed-off-by: Paul Poulain <paul.poulain@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Patch complexity|--- |Trivial patch --- Comment #36 from Paul Poulain <paul.poulain@biblibre.com> --- QA comment: * despite I don't like the place where the link is placed, I QA this patch because I agree with Nicole & Owen it's an interesting link for librarians * tiny patch passes QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #37 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6945 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org