[koha-commits] main Koha release repository branch master updated. v3.16.00-612-g9351cde

Thu Sep 18 02:57:34 CEST 2014

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  9351cde90f4be04c98a5760e203ba7db40061dfb (commit)
       via  77e8cf024d4da039356db1e7e1c318d2bf4a77c2 (commit)
       via  3996fa7b47c519cbb7bd9adf3e76af5fd0d7bb24 (commit)
       via  781142224cde1df1c7cd08ca45f0d1ac7b763096 (commit)
       via  f10f88266b7af19205bcd210b060dfd656945e53 (commit)
       via  dbe1e920d3fd448fe5490442fb98f4a6599b71ac (commit)
      from  33d7a28afa927ae190f819ffc98b578dcef4f296 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9351cde90f4be04c98a5760e203ba7db40061dfb
Author: Jacek Ablewicz <abl at biblos.pk.edu.pl>
Date:   Sat Aug 2 10:13:16 2014 +0200

    Bug 12697 - Surplus closing </div> in OPAC advanced search (bootstrap)

    In koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt template,
    there is a surplus closing </div> in line 336 (= line 332, in case
    Bug 12521 is applied first):

       336: </div> <!-- / .span4 -->

    which causes some minor layout problems for this page - i.e.: page
    footer ("Powered by Koha" + custom footer set up in system prefrences -
    if any) is "glued" to the bottom (to the language selection bar);
    unless Bug 12521 is applied first, this affects the placement of the
    2nd search bar as well.

    To reproduce:

    1/ go to the OPAC advanced search page
    2/ zoom out (e.g., use Ctrl-<minus> several times) and obeserve the
    result
    3/ no need for step 2/ (zooming out) when using a display with
    (relatively) high vertical resolution; also, this particullar problem
    gets more apparent/more noticeable with MARC21/USMASRC "subtype"
    options being hidden in the OPAC advanced search (Bug 9043).

    To test:

    1/ ensure that Bug 12521 is applied first/already applied
    2/ apply patch
    3/ confirm that this issue is no longer reroductible

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    On top of 12521, don't know exactly what to reproduce, but
    surplus '</div>' is real (33 open, 34 close).
    Everything looks correct, no koha-qa errors.

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 77e8cf024d4da039356db1e7e1c318d2bf4a77c2
Author: Liz Rea <liz at catalyst.net.nz>
Date:   Thu Jul 3 10:09:22 2014 +1200

    Bug 12521 - Change layout a bit of the opac advanced search

    To test:

    Load up the advanced search and make sure it displays properly, with all of the elements within the main div, and only one container-fluid.

    Patch behaves as expected. Advanced search looks much better now.
    Signed-off-by: Marc Véron <veron at veron.ch>

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Checked visually and with HTML validator.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 3996fa7b47c519cbb7bd9adf3e76af5fd0d7bb24
Author: Marc Véron <veron at veron.ch>
Date:   Wed Sep 10 16:10:21 2014 +0200

    Bug 12882 - Translations: Resolve ambiguity for word "checked" in Z39.50 server administration

    This patch resolves the ambiguity of the word "Checked" for Z39.50
    server administration.

    To test:
    Apply patch
    In staff client, go to Home:Administration:Z39.50/SRU servers
    Result: Column heading of 6th column should show "Preselected" (instead
    of "Checked")

    Edit or create new server for Z39.50 and SRU
    Result: Checkbox underneath "Password" should have label "Preselected
    (searched by default):"

    Click "Help" at top right of screen
    Result: Wording changed for selecting targets as default.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 781142224cde1df1c7cd08ca45f0d1ac7b763096
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Sep 5 09:04:41 2014 -0400

    Bug 12863 [CSS follow-up] News in OPAC: HTML broken with nested p tags and multiple thead in same table

    This patch contains the revised and compiled CSS which will style the
    OPAC's new news markup to match the previous version.

    To test, apply the patch and clear your browser cache if necessary. View
    news in the OPAC and confirm that it is styled like it was before the
    patch making markup changes.

    CSS patch behaves as expected. Thanks to Owen for adding this.
    Signed-off-by: Marc Véron <veron at veron.ch>

    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit f10f88266b7af19205bcd210b060dfd656945e53
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Sep 5 09:02:28 2014 -0400

    Bug 12863 [LESS follow-up] News in OPAC: HTML broken with nested p tags and multiple thead in same table

    This follow-up modifies the LESS file to add style to the new markup to
    match the previous markup. A subsequent patch will include the revised
    and compiled CSS.

    Signed-off-by: Marc Véron <veron at veron.ch>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit dbe1e920d3fd448fe5490442fb98f4a6599b71ac
Author: Marc Véron <veron at veron.ch>
Date:   Wed Sep 3 12:10:26 2014 +0200

    Bug 12863 - News in OPAC: HTML broken with nested p tags and multiple thead in same table

    This patch fixes broken HTML in OPAC news by replacing table display
    with divs.

    To test:

    Create multiple news for OPAC
    Validate with HTML validator or check source code
    Result:
    Error with nested p tags
    Table header thead is repeated for every news (not allowed)
    Bonus testing: Let a screen reader like NVDA read the news and try to
    understand it with closed eyes.

    Apply patch
    Result:
    No more HTML errors in news
    Screen reader reads news properly

    To style you can use classes newscontainer, newsitem, newsheader,
    newsbody and newsfooter.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Works as described, passes tests and QA script.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

-----------------------------------------------------------------------

Summary of changes:
 .../prog/en/modules/admin/z3950servers.tt          |    4 ++--
 .../prog/en/modules/help/admin/z3950servers.tt     |    2 +-
 koha-tmpl/opac-tmpl/bootstrap/css/opac.css         |    2 +-
 .../bootstrap/en/modules/opac-advsearch.tt         |   13 ++++-------
 .../opac-tmpl/bootstrap/en/modules/opac-main.tt    |   16 ++++++-------
 koha-tmpl/opac-tmpl/bootstrap/less/opac.less       |   24 ++++++++++++++++++++
 6 files changed, 40 insertions(+), 21 deletions(-)

hooks/post-receive
-- 
main Koha release repository