The "MARC" and "Card" links on the cataloging search results page should trigger previews in a Bootstrap modal instead of a Greybox modal.
Created attachment 45419 [details] [review] Bug 15309 - Use Bootstrap modal for cataloging search MARC and Card preview In order to reduce the number of dependencies for common interactions we should attempt to use Bootstrap modals in place of Greybox ones. This patch modifies the cataloging search results page so that "MARC" and "Card" previews are displayed in Bootstrap modals. To test: 1. Apply the patch and navigate to Cataloging. 2. Perform a search which will return results from the catalog. 3. Test the "MARC" and "Card" links for multiple results and confirm that the correct data is shown each time. 4. Perform a search which will return results from the reservoir. 5. Test the corresonding "MARC" and "Card" links for those results.
Created attachment 45430 [details] [review] Bug 15309 - Use Bootstrap modal for cataloging search MARC and Card preview In order to reduce the number of dependencies for common interactions we should attempt to use Bootstrap modals in place of Greybox ones. This patch modifies the cataloging search results page so that "MARC" and "Card" previews are displayed in Bootstrap modals. To test: 1. Apply the patch and navigate to Cataloging. 2. Perform a search which will return results from the catalog. 3. Test the "MARC" and "Card" links for multiple results and confirm that the correct data is shown each time. 4. Perform a search which will return results from the reservoir. 5. Test the corresonding "MARC" and "Card" links for those results. Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Created attachment 45440 [details] [review] [PASSED QA] Bug 15309 - Use Bootstrap modal for cataloging search MARC and Card preview In order to reduce the number of dependencies for common interactions we should attempt to use Bootstrap modals in place of Greybox ones. This patch modifies the cataloging search results page so that "MARC" and "Card" previews are displayed in Bootstrap modals. To test: 1. Apply the patch and navigate to Cataloging. 2. Perform a search which will return results from the catalog. 3. Test the "MARC" and "Card" links for multiple results and confirm that the correct data is shown each time. 4. Perform a search which will return results from the reservoir. 5. Test the corresonding "MARC" and "Card" links for those results. Signed-off-by: Galen Charlton <gmcharlt@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master, thanks Owen! On a side note, it would be nice if the cataloging modal was large like the stage records modal. Not sure what is making it smaller. Maybe this should be considered a minor bug?
Switching severity to 'enhancement' as the patch does not fix any bug