Bug 37812 - Adjust Vue modals for Bootstrap 5
Summary: Adjust Vue modals for Bootstrap 5
Status: In Discussion
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low major
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords: rel_24_11_candidate, RM_priority
Depends on: 35402
Blocks:
  Show dependency treegraph
 
Reported: 2024-09-03 07:15 UTC by Jonathan Druart
Modified: 2024-09-12 07:04 UTC (History)
5 users (show)

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


Attachments
Bug 37812: Try to fix modal in the Preservation module (2.53 KB, patch)
2024-09-03 07:17 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 37812: Adjust modals from the Preservation module - bootstrap5 (18.24 KB, patch)
2024-09-10 14:47 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 37812: Adjust Dialog.vue component (7.54 KB, patch)
2024-09-10 14:47 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2024-09-03 07:15:01 UTC
Modals in the Vue components (ERM, Preservation at least) are not displayed nicely.

From bug 37733 comment 0

"""
2. "Add to waiting list" modal is not centered

http://localhost:8081/cgi-bin/koha/preservation/waiting-list 

https://snipboard.io/mIBYkj.jpg

3. Confirmation box "Are you sure you want to remove this item" is not centered correctly

https://snipboard.io/ZgJFGC.jpg
"""
Comment 1 Jonathan Druart 2024-09-03 07:17:44 UTC
Created attachment 170956 [details] [review]
Bug 37812: Try to fix modal in the Preservation module

We certainly need more as ERM module also use modals. And the Dialog
component seems to have more elements affected.
Comment 2 Martin Renvoize 2024-09-03 09:06:52 UTC
I also started converting some from dialog to actual modal in Bug 37605 - Why do the vue apps use 'dialog' in place of modals?
Comment 3 Jonathan Druart 2024-09-10 14:47:11 UTC
Created attachment 171252 [details] [review]
Bug 37812: Adjust modals from the Preservation module - bootstrap5
Comment 4 Jonathan Druart 2024-09-10 14:47:13 UTC
Created attachment 171253 [details] [review]
Bug 37812: Adjust Dialog.vue component
Comment 5 Jonathan Druart 2024-09-10 14:49:50 UTC
This is how we should fix it IMO.

However there are still quite a lot of work to do.

https://snipboard.io/TJ9zek.jpg => OK
https://snipboard.io/MQHoCt.jpg => Dropdown list's options are hidden behind the modal's footer
https://snipboard.io/5ru41B.jpg => Could be better styled

Do we want to go with toasts? (it's not introduced in Koha yet so I don't know if it's a good idea).

There are also some "close"/"x" buttons that does not close/hide the modal. However the code is similar with other places where it is working... ?

For discussion, please help!
Comment 6 Jonathan Druart 2024-09-10 14:50:57 UTC
And I forgot this one: https://snipboard.io/rY90KF.jpg
Comment 7 Jonathan Druart 2024-09-10 14:51:22 UTC
This must be part of 24.11. At least the preservation module is unusable.