| Summary: | Add a span with class around serial enumeration/chronology data in list of checkouts for better styling | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Mathieu Saby <mathsabypro> |
| Component: | Circulation | Assignee: | Katrin Fischer <katrin.fischer> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | arthur.suzuki, gmcharlt, kyle.m.hall, lucas, martin.renvoize, wainuiwitikapark |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
23.05.00,22.11.04,22.05.11, 21.11.19
|
|
| Circulation function: | |||
| Attachments: |
Bug 31209: Add a span with a class for items.enumchron in the checkouts list
Bug 31209: Add a span with a class for items.enumchron in the checkouts list Bug 31209: Add a span with a class for items.enumchron in the checkouts list |
||
|
Description
Mathieu Saby
2022-07-21 07:06:35 UTC
I like the idea of it having its own column, that way you can sort issues by enumchron if needed I think we should start with the span. More columns make the screen much wider and we already have a lot that libraries will want to display. The enumchron is also placed in the same spot in the list of checkins, but it has a class there that is also used here now. Created attachment 146456 [details] [review] Bug 31209: Add a span with a class for items.enumchron in the checkouts list To test: * Add or edit an item and add some info to the 'Serial Enumeration / chronology' field * Check the item out * Verify by checking the source code or using the element inspector that the title and enumchron have been merged into one string * Apply patch * Reload and verify that now the enumchron is wrapped in a span with class="item_enumeration" matching the class in the list of checkins Created attachment 146673 [details] [review] Bug 31209: Add a span with a class for items.enumchron in the checkouts list To test: * Add or edit an item and add some info to the 'Serial Enumeration / chronology' field * Check the item out * Verify by checking the source code or using the element inspector that the title and enumchron have been merged into one string * Apply patch * Reload and verify that now the enumchron is wrapped in a span with class="item_enumeration" matching the class in the list of checkins Signed-off-by: David Nind <david@davidnind.com> Created attachment 146902 [details] [review] Bug 31209: Add a span with a class for items.enumchron in the checkouts list To test: * Add or edit an item and add some info to the 'Serial Enumeration / chronology' field * Check the item out * Verify by checking the source code or using the element inspector that the title and enumchron have been merged into one string * Apply patch * Reload and verify that now the enumchron is wrapped in a span with class="item_enumeration" matching the class in the list of checkins Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Pushed to master for 23.05. Nice work everyone, thanks! Many hands makes light work, thankyou everyone! Pushed to 22.11.x for the next release Backported to 22.05.x for upcoming 22.05.11 applied to 21.11.x for 21.11.19 Not backported to 21.05.x |