Bug 8290

Summary: Replace usage of Greybox modals with Bootstrap modals
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: NEW --- QA Contact:
Severity: enhancement    
Priority: P5 - low CC: dcook, josef.moravec, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12325
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34710
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 32333, 34710, 15309, 15313, 15314, 15315, 15316, 15317, 15318, 15319, 15320, 16592, 16600, 16778, 24059    
Bug Blocks: 22882    

Description Owen Leonard 2012-06-22 11:50:16 UTC
jQueryUI includes a Dialog widget which can replace the Greybox library. Doing so will make the UI of dialogs in Koha consistent with other jQueryUI components and offer a tool which can be expanded into a wider array of usages.

http://jqueryui.com/demos/dialog/
Comment 1 David Cook 2014-05-30 01:09:26 UTC
Yes, I would be so keen for this, especially in light of bug 12325.

Owen, do you have a branch for this sitting somewhere or even some tips? I'm hating on Greybox so much at the moment...
Comment 2 David Cook 2014-05-30 02:26:06 UTC
Actually, after getting frustrated with Greybox in IE 8, I used a Bootstrap modal populated via Ajax, which seems to work a treat cross-browser. 

Since it seems we're already Bootstrap-ifying the staff client, I figure it might make more sense to go with Bootstrap now rather than jQueryUI.

Since this is a rather old bug, I imagine that's probably your thoughts too, Owen?
Comment 3 Owen Leonard 2016-05-26 13:51:07 UTC
The simplest Greybox uses have been taken care of. The remaining cases will all require more complex AJAX handling because of the kind of in-modal operations used:

Localization:

admin/itemtypes.tt
admin/localization.tt

Label and card export:

labels/label-edit-batch.tt
labels/label-manage.tt
patron_lists/lists.tt
patroncards/edit-batch.tt
patroncards/manage.tt
Comment 4 Katrin Fischer 2019-04-28 13:23:29 UTC
Hi Owen, do you want to leave this open?
Comment 5 Owen Leonard 2019-04-29 12:39:50 UTC
I would really like to replace Greybox. The original site (http://orangoo.com/labs/GreyBox/) appears to no longer exist. I realize it has been working fine, but I hate to be using something which is abandoned.
Comment 6 Katrin Fischer 2019-04-29 12:57:06 UTC
(In reply to Owen Leonard from comment #5)
> I would really like to replace Greybox. The original site
> (http://orangoo.com/labs/GreyBox/) appears to no longer exist. I realize it
> has been working fine, but I hate to be using something which is abandoned.

I agree, but not sure how hard it would be. Do you need help?
Comment 7 Katrin Fischer 2023-07-29 13:55:52 UTC
Should we pick this up again? It looks like there is only one pending bug still open, but not sure if these were all cases.
Comment 8 David Cook 2023-09-05 02:43:19 UTC
(In reply to Katrin Fischer from comment #7)
> Should we pick this up again? It looks like there is only one pending bug
> still open, but not sure if these were all cases.

I think there are some other cases in the staff interface but not too many, and I can only see 1 in the OPAC... bug 34710