Bug 25007: (follow-up) Fix the way image check alters layout
authorOwen Leonard <oleonard@myacpl.org>
Sat, 28 Mar 2020 17:42:50 +0000 (17:42 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 16 Apr 2020 18:24:21 +0000 (19:24 +0100)
commit429c57d8d6fd88053f2bff93465ab78f1ef82aa3
tree03a5a81339d8ba41047c88e46dbbb8965d85494f
parenta348fc18e414d3d68916ae863c18344233eafdc9
Bug 25007: (follow-up) Fix the way image check alters layout

A question in the bug report asks: Why is is necessary to change the
Bootstrap class to col-xs-12? The JavaScript which checks for the
existence of an Amazon cover image is designed to remove the Amazon
cover's container if there is none. Changing the class of
cover image column is gone.

However, the check is incorrect because it doesn't account for Local
Covers. There might be both a 1 x 1 pixel Amazon image and a local cover
image, so the layout shouldn't be changed unless there are no images
present.

To test, apply the patch and view the bibliographic details page of
various titles including:
- Amazon cover present, no local cover
- No Amazon cover, local cover image present
- Both Amazon and local cover image

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt