Bug 35862

Summary: Display patron search result on the right of the form (modal)
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Staff interfaceAssignee: Lucas Gass <lucas>
Status: Pushed to main --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, lucas, martin.renvoize, oleonard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00
Bug Depends on: 35329    
Bug Blocks:    
Attachments: Bug 35862: Display modal search results to the right of form above 1200px
Bug 35862: Display modal search results to the right of form above 1200px
Bug 35862: Display modal search results to the right of form above 1200px

Description Jonathan Druart 2024-01-22 20:44:22 UTC
From bug 35329 comment 59

> Next comment.. could we perhaps put the results next to rather than under
> the filters form in the modal?
> 
> It feels odd to have so much whitespace in the modal and have the actual
> useful content almost hidden off screen.. widescreen aspect ratios are
> pretty common these days?
Comment 1 Jonathan Druart 2024-01-23 07:51:23 UTC
*** Bug 35846 has been marked as a duplicate of this bug. ***
Comment 2 Lucas Gass 2024-03-06 23:48:25 UTC
Created attachment 162858 [details] [review]
Bug 35862: Display modal search results to the right of form above 1200px

To test:
1. Apply patch
2. Rebuild CSS ( yarn build )
3. Look at some of the places where there is a modal patron search.
    -suggestion/suggestion.pl: Set to patron
    -members/memberentry.pl: Add guarantor

4. Make sure the modal looks good before and after you have searched for a patron.
5. Play with many different screen sizes, ensuring everything looks good and makes sense.
Comment 3 Martin Renvoize 2024-03-07 07:58:30 UTC
Created attachment 162862 [details] [review]
Bug 35862: Display modal search results to the right of form above 1200px

To test:
1. Apply patch
2. Rebuild CSS ( yarn build )
3. Look at some of the places where there is a modal patron search.
    -suggestion/suggestion.pl: Set to patron
    -members/memberentry.pl: Add guarantor

4. Make sure the modal looks good before and after you have searched for a patron.
5. Play with many different screen sizes, ensuring everything looks good and makes sense.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2024-03-07 08:00:18 UTC
This is a huge improvement, thanks Lucas.

Signing off.
Comment 5 Jonathan Druart 2024-03-07 08:05:37 UTC
Do we want the 100% width for inputs/select on small screens? https://snipboard.io/iwo0lI.jpg
Comment 6 Lucas Gass 2024-03-07 14:40:08 UTC
(In reply to Jonathan Druart from comment #5)
> Do we want the 100% width for inputs/select on small screens?
> https://snipboard.io/iwo0lI.jpg

It's a good question. I personally think so. But I am happy to change it if we don't think that will be well received.
Comment 7 Martin Renvoize 2024-03-08 09:57:34 UTC
I like the 100% myself.
Comment 8 Jonathan Druart 2024-03-08 12:48:31 UTC
Created attachment 162959 [details] [review]
Bug 35862: Display modal search results to the right of form above 1200px

To test:
1. Apply patch
2. Rebuild CSS ( yarn build )
3. Look at some of the places where there is a modal patron search.
    -suggestion/suggestion.pl: Set to patron
    -members/memberentry.pl: Add guarantor

4. Make sure the modal looks good before and after you have searched for a patron.
5. Play with many different screen sizes, ensuring everything looks good and makes sense.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 9 Katrin Fischer 2024-04-19 17:56:00 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 10 Martin Renvoize 2024-04-26 06:19:26 UTC
This one appears to break the patron_search selenium tests.. I'm working on the resolution now.
Comment 11 Martin Renvoize 2024-04-26 06:45:15 UTC
False alarm.. it's actually bug 35362 that breaks the test (and the functionality too)