Bug 32616 - Add 'page-section' to various acquisitions pages
Summary: Add 'page-section' to various acquisitions pages
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 30952
Blocks: 31757
  Show dependency treegraph
 
Reported: 2023-01-12 16:51 UTC by Owen Leonard
Modified: 2023-06-08 22:32 UTC (History)
3 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:
23.05.00,22.11.02


Attachments
Bug 32616: Add 'page-section' to various acquisitions pages (37.40 KB, patch)
2023-01-12 17:11 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 32616: Add 'page-section' to various acquisitions pages (37.40 KB, patch)
2023-01-12 18:02 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 32616: Add 'page-section' to various acquisitions pages (37.45 KB, patch)
2023-01-12 21:29 UTC, David Nind
Details | Diff | Splinter Review
Bug 32616: Add 'page-section' to various acquisitions pages (37.51 KB, patch)
2023-01-16 19:37 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 Owen Leonard 2023-01-12 16:51:53 UTC
Several acquisitions templates need to have main content wrapped in a "page-section" container.

 - acqui/edi_ean.tt
 - acqui/edifactmsgs.tt
 - acqui/histsearch.tt
 - acqui/lateorders.tt
 - acqui/newordersubscription.tt
 - acqui/newordersuggestion.tt
 - acqui/ordered.tt
 - acqui/spent.tt
 - acqui/z3950_search.tt
Comment 1 Owen Leonard 2023-01-12 17:11:18 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2023-01-12 18:02:24 UTC
Created attachment 145241 [details] [review]
Bug 32616: Add 'page-section' to various acquisitions pages

This patch adds a "page-section" container div around the main section
of acquisitions page which lack it.

This patch contains indentation changes, so please ignore whitespace
when looking at the diff.

To test, apply the patch and view the following pages to confirm that
the main content is contained in a white box:

- Acquisitions -> Vendor -> Basket -> Create EDIFACT order
- Acquisitions -> EDIFACT messages
- Acquisitions -> Orders search -> Results
- Acquisitions -> Late orders
- Acquisitions -> Vendor -> Add to basket -> From a subscription
- Acquisitions -> Vendor -> Add to basket -> From a suggestion
- Acquisitions -> All available funds -> Ordered
- Acquisitions -> All available funds -> Spent
- Acquisitions -> Vendor -> Add to basket -> From an external source ->
  Results
Comment 3 David Nind 2023-01-12 21:29:46 UTC
Created attachment 145247 [details] [review]
Bug 32616: Add 'page-section' to various acquisitions pages

This patch adds a "page-section" container div around the main section
of acquisitions page which lack it.

This patch contains indentation changes, so please ignore whitespace
when looking at the diff.

To test, apply the patch and view the following pages to confirm that
the main content is contained in a white box:

- Acquisitions -> Vendor -> Basket -> Create EDIFACT order
- Acquisitions -> EDIFACT messages
- Acquisitions -> Orders search -> Results
- Acquisitions -> Late orders
- Acquisitions -> Vendor -> Add to basket -> From a subscription
- Acquisitions -> Vendor -> Add to basket -> From a suggestion
- Acquisitions -> All available funds -> Ordered
- Acquisitions -> All available funds -> Spent
- Acquisitions -> Vendor -> Add to basket -> From an external source ->
  Results

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2023-01-12 21:31:55 UTC
Testing notes (using KTD):

1. EDIFACT setup:
   . Enable EDIFACT
   . Administration > Acquisition parameters:
     - EDI accounts > New account > Add details (including 'Orders enabled', details don't need to be valid)
     - Library EANs > New EAN

2. Suggestions: add a suggestion and change the status to accepted

3. Add to order from a subscription: set up a serial first
Comment 5 Katrin Fischer 2023-01-16 19:37:05 UTC
Created attachment 145338 [details] [review]
Bug 32616: Add 'page-section' to various acquisitions pages

This patch adds a "page-section" container div around the main section
of acquisitions page which lack it.

This patch contains indentation changes, so please ignore whitespace
when looking at the diff.

To test, apply the patch and view the following pages to confirm that
the main content is contained in a white box:

- Acquisitions -> Vendor -> Basket -> Create EDIFACT order
- Acquisitions -> EDIFACT messages
- Acquisitions -> Orders search -> Results
- Acquisitions -> Late orders
- Acquisitions -> Vendor -> Add to basket -> From a subscription
- Acquisitions -> Vendor -> Add to basket -> From a suggestion
- Acquisitions -> All available funds -> Ordered
- Acquisitions -> All available funds -> Spent
- Acquisitions -> Vendor -> Add to basket -> From an external source ->
  Results

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Katrin Fischer 2023-01-16 19:38:23 UTC
Tiny note/question: 

On the EDIFACT messages page there is a page-section with a dialog box within:

There are no EDIFACT messages.

On other pages we have the alerts/dialogs outside the page section. Maybe we should move this above the page section and make sure the empty page section doesn't display?
Comment 7 Martin Renvoize 2023-01-17 09:40:42 UTC
Nice work everyone!

Pushed to 23.05.x for the next release
Comment 8 Matt Blenkinsop 2023-01-17 15:00:41 UTC
Nice work everyone!

Pushed to stable for 22.11.x
Comment 9 Matt Blenkinsop 2023-01-17 15:11:48 UTC
Nice work everyone!

Pushed to stable for 22.11.x