Use a placeholder attribute for search input rather than custom JS

VegGuide · Sep 15, 2014 · dbbe5c8 · dbbe5c8
1 parent fabf1ea
commit dbbe5c8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/share/js-source/VegGuide/SitewideSearch.js b/share/js-source/VegGuide/SitewideSearch.js
@@ -7,8 +7,6 @@ if ( typeof VegGuide == "undefined" ) {
 
 VegGuide.SitewideSearch = {};
 
-VegGuide.SitewideSearch._defaultValue = "name, city, or address";
-
 VegGuide.SitewideSearch.instrumentPage = function () {
     var input = $("sitewide-search-input");
 
@@ -17,20 +15,12 @@ VegGuide.SitewideSearch.instrumentPage = function () {
         return;
     }
 
-    if ( input.value == undefined || ! input.value.length ) {
-        input.value = VegGuide.SitewideSearch._defaultValue;
-    }
-
     var help_div = $("sitewide-search-help");
 
     DOM.Events.addListener(
         input,
         "focus",
         function (e) {
-            if ( e.target.value == VegGuide.SitewideSearch._defaultValue ) {
-                e.target.value = "";
-            }
-
             VegGuide.SitewideSearch._positionHelpDiv( e.target, help_div );
             DOM.Element.show(help_div);
         }

diff --git a/share/mason/sitewide-search.mas b/share/mason/sitewide-search.mas
@@ -1,6 +1,6 @@
   <div id="sitewide-search-text-and-form">
     <form id="sitewide-search" action="/site/search">
-      <input id="sitewide-search-input" class="text" type="text" name="search_text" value="<% $search_text | h %>" />
+      <input id="sitewide-search-input" class="text" type="text" name="search_text" value="<% $search_text | h %>" placeholder="name, city, or address" />
       <button class="action-button" type="submit"><% $button | h %></button>
     </form>
   </div>