Bug 28933 - Hard to parse OPAC-detail subscription information
Summary: Hard to parse OPAC-detail subscription information
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Lucas Gass
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-31 20:35 UTC by Lucas Gass
Modified: 2022-06-06 20:25 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.11.00


Attachments
Bug 28933: Add subscription class to each subscription in opac-detail (16.67 KB, patch)
2021-08-31 20:41 UTC, Lucas Gass
Details | Diff | Splinter Review
Bug 28933: Add subscription class to each subscription in opac-detail (16.73 KB, patch)
2021-09-02 18:32 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 28933: Add subscription class to each subscription in opac-detail (16.80 KB, patch)
2021-09-26 11:52 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass 2021-08-31 20:35:22 UTC
I received a request to hide all closed subscriptions from the OPAC detail subscriptions tab. 

When a subscription is closed a '.subscription_closed' class is added, which is great. 

The problem there is no hierarchy to the HTML structure, every subscription detail (.subscription_closed, .subscription_location, etc. ) is a sibling of the next. 

It would be trivial to add a parent element to each subscription to make it easier to target with CSS/jQuery.
Comment 1 Lucas Gass 2021-08-31 20:41:20 UTC
Created attachment 124314 [details] [review]
Bug 28933: Add subscription class to each subscription in opac-detail

To Test:
1. Go to a record with some subscriptions, have some closed and some open.
2. Go to the opac-detail page, click on the subscription tab and use your developer tools to inscept the element.
3. Notice the hierarchy of HTML structure, everything is a sibling of the next.
4. Apply patch
5. Reload the same opac-detail page and again inscept the element.
6. Now every individual subscription should be wrapped in an element with the class name of 'subscription'.
Comment 2 Owen Leonard 2021-09-02 18:32:35 UTC
Created attachment 124436 [details] [review]
Bug 28933: Add subscription class to each subscription in opac-detail

To Test:
1. Go to a record with some subscriptions, have some closed and some
   open.
2. Go to the opac-detail page, click on the subscription tab and use
   your developer tools to inscept the element.
3. Notice the hierarchy of HTML structure, everything is a sibling of
   the next.
4. Apply patch
5. Reload the same opac-detail page and again inscept the element.
6. Now every individual subscription should be wrapped in an element
   with the class name of 'subscription'.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Katrin Fischer 2021-09-26 11:52:49 UTC
Created attachment 125321 [details] [review]
Bug 28933: Add subscription class to each subscription in opac-detail

To Test:
1. Go to a record with some subscriptions, have some closed and some
   open.
2. Go to the opac-detail page, click on the subscription tab and use
   your developer tools to inscept the element.
3. Notice the hierarchy of HTML structure, everything is a sibling of
   the next.
4. Apply patch
5. Reload the same opac-detail page and again inscept the element.
6. Now every individual subscription should be wrapped in an element
   with the class name of 'subscription'.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Jonathan Druart 2021-09-29 10:44:27 UTC
Pushed to master for 21.11, thanks to everybody involved!