prefer to use ids to names; names aren't even valid in divs

[ypp-sc-tools.web-live.git] / yarrg / web / pirate-route

diff --git a/yarrg/web/pirate-route b/yarrg/web/pirate-route
index e414e6e87513cfda4f030fbd1f224bb273e6b349..c80bacffa25b280d6edd666c51d80c2c6f9ab13f 100644 (file)
--- a/yarrg/web/pirate-route
+++ b/yarrg/web/pirate-route
@@ -82,7 +82,7 @@ Enter route (islands, or archipelagoes, separated by |s or commas;
  abbreviations are OK):<br/>
 
 <script type="text/javascript">
  abbreviations are OK):<br/>
 
 <script type="text/javascript">

-tr_uri= "routetextstring?format=json"
+tr_uri= "routetextstring?format=json&type=text/xml"

                + "&ocean=<% uri_escape($a{Ocean}) %>";
 
 tr_timeout=false;
                + "&ocean=<% uri_escape($a{Ocean}) %>";
 
 tr_timeout=false;


@@ -95,7 +95,7 @@ function tr_Later(){
 }
 function tr_Needed(){
   window.clearTimeout(tr_timeout);
 }
 function tr_Needed(){
   window.clearTimeout(tr_timeout);

-  //tr_element= document.getElementsByName('routestring').item(0);
+  tr_element= document.getElementById('routestring');

   //alert(tr_element.name);
   tr_needed= tr_element.value;
   tr_Request();
   //alert(tr_element.name);
   tr_needed= tr_element.value;
   tr_Request();


@@ -115,7 +115,7 @@ function tr_Ready() {
     response= tr_request.responseText;
     //alert('got [[ '+response+' ]]');
     eval('results='+response);
     response= tr_request.responseText;
     //alert('got [[ '+response+' ]]');
     eval('results='+response);

-    toedit= document.getElementsByName('routeresults').item(0);
+    toedit= document.getElementById('routeresults');

     toedit.innerHTML= results['show'];
   }
   tr_request= false;
     toedit.innerHTML= results['show'];
   }
   tr_request= false;


@@ -123,11 +123,10 @@ function tr_Ready() {
 }
 </script>
 
 }
 </script>
 

-<input type="text" name="routestring" size=80
- onchange="tr_element= event.currentTarget; tr_Needed();"
- onkeydown="tr_element= event.currentTarget; tr_Later();"><br>
-<div name="routeresults">&nbsp;</div><br/>
-
+<input type="text" id="routestring" name="routestring" size=80
+ onchange="tr_Needed();"
+ onkeyup="tr_Later();"><br>
+<div id="routeresults">&nbsp;</div><br/>

 % } else {
 
 <%perl>
 % } else {
 
 <%perl>