The JS function LoadResultsCovers is not used from the Staff interface. It is supposed to load the local cover images in JS but actually we are embedding the thumbnail in the img tag.
Created attachment 159783 [details] [review] Bug 35557: Remove LoadResultsCovers The JS function LoadResultsCovers is not used from the Staff interface. It is supposed to load the local cover images in JS but actually we are embedding the thumbnail in the img tag. Test plan: Confirm that LoadResultsCovers was called but had no effect. Note that the selector ^local-thumbnail only exists in shelfbrowser.inc but results.js is only used from results.tt (not shelfbrowser.inc) Current status: ASSIGNED
Created attachment 159784 [details] [review] Bug 35557: Remove LoadResultsCovers The JS function LoadResultsCovers is not used from the Staff interface. It is supposed to load the local cover images in JS but actually we are embedding the thumbnail in the img tag. Test plan: Confirm that LoadResultsCovers was called but had no effect. Note that the selector ^local-thumbnail only exists in shelfbrowser.inc but results.js is only used from results.tt (not shelfbrowser.inc)
Created attachment 159913 [details] [review] Bug 35557: Remove LoadResultsCovers The JS function LoadResultsCovers is not used from the Staff interface. It is supposed to load the local cover images in JS but actually we are embedding the thumbnail in the img tag. Test plan: Confirm that LoadResultsCovers was called but had no effect. Note that the selector ^local-thumbnail only exists in shelfbrowser.inc but results.js is only used from results.tt (not shelfbrowser.inc) Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 160225 [details] [review] Bug 35557: Remove LoadResultsCovers The JS function LoadResultsCovers is not used from the Staff interface. It is supposed to load the local cover images in JS but actually we are embedding the thumbnail in the img tag. Test plan: Confirm that LoadResultsCovers was called but had no effect. Note that the selector ^local-thumbnail only exists in shelfbrowser.inc but results.js is only used from results.tt (not shelfbrowser.inc) Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Created attachment 160226 [details] [review] Bug 35557: Remove references to unused PREF_LocalCoverImages Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Pushed for 24.05! Well done everyone, thank you!
Pushed to 23.11.x for 23.11.02
Backported to 23.05.x for upcoming 23.05.08