In the opac results and detail page multiple 856 online resource links are seperated by a '|', which is added via XSLT. Since they are not wrapped inside any HTML element it can be hard to customize with CSS/JS. If they were instead psuedo elements it would be easier to remove them or style them differently.
Created attachment 116298 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still.
Created attachment 116328 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I am not completely sure about this one. The existing pattern at the moment would be to use <span class="separator"> | </span>. What is the advantage of the pseudo element? That it can be overwritten using CSS instead of jQuery? I tried to remove the 3rd link out of 4 by using: $("a:contains('Table')").remove(); Sample data: Philippics. I-II / Cicero ; edited by John T. Ramsey. In this case I end up with || and have no idea how to fix that. Previously I would just hide the preceeding one with jQuery as well and that would solve he issue. I am not sure if and how I could do that now. We often get requests for hiding some or highlighting other URLs, so that's a common use case for me. Remember: Our records are created in a union catalog, there are often tons of links, but changing the records locally is not an option as the union catalog is the primary database. I am failing this, because I think if we change it on the detail page, we should really change it in the results list too. And personally I'd really like to know if you can still hide elements in a nice way somehow!
Cait, 1. This patch does change both the OPAC detail and the OPAC results page. 2. I think we can still easily, even more easily, remove links with jQuery. In your example I would use this: $('a[href="http://www.loc.gov/catdir/toc/cam041/2003273871.html"] ').parent().hide(); // This works for hiding it on both the results and detials. 3. I think to semantically describe the online_resource section as a list of links is good.
Hi Lucas, I will test it, but meant all the other repeating fields using the existing pattern. I think having 2 ways do to things is not necessarily good, better to agree to switch or keep.
Created attachment 125742 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one.
*** Bug 29176 has been marked as a duplicate of this bug. ***
Created attachment 127453 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 127454 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one.
Created attachment 127542 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 127543 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Looks great to me, signing off.
Ok, I came around to the change :) Only one question: could we throw in the contributurs/authors list to make this more complete of a change? There might be some more in the Utils file that need attention.
Created attachment 128425 [details] [review] Bug 27613: (follow-up) add authors and contributors
Created attachment 129051 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 129052 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 129053 [details] [review] Bug 27613: (follow-up) add authors and contributors Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 130973 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 130974 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 130975 [details] [review] Bug 27613: (follow-up) add authors and contributors Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Forgot my sign-off lines!
Created attachment 130977 [details] [review] Bug 27613: Change pipe seperated online resource links to psuedo elements To test: 1. Have a record with many 856 links 2. Look at the links on both the results and detail pages. 3. Look at the '|' and notice it is outside any HTML element, making it hard to remove. 4. Apply the patch and regenerate the CSS 5. Look at both the pages again and notice that the '|' is now a psuedo element. 6. It should look identical to the way it did without the patch 7. Switch the system pref OPACURLOpenInNewWindow to 'do' and make sure the the links look and function right still. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 130978 [details] [review] Bug 27613: (follow-up) Modify additional repeating entries for consistency This patch makes further changes to the detail and results XSLT in the OPAC so that previously pipe-separated fields are styled with CSS. The following tags are modified on the detail page: 020 ISBN 022 ISSN 050 LOC classification 060 NLM classification 082 DDC classification 084 Other classification 130/240/730 Uniform titles 246 Other title 336 Content type 337 Media type 338 Carrier type 385 Audience 508 Production credits 583 Action note 586 Awards 655 Genre/Form 700/710/711 Contained works 700/710/711 Related works 770 Supplement 772 Supplement to 856 Online resources These are modified on the results page: 028 Publisher number 246 Other title To test, apply the patch and rebuild the OPAC CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client). In order to better see the effect of the CSS change you can add something ike this to your OPACUserCSS system preference: .resource_list li::after { content: "♦"; } View catalog search results and detail pages for records which contain repeated entries from the tag list above. The entries should be separated by "|" without customization, or the symbol from your OPACUserCSS if you specified one. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 130979 [details] [review] Bug 27613: (follow-up) add authors and contributors Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
(In reply to Fridolin Somers from comment #26) > Pushed to master for 22.05, thanks to everybody involved [U+1F984] I found some display issues that might be related to this bug: Example in ktd sample database: http://localhost:8080/cgi-bin/koha/opac-detail.pl?biblionumber=70 The label by is 'below' the content that is 'above' it. Could some CSS be missing? I think it looked ok in testing.
(In reply to Katrin Fischer from comment #27) > I found some display issues that might be related to this bug: > > Example in ktd sample database: > http://localhost:8080/cgi-bin/koha/opac-detail.pl?biblionumber=70 > > The label by is 'below' the content that is 'above' it. > > Could some CSS be missing? I think it looked ok in testing. I confirm this is not normal. I've opened Bug 33830.