From 8b3b4b4cc77e6b77a7a79a80793dee3b3d8c4b22 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 5 Mar 2012 14:47:51 -0500 Subject: [PATCH] Bug 7644 - Invalid markup in staff client language chooser This patch removes invalid markup from the language chooser in the intranet. It also copies the markup and style of the staff client include to the OPAC so the two are more consistent. I hope that will make future changes and debugging a little easier. I believe this patch also fixes Bug 7366, "Language chooser display problems." Signed-off-by: Katrin Fischer Patch fixes bug 7366, but there is still a difference between staff and OPAC. Installed languages: en, en-NZ, de-DE, fr-FR Activated langauges: en, en-NZ, fr-FR OPC shows English, staff shows en-NZ as label for the list. Signed-off-by: Paul Poulain Signed-off-by: Jared Camins-Esakov --- .../prog/en/css/staff-global.css | 9 ++- .../prog/en/includes/intranet-bottom.inc | 4 +- koha-tmpl/opac-tmpl/prog/en/css/opac.css | 61 +++++++++++++----- .../prog/en/includes/opac-bottom.inc | 64 ++++++++++--------- 4 files changed, 87 insertions(+), 51 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css index c61f946022..0c579b980f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css +++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css @@ -1734,10 +1734,15 @@ ul#i18nMenu { ul#i18nMenu li { border-left : 1px solid #CCC; display : inline; + float: left; list-style : none; padding : 0 10px; - } - +} + +ul#i18nMenu li.yuimenuitem { + float: none; +} + ul#i18nMenu li:first-child { border-left : 0; } diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/intranet-bottom.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/intranet-bottom.inc index ebe7f370cd..7dd5d178c2 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/intranet-bottom.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/intranet-bottom.inc @@ -5,7 +5,7 @@
diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css index 21dc08ff39..4847533245 100644 --- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css +++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css @@ -1461,29 +1461,58 @@ div#changelanguage{ div#changelanguage a { font-weight : normal; - padding : .5em; -} - -div#changelanguage a.more { - background-image:url(../../images/more-up-arrow.gif); - background-position:right center; - background-repeat:no-repeat; - padding-right:13px; + padding : .5em 0; } - + div#changelanguage li.yuimenuitem { - font-size : 120%; - font-weight : normal; - margin : 0; - padding : 0 1em; + font-size: 120%; + font-weight: normal; + margin: 0; + padding: 0 1em; } - div#changelanguage li.yuimenuitem a.yuimenuitemlabel { + padding: 0; +} + +ul#i18nMenu { + margin : 0; + padding : .2em 0; + } + +ul#i18nMenu li { + border-left : 1px solid #CCC; + display : inline; + list-style : none; + padding : 0 .4em; + } + +ul#i18nMenu li:first-child { + border-left : 0; + } + +ul#i18nMenu li ul li { + border : 0; + display : block; padding : 0; + } + + +ul#i18nMenu li.more a { + background-image:url(../../images/more-up-arrow.gif); + background-position:right center; + background-repeat:no-repeat; + padding-right: 1.3em; } -div.lang{ - float:left; +ul#i18nMenu li.more ul li a { + background-image : none; + padding: 0 1.3em; +} + +span.lang{ + float:left; + border-right : 1px solid black; + padding : 0 .5em; } #cartDetails,#cartUpdate,#holdDetails,#listsDetails { diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/opac-bottom.inc b/koha-tmpl/opac-tmpl/prog/en/includes/opac-bottom.inc index a264af3ef4..46e195dac6 100644 --- a/koha-tmpl/opac-tmpl/prog/en/includes/opac-bottom.inc +++ b/koha-tmpl/opac-tmpl/prog/en/includes/opac-bottom.inc @@ -6,43 +6,46 @@ [% IF ( opaclanguagesdisplay || OpacKohaUrl ) %] -
-[% IF ( languages_loop && opaclanguagesdisplay ) %] - [% UNLESS ( one_language_enabled ) %] -
Languages: 
- [% FOREACH languages_loo IN languages_loop %] -
- [% IF ( languages_loo.group_enabled ) %] - [% IF ( languages_loo.plural ) %] - [% IF ( languages_loo.native_description ) %][% languages_loo.native_description %][% ELSE %][% languages_loo.rfc4646_subtag %][% END %][% UNLESS ( loop.last ) %] | [% END %] -
-
+ [% ELSE %] - [% IF ( languages_loo.native_description ) %][% languages_loo.native_description %][% ELSE %][% languages_loo.rfc4646_subtag %][% END %] + [% IF ( languages_loo.group_enabled ) %] + [% IF ( languages_loo.current ) %] +
  • [% IF ( languages_loo.native_description ) %][% languages_loo.native_description %][% ELSE %][% languages_loo.rfc4646_subtag %][% END %]
    • + [% ELSE %] +
  • [% IF ( languages_loo.native_description ) %][% languages_loo.native_description %][% ELSE %][% languages_loo.rfc4646_subtag %][% END %]
    • + [% END %] + [% END %] + [% END %] [% END %] - [% END %] + [% END %] - [% END %][% UNLESS ( loop.last || languages_loo.plural ) %] | [% END %] +
    [% END %] [% END %] -[% END %] + [% IF ( OpacKohaUrl ) %] @@ -50,7 +53,6 @@ [% END %] -
    [% END %] [% IF ( Babeltheque ) %] -- 2.39.5