Bugzilla – Attachment 25737 Details for
Bug 11686
"please note your reason here..." does not get translated
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 11686 - "please note your reason here..." does not get translated
Bug-11686---please-note-your-reason-here-does-not-.patch (text/plain), 2.87 KB, created by
Kyle M Hall (khall)
on 2014-02-28 17:47:13 UTC
(
hide
)
Description:
Bug 11686 - "please note your reason here..." does not get translated
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2014-02-28 17:47:13 UTC
Size:
2.87 KB
patch
obsolete
>From 605a953794fde6f983769639a326491b03f00f64 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 5 Feb 2014 10:35:47 -0500 >Subject: [PATCH] Bug 11686 - "please note your reason here..." does not get translated > >Through some quirk of the translation script a predefined value on a >text input is not properly translated even though the string appears in >the po file. On the suggestions form the problem can be solved by using >the HTML5 placeholder attribute. > >To test you must have existing suggestions to manage. Apply the patch >and: > >- Update a translation by running tranlsate update [language code] >- Install/reinstall the translation by running translate install > [language code] > >- In the English templates: > > - Navigate to Acquisitions -> Suggestions and find the "Mark selected > as:" section below the table of existing suggestions. > - Select "Other" from the "choose a reason" field. A text input field > should appear with the placeholder text "please note your reason > here." > >- In the templates you updated and installed, follow the same procedure > above. The placeholder text should now appear with the correct > translation. >- In both English and translated templates using this bulk-status option > should save your chosen status correctly with the correct reason, > whether that reason be predefined or manually entered. > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > .../prog/en/modules/suggestion/suggestion.tt | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt >index ecd8a8e..812436b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt >@@ -74,7 +74,6 @@ $(document).ready(function() { > e.preventDefault(); > }); > $("#other_reason[% suggestion.suggestiontype %]").hide(); >- $("input[name*=other_reason[% suggestion.suggestiontype %]]").focus(function(){ $(this).val(""); }); > $("#reason[% suggestion.suggestiontype %]").change(function(){ > if($(this).val() == "other"){ > $(this).hide(); >@@ -580,7 +579,7 @@ h4.local_collapse a { font-size : 80%; text-decoration: none; } fieldset.brief o > </select> > > <span id="other_reason[% suggestion.suggestiontype %]"> >- <input type="text" size="31" id="select-other_reason[% suggestion.suggestiontype %]" name="other_reason[% suggestion.suggestiontype %]" value="please note your reason here..." /> >+ <input type="text" size="31" id="select-other_reason[% suggestion.suggestiontype %]" name="other_reason[% suggestion.suggestiontype %]" placeholder="please note your reason here..." /> > <a href="#back[% suggestion.suggestiontype %]">Cancel</a> > </span> > >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11686
:
25057
|
25737
|
26157