[koha-commits] main Koha release repository branch 3.8.x updated. v3.08.10-6-g9a82021

[Git repo owner]

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, 3.8.x has been updated
       via  9a820217542409054da04511297d5027dd1e9ec8 (commit)
       via  aef12235f52e9f7465cdbaefd52986d6f82b7ecc (commit)
       via  338148f3e356bd22e9457643309c9f47f0adac8d (commit)
       via  2771ef44e1556d0b2c8807aa70c79bba48972404 (commit)
      from  c6fcac6e994a574b18b62933b6eafe9bb5aefefa (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 9a820217542409054da04511297d5027dd1e9ec8
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Feb 11 12:12:58 2013 -0500

    Bug 9385 - patron add labels right aligned in chrome
    
    The <legend> element has been given right alignment which seems to only
    be respected by WebKit-based browsers. Users of Chrome and Safari see a
    right-aligned legend while others see a left-aligned one. Given that the
    legend is often applied to fieldsets spanning much of the screen it
    seems wiser to have them left-aligned.
    
    This patch removes "text-align:right" from the CSS so that <legend> will
    inherit the default left alignment. To test, view any page with a legend
    element, for instance the patron add form. In Firefox, Chrome, Safari,
    IE, etc. the legend ("Patron identity," "Main address," etc.) should be
    left-aligned in the fieldset.
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    Comment: Works as described. Tested in IE, Chrome, Firefox, Opera.
    No errors.
    
    Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
    All tests pass
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit aef12235f52e9f7465cdbaefd52986d6f82b7ecc
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Thu Feb 14 15:23:01 2013 +0100

    Bug 9545 - Followup Fix import MARC bibliographic framework text label
    
    The export action for the default framework did not work.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 338148f3e356bd22e9457643309c9f47f0adac8d
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Feb 11 05:49:05 2013 -0500

    Bug 9545 - Fix import MARC bibliographic framework text label
    
    This patch improves the labeling of import controls on the
    MARC frameworks page by making import options more explicit. Other
    changes:
    
    - Form markup fixed to match standard form constructions
    - JavaScript corrected to make more embedded English strings
      translatable.
    - Form labels made clickable by making "for" attributes match unique
      form field ids.
    - Some warning re-worded for readability and translatability.
    - Large "loading" icon replaced with more proportional smaller one.
    
    To test, view the MARC frameworks page (admin/biblio_framework.pl) and
    test Export and Import functionality. Clicking each should display a
    form with options clearly labeled. Import and Export operations should
    complete correctly.
    
    Signed-off-by: David Cook <dcook at prosentient.com.au>
    
    Works as described, although I see what Owen means about it being a good candidate for replacement with a modal dialog box in the future.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 2771ef44e1556d0b2c8807aa70c79bba48972404
Author: David Cook <dcook at prosentient.com.au>
Date:   Mon Feb 11 16:24:20 2013 +1100

    Bug 9585 - Fix spelling mistake in default tab conditional for opac-detail
    
    There is just a small typo where one of the results of a conditional is
    "subscription" rather than "subscriptions".
    
    If this comes up, the correct tab (subscriptions) will not be chosen.
    
    I'm not sure what will happen...it'll probably just have the
    "holdings", "subscriptions", and "serial collection" tabs all default
    to closed with none of them being open.
    
    Test Plan:
    
    The easiest way to test is just to double-check the other strings in the
    conditional which say "subscriptions" rather than "subscription".
    
    You can also look at opac-detail.tt where the param is shown, and note
    that there is no condition for "subscription" - just "subscriptions".
    
    Any tests other than that would be overly complex for such a trivial
    patch.
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Jared Camins-Esakov <jcamins at cpbibliography.com>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

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

Summary of changes:
 .../intranet-tmpl/prog/en/css/staff-global.css     |   18 ++---
 .../prog/en/modules/admin/biblio_framework.tt      |   84 +++++++++++---------
 opac/opac-detail.pl                                |    2 +-
 3 files changed, 52 insertions(+), 52 deletions(-)


hooks/post-receive
-- 
main Koha release repository