Bug 33195 - Reindent the bibliographic details page
Summary: Reindent the bibliographic details page
Status: BLOCKED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 33568
Blocks: 33526
  Show dependency treegraph
 
Reported: 2023-03-10 12:49 UTC by Owen Leonard
Modified: 2024-01-18 16:56 UTC (History)
2 users (show)

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


Attachments
Bug 33195: Reindent the bibliographic details page (222.31 KB, patch)
2023-05-18 18:02 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 33195: (follow-up) Add markup comments (37.65 KB, patch)
2023-05-18 18:02 UTC, Owen Leonard
Details | Diff | Splinter Review
Three related bibliographic records (5.68 KB, text/plain)
2023-05-19 16:51 UTC, Owen Leonard
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2023-03-10 12:49:59 UTC
I would like to reformat the bibliographic details page (catalogue/detail.tt) in the staff interface so that it has consistent spacing and indentation.
Comment 1 Owen Leonard 2023-05-18 18:02:15 UTC
Created attachment 151432 [details] [review]
Bug 33195: Reindent the bibliographic details page

This patch reindents the bibliographic details template so that
it has consistent indentation. These changes should have no visible
effect on the page.

To test, apply the patch and view the detail page for a bibliographic
record. Test as many variations as you can:

- With and without cover images enabled
- With FRBRizeEditions enabled (and multiple editions in your catalog)
- With LocalCoverImages enabled
- With AcquisitionDetails enabled
- With a serial record
- With a record with component parts
- With EnableItemGroups enabled
- Wtih CatalogConcerns enabled
- With HTML5MediaEnabled and a record with media links

Viewing the diff while ignoring whitespace changes should show only
places where a line break was added.
Comment 2 Owen Leonard 2023-05-18 18:02:17 UTC
Created attachment 151433 [details] [review]
Bug 33195: (follow-up) Add markup comments

This patch adds comments to the template to highlight the markup
structure.

This patch should have no effect on the page's appearance or
functionality.
Comment 3 Owen Leonard 2023-05-19 16:51:52 UTC
Created attachment 151486 [details]
Three related bibliographic records

This MARC file can be imported and used for testing FRBRizeEditions with ThingISBN
Comment 4 Owen Leonard 2023-05-19 16:54:50 UTC
Bug 8377 has a sample MARC file for testing HTML5MediaEnabled: https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10685
Comment 5 Jonathan Druart 2023-05-23 09:45:52 UTC
Please don't do this, I am currently working heavily on this template, see bug 33568.
Comment 6 Owen Leonard 2023-05-23 10:40:52 UTC
(In reply to Jonathan Druart from comment #5)
> Please don't do this

Okay cool