Just like Bug 19865 we should fix this view in the z39.50 acquisition search so one doesn't have to scroll all the way to the bottom in order to see it
Created attachment 129914 [details] [review] Bug 29961: Make horizontal scroll bar always visible on acqui/z3950_search.pl To test: 1. In Acquisitions make a basket and add something "From an external source" 2. Search for a title and click on the 'MARC' button, make sure the MARC prview is both tall and wide. 3. You will not see the horizontal scroll bar unless you scroll all the way to the bottom of the preview. 4. Apply patch. 5. Try steps 1 & 2 again, this time you should see the horizontal scroll bar as soon as the MARC prveiw loads. Note: I put the CSS on multiple lines for better readability
Created attachment 135459 [details] [review] Bug 29961: Make horizontal scroll bar always visible on acqui/z3950_search.pl To test: 1. In Acquisitions make a basket and add something "From an external source" 2. Search for a title and click on the 'MARC' button, make sure the MARC prview is both tall and wide. 3. You will not see the horizontal scroll bar unless you scroll all the way to the bottom of the preview. 4. Apply patch. 5. Try steps 1 & 2 again, this time you should see the horizontal scroll bar as soon as the MARC prveiw loads. Note: I put the CSS on multiple lines for better readability Signed-off-by: David Nind <david@davidnind.com>
As you are reusing the same code as bug 19865 it would have been better to have the rules defined in a .css file.
Created attachment 135477 [details] [review] Bug 29961: Make horizontal scroll bar always visible on acqui/z3950_search.pl To test: 1. In Acquisitions make a basket and add something "From an external source" 2. Search for a title and click on the 'MARC' button, make sure the MARC prview is both tall and wide. 3. You will not see the horizontal scroll bar unless you scroll all the way to the bottom of the preview. 4. Apply patch. 5. Try steps 1 & 2 again, this time you should see the horizontal scroll bar as soon as the MARC prveiw loads. Note: I put the CSS on multiple lines for better readability Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 22.11. Nice work everyone, thanks!
Pushed to 22.05.x for 22.05.01
thx, this is pushed to 21.11.x for 21.11.07
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed.