From: Kathy Lussier <klussier@masslnc.org>
Date: Wed, 13 Sep 2017 18:12:44 +0000 (-0400)
Subject: LP#1716475: Add label for mobile carrier select menu
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2016047f2a73acb76e7122b2c17a5810fea8c771;p=evergreen%2Fpines.git

LP#1716475: Add label for mobile carrier select menu

Replace the empty string with a label in the SMS carrier select menu due to
issues that arose when putting an empty string in the translation file. It
also is good usability.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
---

diff --git a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2 b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
index 7eb529710f..6ffd41825a 100644
--- a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
+++ b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
@@ -19,7 +19,7 @@
 %]
 [% IF NOT sms_carrier_hide_label; l('Mobile carrier:'); END; %]
 <select name="sms_carrier">
-    <option value="">[% l('') %]</option>
+    <option value="">[% l('Please select your mobile carrier') %]</option>
     [% FOR carrier IN carriers.sort('name','region') -%]
     <option value='[% carrier.id | html %]'[%
         default_carrier == carrier.id ? ' selected="selected"' : ''