Created attachment 15160 [details] screenshot While testing new features in 3.10.2 I added following HTML code (sample opening times in German) to the OPAC info field of one of the branches: <b>Öffnungszeiten</b> <table> <tr> <td>Montag - Donnerstag</td><td>8-20 Uhr</td> </tr> <tr> <td>Freitag</td><td>8-18 Uhr</td> </tr> </table> In OPAC the tooltip shows up when hovering the mouse over the library name, but it's below the footer on the page and for that sometimes outside of view depending on the size of the browser window (see screenshot). It would be nice if the tooltip could be positioned closer to the library name/items table/mouse cursor.
This feature broke when we upgraded tabs handling to the latest version of jQueryUI. Something about the way tabs are displayed confuses how the mouse position is reported by JavaScript. This should probably be reworked to use one of the UI libraries we already have (or will have): jQueryUI or Bootstrap. A jQueryUI version will require a jQueryUI upgrade, and of course a Bootstrap version will require approval of the Bootstrap patches on Bug 9420.
Hi Owen, I noticed 9420 has been pushed, maybe you can take another look at this?
Created attachment 15923 [details] [review] Bug 9572 - Opac info tooltip from branches is not well positioned This patch removes the jQuery plugin previously used to display the library information tooltip on opac-detail.pl and implements the functionality using jQueryUI's tooltip widget. jQueryUI has been upgraded to 1.9.2 in the process, which required some ammendments to jQueryUI-related CSS. To test, make sure at least one of your libraries has "OPAC info" stored as part of its configuration. View a record in the OPAC which is owned by that library. Hover over the library's name in the holdings table and you should see a properly-positioned tooltip showing the "OPAC info." Also test other jQueryUI-related functionality to confirm that the upgrade didn't break anything: Tabs and datepickers should still work correctly.
Handhover is correct. But Opac Info also appears under the library name. Is it normal ?
I've seen it with Fridolyn, it s about sandbox using specific CSS. I ve tested it without it, all is OK.
Patch tested with a sandbox, by Frederic Durand <frederic.durand@unilim.fr>
Created attachment 16337 [details] [review] Bug 9572 - Opac info tooltip from branches is not well positioned This patch removes the jQuery plugin previously used to display the library information tooltip on opac-detail.pl and implements the functionality using jQueryUI's tooltip widget. jQueryUI has been upgraded to 1.9.2 in the process, which required some ammendments to jQueryUI-related CSS. To test, make sure at least one of your libraries has "OPAC info" stored as part of its configuration. View a record in the OPAC which is owned by that library. Hover over the library's name in the holdings table and you should see a properly-positioned tooltip showing the "OPAC info." Also test other jQueryUI-related functionality to confirm that the upgrade didn't break anything: Tabs and datepickers should still work correctly. Signed-off-by: Frederic Durand <frederic.durand@unilim.fr>
Created attachment 16925 [details] [review] [PASSED QA] Bug 9572 - Opac info tooltip from branches is not well positioned This patch removes the jQuery plugin previously used to display the library information tooltip on opac-detail.pl and implements the functionality using jQueryUI's tooltip widget. jQueryUI has been upgraded to 1.9.2 in the process, which required some ammendments to jQueryUI-related CSS. To test, make sure at least one of your libraries has "OPAC info" stored as part of its configuration. View a record in the OPAC which is owned by that library. Hover over the library's name in the holdings table and you should see a properly-positioned tooltip showing the "OPAC info." Also test other jQueryUI-related functionality to confirm that the upgrade didn't break anything: Tabs and datepickers should still work correctly. Signed-off-by: Frederic Durand <frederic.durand@unilim.fr> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Works nicely in prog theme, feature seems not to be available for ccsr yet.
Created attachment 17123 [details] [review] Bug 9572 follow-up: duplicate jQueryUI upgrade in CCSR
This patch has been pushed to master.