Bug 35426

Summary: Improve layout of bookings modal form
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: RESOLVED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: barbara.johnson, fridolin.somers, lucas, martin.renvoize
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:
24.05.00,23.11.04
Circulation function:
Bug Depends on: 29002    
Bug Blocks: 35532    
Attachments: Bug 35426: Improve layout of bookings modal form
Bug 35426: Improve layout of bookings modal form
Bug 35426: Improve layout of bookings modal form
Old vs New

Description Owen Leonard 2023-11-28 18:45:38 UTC
The bookings modal form, triggered on the staff interface page when you click the "Place bookings" button, has an awkward layout because the fieldset with "rows" class doesn't work well there. Changing the class to "brief" makes it work better.
Comment 1 Owen Leonard 2023-11-28 19:03:37 UTC
Created attachment 159312 [details] [review]
Bug 35426: Improve layout of bookings modal form

This patch changes the class on the bookings modal form's fieldset in
order to make the form work better in the small space offered by the
modal.

Also changed: The hint for the patron search field has been moved below
the field for consistency.

To test, apply the patch and search for a title in the staff interface.

- View the detail page.
- If you don't see a "Place booking" button in the toolbar,
  click "Items" in the sidebar menu.
  - Locate the "Priority" heading, and change the "Bookable" setting to
    "Yes."
- Click the "PLace booking" button in the toolbar.
  - The bookings modal form should appear, and the form layout should be
    improved.
Comment 2 ByWater Sandboxes 2023-12-01 15:31:25 UTC
Created attachment 159454 [details] [review]
Bug 35426: Improve layout of bookings modal form

This patch changes the class on the bookings modal form's fieldset in
order to make the form work better in the small space offered by the
modal.

Also changed: The hint for the patron search field has been moved below
the field for consistency.

To test, apply the patch and search for a title in the staff interface.

- View the detail page.
- If you don't see a "Place booking" button in the toolbar,
  click "Items" in the sidebar menu.
  - Locate the "Priority" heading, and change the "Bookable" setting to
    "Yes."
- Click the "PLace booking" button in the toolbar.
  - The bookings modal form should appear, and the form layout should be
    improved.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 3 Tomás Cohen Arazi (tcohen) 2023-12-18 12:56:22 UTC
Created attachment 159963 [details] [review]
Bug 35426: Improve layout of bookings modal form

This patch changes the class on the bookings modal form's fieldset in
order to make the form work better in the small space offered by the
modal.

Also changed: The hint for the patron search field has been moved below
the field for consistency.

To test, apply the patch and search for a title in the staff interface.

- View the detail page.
- If you don't see a "Place booking" button in the toolbar,
  click "Items" in the sidebar menu.
  - Locate the "Priority" heading, and change the "Bookable" setting to
    "Yes."
- Click the "PLace booking" button in the toolbar.
  - The bookings modal form should appear, and the form layout should be
    improved.

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Katrin Fischer 2024-03-07 14:04:02 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 5 Martin Renvoize (ashimema) 2024-03-08 16:05:59 UTC
I'm not sure I agree this looks better...  now it's pushed the modal looks more odd to me than it did before.
Comment 6 Martin Renvoize (ashimema) 2024-03-08 16:15:04 UTC
Created attachment 162974 [details]
Old vs New
Comment 7 Martin Renvoize (ashimema) 2024-03-08 16:15:52 UTC
Can we do better here in general for modal based forms Owen?
Comment 8 Martin Renvoize (ashimema) 2024-03-08 16:18:18 UTC
In fact.. all the modals I've created that include a form use 'rows' rather than 'brief' and personally I think they're clearer looking.. see all the modals available on the accounts pages?
Comment 9 Fridolin Somers 2024-03-11 09:09:43 UTC
Small enhancement, I choose to backport

Pushed to 23.11.x for 23.11.04
Comment 10 Fridolin Somers 2024-03-11 09:10:42 UTC
Ah reading comments I prefer not backport
Comment 11 Martin Renvoize (ashimema) 2024-03-18 13:08:47 UTC
I think we can backport this regardless of my negative comments.. I'd just love to see us get more consistent overall somehow in the future.
Comment 12 Fridolin Somers 2024-03-19 08:26:11 UTC
Pushed to 23.11.x for 23.11.04
Comment 13 Lucas Gass (lukeg) 2024-03-19 21:54:35 UTC
Missing dependencies for 23.05.x, no backport.