Bug 35350 - Update label creator pop-up windows with consistent footer markup
Summary: Update label creator pop-up windows with consistent footer markup
Status: Pushed to stable
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Owen Leonard
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords:
Depends on: 35349 34343
Blocks:
  Show dependency treegraph
 
Reported: 2023-11-16 13:47 UTC by Owen Leonard
Modified: 2024-07-25 11:11 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:
24.05.00,23.11.03
Circulation function:


Attachments
Bug 35350: Update label creator pop-up windows with consistent footer markup (4.29 KB, patch)
2023-11-16 14:03 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 35350: Update label creator pop-up windows with consistent footer markup (4.35 KB, patch)
2023-12-01 15:46 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35350: Update label creator pop-up windows with consistent footer markup (4.35 KB, patch)
2023-12-01 17:01 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35350: Update label creator pop-up windows with consistent footer markup (4.35 KB, patch)
2023-12-01 17:01 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35350: Update label creator pop-up windows with consistent footer markup (4.42 KB, patch)
2024-01-26 10:23 UTC, Martin Renvoize (ashimema)
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-11-16 13:47:47 UTC
I would like to update two label creator pop-up windows to be consistent with Bug 34343 and its children:

- Label item search form
- Label item search results
Comment 1 Owen Leonard 2023-11-16 14:03:14 UTC
Created attachment 159051 [details] [review]
Bug 35350: Update label creator pop-up windows with consistent footer markup

This patch updates two label creator templates in order to style
submission and close buttons in a fixed footer at the bottom of the
window, with markup consistent with other pop-up windows.

To test, apply the patch and go to Cataloging -> Label creator.

- Click New -> Label batch.
- Click the "Add items" button under the form.
- A "Search for items" window should pop up.
  - The window's fixed footer should have the "Search" and "Close
    window" buttons. Test that each work correctly.
- Submit a search which will return results.
  - The search results window should also have a fixed footer. I moved
    the "Add checked" and "Done" buttons out of a floating toolbar and
    put the controls there for consistency's sake.
  - Confirm that both controls work correctly.
Comment 2 ByWater Sandboxes 2023-12-01 15:46:50 UTC
Created attachment 159457 [details] [review]
Bug 35350: Update label creator pop-up windows with consistent footer markup

This patch updates two label creator templates in order to style
submission and close buttons in a fixed footer at the bottom of the
window, with markup consistent with other pop-up windows.

To test, apply the patch and go to Cataloging -> Label creator.

- Click New -> Label batch.
- Click the "Add items" button under the form.
- A "Search for items" window should pop up.
  - The window's fixed footer should have the "Search" and "Close
    window" buttons. Test that each work correctly.
- Submit a search which will return results.
  - The search results window should also have a fixed footer. I moved
    the "Add checked" and "Done" buttons out of a floating toolbar and
    put the controls there for consistency's sake.
  - Confirm that both controls work correctly.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 3 ByWater Sandboxes 2023-12-01 17:01:16 UTC
Created attachment 159465 [details] [review]
Bug 35350: Update label creator pop-up windows with consistent footer markup

This patch updates two label creator templates in order to style
submission and close buttons in a fixed footer at the bottom of the
window, with markup consistent with other pop-up windows.

To test, apply the patch and go to Cataloging -> Label creator.

- Click New -> Label batch.
- Click the "Add items" button under the form.
- A "Search for items" window should pop up.
  - The window's fixed footer should have the "Search" and "Close
    window" buttons. Test that each work correctly.
- Submit a search which will return results.
  - The search results window should also have a fixed footer. I moved
    the "Add checked" and "Done" buttons out of a floating toolbar and
    put the controls there for consistency's sake.
  - Confirm that both controls work correctly.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 4 ByWater Sandboxes 2023-12-01 17:01:56 UTC
Created attachment 159466 [details] [review]
Bug 35350: Update label creator pop-up windows with consistent footer markup

This patch updates two label creator templates in order to style
submission and close buttons in a fixed footer at the bottom of the
window, with markup consistent with other pop-up windows.

To test, apply the patch and go to Cataloging -> Label creator.

- Click New -> Label batch.
- Click the "Add items" button under the form.
- A "Search for items" window should pop up.
  - The window's fixed footer should have the "Search" and "Close
    window" buttons. Test that each work correctly.
- Submit a search which will return results.
  - The search results window should also have a fixed footer. I moved
    the "Add checked" and "Done" buttons out of a floating toolbar and
    put the controls there for consistency's sake.
  - Confirm that both controls work correctly.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 5 Barbara Johnson 2023-12-01 17:02:49 UTC
I tested it on a new sandbox and it worked great.
Comment 6 Martin Renvoize (ashimema) 2024-01-26 10:23:36 UTC
Created attachment 161500 [details] [review]
Bug 35350: Update label creator pop-up windows with consistent footer markup

This patch updates two label creator templates in order to style
submission and close buttons in a fixed footer at the bottom of the
window, with markup consistent with other pop-up windows.

To test, apply the patch and go to Cataloging -> Label creator.

- Click New -> Label batch.
- Click the "Add items" button under the form.
- A "Search for items" window should pop up.
  - The window's fixed footer should have the "Search" and "Close
    window" buttons. Test that each work correctly.
- Submit a search which will return results.
  - The search results window should also have a fixed footer. I moved
    the "Add checked" and "Done" buttons out of a floating toolbar and
    put the controls there for consistency's sake.
  - Confirm that both controls work correctly.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 7 Martin Renvoize (ashimema) 2024-01-26 10:23:57 UTC
Nice consistency fix, no regressions.

Passing QA
Comment 8 Katrin Fischer 2024-01-29 11:28:37 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 9 Fridolin Somers 2024-02-01 13:51:53 UTC
Pushed to 23.11.x for 23.11.03