From 86c2fe532dfa55c7313c2aae3b54c3c96c1df2c8 Mon Sep 17 00:00:00 2001
From: mossroy <mossroy@free.fr>
Date: Sat, 27 Jul 2013 12:14:41 +0200
Subject: [PATCH] Fix on autofocus of prefix field. We don't want it to be
 focused on page load, but each time the archive is selected (either through
 DeviceStorage or manual file selection)

---
 index.html |  2 +-
 js/app.js  | 10 +++++++---
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/index.html b/index.html
index 439d7fc4..8e0b8e12 100644
--- a/index.html
+++ b/index.html
@@ -44,7 +44,7 @@
                 <br />
                 <form id="formTitleSearch" class="form-search">
                     <div class="input-append">
-                        <input type="text" id="prefix" value="" placeholder="Find titles starting with..." class="input-medium search-query" autofocus/>&nbsp;
+                        <input type="text" id="prefix" value="" placeholder="Find titles starting with..." class="input-medium search-query" />&nbsp;
                         <a class="btn" id="searchTitles" href="#"><i class="icon-search"></i></a>
                     </div>
                     <img id="searchingForTitles" src="img/spinner.gif"  style="display: none;"/>
diff --git a/js/app.js b/js/app.js
index 790c1239..489736d8 100644
--- a/js/app.js
+++ b/js/app.js
@@ -55,9 +55,9 @@ define(function(require) {
     else {
         // If DeviceStorage is not available, we display the file select components
         displayFileSelect();
-        setLocalArchiveFromFileSelect();
-        if (localArchive.dataFiles.length>0) {
-            // Hide the configuration elements if already set
+        if (document.getElementById('archiveFiles').files && document.getElementById('archiveFiles').files.length>0) {
+            // Archive files are already selected, 
+            setLocalArchiveFromFileSelect();
             $('#configuration').hide();
         }
     }
@@ -104,6 +104,8 @@ define(function(require) {
         localArchive.readMathFilesFromStorage(storage, archiveDirectory);
         localArchive.readMetadataFileFromStorage(storage, archiveDirectory);
         localArchive.readCoordinateFilesFromStorage(storage, archiveDirectory, 0);
+        // The archive is set : focus on the prefix field to start searching
+        document.getElementById("prefix").focus();
     }
 
     /**
@@ -120,6 +122,8 @@ define(function(require) {
     function setLocalArchiveFromFileSelect() {
         localArchive = new evopedia.LocalArchive();
         localArchive.initializeFromArchiveFiles(document.getElementById('archiveFiles').files);
+        // The archive is set : focus on the prefix field to start searching
+        document.getElementById("prefix").focus();
     }
 
     /**