If you go to OPAC detail and look for the #libraryInfoModal you'll notice it is outside of the <html> tag. Run the HTML through a validator and it will come back with an error.
Created attachment 152017 [details] [review] Bug 33902: Move modal inside HTML body To test: -Go Administration > Libraries and add a URL for a branch. -Also add some OPAC info: ( via HTML Customizations ) -Now find a record belonging to that branch and bring up the OPAC detail page. -In the holdings table there should be a link under the 'Current library' column. -Clicking it should bring up a modal. -Make sure the modal pops up correctly, can be dismissed, and the 'Visit web site' link works. Nothing about the appearance or behavior of the page should change.
Created attachment 152022 [details] [review] Bug 33902: Move modal inside HTML body To test: -Go Administration > Libraries and add a URL for a branch. -Also add some OPAC info: ( via HTML Customizations ) -Now find a record belonging to that branch and bring up the OPAC detail page. -In the holdings table there should be a link under the 'Current library' column. -Clicking it should bring up a modal. -Make sure the modal pops up correctly, can be dismissed, and the 'Visit web site' link works. Nothing about the appearance or behavior of the page should change. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 152031 [details] [review] Bug 33902: Move modal inside HTML body To test: -Go Administration > Libraries and add a URL for a branch. -Also add some OPAC info: ( via HTML Customizations ) -Now find a record belonging to that branch and bring up the OPAC detail page. -In the holdings table there should be a link under the 'Current library' column. -Clicking it should bring up a modal. -Make sure the modal pops up correctly, can be dismissed, and the 'Visit web site' link works. Nothing about the appearance or behavior of the page should change. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: David Cook <dcook@prosentient.com.au>
The dependency is not PQA. Can someone check?
(In reply to Tomás Cohen Arazi from comment #4) > The dependency is not PQA. Can someone check? I am removing the dependency. Neither of these patch sets conflict with one another.
Pushed to master for 23.11. Nice work everyone, thanks!
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to oldstable for 22.11.x