Bug 12399

Summary: opaccredits printing at top on printable version
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: OPACAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: brendan, chris, tomascohen, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: footer printing near top
Bug 12399 - opaccredits printing at top on printable version
Bug 12399 - opaccredits printing at top on printable version
[PASSED QA] Bug 12399 - opaccredits printing at top on printable version
Bug 12399: Small change to remove the whole paging
Bug 12399 - opaccredits printing at top on printable version
Bug 12399: Small change to remove the whole paging

Description Nicole C. Engard 2014-06-10 12:55:27 UTC
Created attachment 28758 [details]
footer printing near top

When printing the OPAC in Bootstrap on 3.14 and master the footer prints at the top - see attached.
Comment 1 Marc Véron 2014-06-10 19:51:36 UTC
I can confirm this behaviour. It displays the Koha credits at the wrong place as well.

To reproduce: In Opac with Bootstrap theme, do a search with many results.

It seems that the credits should not display at all in print version, because in opac-tmpl/bootstrap/css/print.css, line 205, a class .koha_url gets "display:none". 
However, there is no class koha_url in the HTML code, but an Id koha_url that should be addressed as #koha_url

Regarding the Id opaccredits (containing the contents from system preference opaccredits), I do not find any traces in print.css
Comment 2 Marc Véron 2015-02-06 16:39:58 UTC
Still valid in current master 3.19.00.008

Additionally, Language selector displays at top of page 2 and overlays other text.
Comment 3 Marc Véron 2015-02-06 17:51:43 UTC Comment hidden (obsolete)
Comment 4 Brendan Gallagher 2015-03-02 08:00:37 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2015-03-05 08:40:50 UTC
Created attachment 36536 [details] [review]
[PASSED QA] Bug 12399 - opaccredits printing at top on printable version

This patch introduces a new class "noprint" in koha-tmpl/opac-tmpl/bootstrap/css/print.css
With this class, unwanted elements can be hidden when printing.

To test:
In OPAC, do a search with many results (> 1 Page)
Print page or do a print preview
See that footer elements are not displayed correctly (as of screenshot in comment #1) and that on top of page 2 the language selector overlays the list
Apply patch
Repeat steps above.
Verify that printed page looks much nicer now by hiding unwanted elements (including e.g. RSS icon, toolbar etc.)

This mechanism can be used on other pages as well.

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 6 Katrin Fischer 2015-03-05 08:41:26 UTC Comment hidden (obsolete)
Comment 7 Brendan Gallagher 2015-03-05 08:46:54 UTC Comment hidden (obsolete)
Comment 8 Brendan Gallagher 2015-03-05 08:47:10 UTC
Created attachment 36539 [details] [review]
Bug 12399: Small change to remove the whole paging

Previous patch only removed the next and previous buttons.
This patch changes it a bit to remove the whole paging from
the print view of the page.

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Comment 9 Tomás Cohen Arazi 2015-03-15 12:14:46 UTC
Patches pushed to master.

Thanks Marc!
Comment 10 Chris Cormack 2015-03-21 22:03:48 UTC
String changes held for 3.18.6
Comment 11 Chris Cormack 2015-03-23 07:33:21 UTC
Pushed to 3.18.x will be in 3.18.6