pkp/pkp-lib#1807 b) locale placeholder for keyword form input fields

[bozana]

s. b) here: #1807

[NateWr]

This looks good. Placeholders aren't totally reliable as labels for a field due to some accessibility issues, but I think that doesn't matter here because I'm not sure the tag-it component itself is very accessible anyway.

I had just a couple of changes. The first uses the locale name instead of the code. The second fixes an unrelated issue with two globe icons appearing in the tag-it fields:

diff --git a/templates/form/keywordInput.tpl b/templates/form/keywordInput.tpl
index b225890..df52a58f 100755
--- a/templates/form/keywordInput.tpl
+++ b/templates/form/keywordInput.tpl
@@ -14,7 +14,7 @@
                        $(document).ready(function(){ldelim}
                                $("#{$thisFormLocale|escape:jqselector}-{$FBV_id}{$uniqId}").tagit({ldelim}
                                        fieldName: "keywords[{$thisFormLocale|escape}-{$FBV_id|escape}][]",
-                                       {if !$smarty.foreach.formLocales.first && empty($FBV_currentKeywords.$thisFormLocale)}placeholderText: "{$thisFormLocale|escape}",{/if}
+                                       {if !$smarty.foreach.formLocales.first && empty($FBV_currentKeywords.$thisFormLocale)}placeholderText: "{$thisFormLocaleName|escape}",{/if}
                                        allowSpaces: true,
                                        {if $FBV_sourceUrl && !$FBV_disabled}
                                                tagSource: function(search, showChoices) {ldelim}
@@ -47,7 +47,7 @@
                {rdelim});
                </script>
                <span id="{$FBV_id|escape}-localization-popover-container{$uniqId}" class="localization_popover_container pkpTagit">
-                       <ul class="localization_popover_container localizable {if $formLocale != $currentLocale} flag flag_{$formLocale|escape}{/if}" id="{$formLocale|escape}-{$FBV_id|escape}{$uniqId}">
+                       <ul class="localizable {if $formLocale != $currentLocale} flag flag_{$formLocale|escape}{/if}" id="{$formLocale|escape}-{$FBV_id|escape}{$uniqId}">
                                {if $FBV_currentKeywords}{foreach from=$FBV_currentKeywords.$formLocale item=currentKeyword}<li>{$currentKeyword|escape}</li>{/foreach}{/if}
                        </ul>
                        {if $FBV_label_content}<span>{$FBV_label_content}</span>{/if}

[bozana]

👍