[Bug 20268] New: CSS regression
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Bug ID: 20268 Summary: CSS regression Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Staff Client Assignee: koha-bugs@lists.koha-community.org Reporter: victor.grousset@biblibre.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com Created attachment 72068 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72068&action=edit Screenshot of the new white gap on the top of the page. There is a new white gap on the top of the page, see screenshot. git bisect allowed to find that this was caused by commit 3eb50270b1a02fe98cde767ad0240b2943e23321 Bug 19981: Switch single-column templates to Bootstrap grid: Course reserves -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Victor Grousset/tuxayo <victor.grousset@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=19981 Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Victor Grousset/tuxayo <victor.grousset@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|CSS regression |CSS regression: white gap | |on the top of the staff | |pages -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |oleonard@myacpl.org |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #1 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Comes from staff-global.css : .main { margin-top: 1em; } main class is on body in intranet-main.tt. It is usually in : <div class="main container-fluid"> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |7760 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7760 [Bug 7760] Add ids and classes to every staff page to help with customizaton -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 --- Comment #2 from Victor Grousset/tuxayo <victor.grousset@biblibre.com> --- @Fridolin: Why does it depends on bug 7760 ? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 72310 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72310&action=edit Bug 20268: CSS regression: white gap on the top of the staff pages This patch makes template and CSS changes so that the staff client main page doesn't get an unwanted top margin from the Bootstrap-grid conversion. - The unused "main" class is removed from the login page - The "main" class on the staff client home page is changed to "intranet-main." - The CSS for the staff client home page has been modified accordingly. To test, apply the patch and clear your browser cache if necessary. - Open the staff client login page. It should look as it always does. - Log in and check the style of the main page. There should be no white margin at the top of the page. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #72310|0 |1 is obsolete| | --- Comment #4 from Claire Gravely <claire_gravely@hotmail.com> --- Created attachment 72339 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72339&action=edit Bug 20268: CSS regression: white gap on the top of the staff pages This patch makes template and CSS changes so that the staff client main page doesn't get an unwanted top margin from the Bootstrap-grid conversion. - The unused "main" class is removed from the login page - The "main" class on the staff client home page is changed to "intranet-main." - The CSS for the staff client home page has been modified accordingly. To test, apply the patch and clear your browser cache if necessary. - Open the staff client login page. It should look as it always does. - Log in and check the style of the main page. There should be no white margin at the top of the page. Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #72339|0 |1 is obsolete| | --- Comment #5 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 72344 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72344&action=edit Bug 20268: CSS regression: white gap on the top of the staff pages This patch makes template and CSS changes so that the staff client main page doesn't get an unwanted top margin from the Bootstrap-grid conversion. - The unused "main" class is removed from the login page - The "main" class on the staff client home page is changed to "intranet-main." - The CSS for the staff client home page has been modified accordingly. To test, apply the patch and clear your browser cache if necessary. - Open the staff client login page. It should look as it always does. - Log in and check the style of the main page. There should be no white margin at the top of the page. Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alexbuckley@catalyst.net.nz --- Comment #6 from Claire Gravely <claire_gravely@hotmail.com> --- *** Bug 20328 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #7 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 18.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 --- Comment #8 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- A test is failing, please fix ASAP: xt/tt_valid.t .. 1/3 # Failed test '<body> tag with id and class attributes' # at xt/tt_valid.t line 91. # Files list: # intranet-tmpl/prog/en/modules/auth.tt: 15 https://jenkins.koha-community.org/job/Koha_Master_D8/398/ -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 --- Comment #9 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 73180 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=73180&action=edit Bug 20268 (follow-up) Correct template error The login template must have a class attribute on the body tag in order for the template to pass tests. To test, apply the patch and confirm that the staff client login form still looks the same. For further confirmation you could update the IntranetUserCSS system preference with something like this: .main_main-auth { background-color: #CCF; } The login form should now have a different background color. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |19981 See Also|https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=19981 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19981 [Bug 19981] Switch single-column templates to Bootstrap grid: Course reserves -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20268 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #10 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Owen Leonard from comment #9)
Created attachment 73180 [details] [review] Bug 20268 (follow-up) Correct template error
Pushed to master. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org