From 7f826712766fee3f873b7c2a3dd41efd0784ad22 Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Mon, 23 Dec 2013 16:02:25 +0100
Subject: [PATCH 1/7] remove unused imports and fields; code formatting

---
 .../com/cradle/iitc_mobile/IITC_WebView.java  | 26 ++++++++-----------
 .../iitc_mobile/async/CheckHttpResponse.java  | 20 +++++---------
 2 files changed, 18 insertions(+), 28 deletions(-)

diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_WebView.java b/mobile/src/com/cradle/iitc_mobile/IITC_WebView.java
index f5f9ac6f..00cd0464 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_WebView.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_WebView.java
@@ -2,7 +2,6 @@ package com.cradle.iitc_mobile;
 
 import android.annotation.SuppressLint;
 import android.annotation.TargetApi;
-import android.app.Activity;
 import android.content.Context;
 import android.content.SharedPreferences;
 import android.net.ConnectivityManager;
@@ -13,9 +12,6 @@ import android.util.AttributeSet;
 import android.util.Log;
 import android.view.MotionEvent;
 import android.view.WindowManager;
-import android.webkit.ConsoleMessage;
-import android.webkit.GeolocationPermissions;
-import android.webkit.WebChromeClient;
 import android.webkit.WebSettings;
 import android.webkit.WebView;
 import android.widget.Toast;
@@ -70,17 +66,17 @@ public class IITC_WebView extends WebView {
         mNavHider = new Runnable() {
             @Override
             public void run() {
-            if (isInFullscreen() && (getFullscreenStatus() & (FS_NAVBAR)) != 0) {
-                int systemUiVisibility = SYSTEM_UI_FLAG_HIDE_NAVIGATION;
-                // in immersive mode the user can interact with the app while the navbar is hidden
-                // this mode is available since KitKat
-                // you can leave this mode by swiping down from the top of the screen. this does only work
-                // when the app is in total-fullscreen mode
-                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT && (mFullscreenStatus & FS_SYSBAR) != 0) {
-                    systemUiVisibility |= SYSTEM_UI_FLAG_IMMERSIVE;
+                if (isInFullscreen() && (getFullscreenStatus() & (FS_NAVBAR)) != 0) {
+                    int systemUiVisibility = SYSTEM_UI_FLAG_HIDE_NAVIGATION;
+                    // in immersive mode the user can interact with the app while the navbar is hidden
+                    // this mode is available since KitKat
+                    // you can leave this mode by swiping down from the top of the screen. this does only work
+                    // when the app is in total-fullscreen mode
+                    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT && (mFullscreenStatus & FS_SYSBAR) != 0) {
+                        systemUiVisibility |= SYSTEM_UI_FLAG_IMMERSIVE;
+                    }
+                    setSystemUiVisibility(systemUiVisibility);
                 }
-                setSystemUiVisibility(systemUiVisibility);
-            }
             }
         };
 
@@ -131,7 +127,7 @@ public class IITC_WebView extends WebView {
             }
 
             // disable splash screen if a http error code is responded
-            new CheckHttpResponse(mJsInterface, mIitc).execute(url);
+            new CheckHttpResponse(mIitc).execute(url);
             Log.d("iitcm", "loading url: " + url);
             super.loadUrl(url);
         }
diff --git a/mobile/src/com/cradle/iitc_mobile/async/CheckHttpResponse.java b/mobile/src/com/cradle/iitc_mobile/async/CheckHttpResponse.java
index dde6e89c..619c6c11 100644
--- a/mobile/src/com/cradle/iitc_mobile/async/CheckHttpResponse.java
+++ b/mobile/src/com/cradle/iitc_mobile/async/CheckHttpResponse.java
@@ -1,12 +1,10 @@
 package com.cradle.iitc_mobile.async;
 
 import android.app.AlertDialog;
-import android.content.Context;
 import android.content.DialogInterface;
 import android.os.AsyncTask;
 import android.util.Log;
 
-import com.cradle.iitc_mobile.IITC_JSInterface;
 import com.cradle.iitc_mobile.IITC_Mobile;
 
 import org.apache.http.HttpResponse;
@@ -24,12 +22,10 @@ import java.io.IOException;
  */
 public class CheckHttpResponse extends AsyncTask<String, Void, Boolean> {
 
-    private final IITC_JSInterface mJsInterface;
-    private final Context mContext;
+    private final IITC_Mobile mIitc;
 
-    public CheckHttpResponse(IITC_JSInterface jsInterface, Context c) {
-        mContext = c;
-        mJsInterface = jsInterface;
+    public CheckHttpResponse(IITC_Mobile iitc) {
+        mIitc = iitc;
     }
 
     @Override
@@ -43,11 +39,10 @@ public class CheckHttpResponse extends AsyncTask<String, Void, Boolean> {
             int code = response.getStatusLine().getStatusCode();
             if (code != HttpStatus.SC_OK) {
                 Log.d("iitcm", "received error code: " + code);
-                final IITC_Mobile iitc = (IITC_Mobile) mContext;
-                iitc.runOnUiThread(new Runnable() {
+                mIitc.runOnUiThread(new Runnable() {
                     @Override
                     public void run() {
-                        iitc.setLoadingState(false);
+                        mIitc.setLoadingState(false);
                     }
                 });
                 // TODO: remove when google login issue is fixed
@@ -68,7 +63,7 @@ public class CheckHttpResponse extends AsyncTask<String, Void, Boolean> {
     protected void onPostExecute(Boolean aBoolean) {
         if (aBoolean) {
             Log.d("iitcm", "google auth error, redirecting to work-around page");
-            AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(mContext);
+            AlertDialog.Builder alertDialogBuilder = new AlertDialog.Builder(mIitc);
 
             // set title
             alertDialogBuilder.setTitle("LOGIN FAILED!");
@@ -86,8 +81,7 @@ public class CheckHttpResponse extends AsyncTask<String, Void, Boolean> {
                         @Override
                         public void onClick(DialogInterface dialog, int which) {
                             dialog.cancel();
-                            IITC_Mobile iitc_mobile = (IITC_Mobile) mContext;
-                            iitc_mobile.reloadIITC();
+                            mIitc.reloadIITC();
                         }
                     });
 

From 2823450857dd8d15711d62226a6c81b169a9feac Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Mon, 23 Dec 2013 20:35:19 +0100
Subject: [PATCH 2/7] mobile/user-location: Do not reload when switching
 between loc+sensor and loc-only

---
 mobile/plugins/user-location.user.js          |  26 +++--
 mobile/res/values/strings.xml                 |  16 ++-
 mobile/res/xml/preferences.xml                |  46 ++++----
 .../com/cradle/iitc_mobile/IITC_Mobile.java   |  28 +++--
 .../cradle/iitc_mobile/IITC_UserLocation.java | 106 +++++++++---------
 .../iitc_mobile/fragments/MainSettings.java   |  18 ++-
 6 files changed, 130 insertions(+), 110 deletions(-)

diff --git a/mobile/plugins/user-location.user.js b/mobile/plugins/user-location.user.js
index 03cf5f7e..24a76147 100644
--- a/mobile/plugins/user-location.user.js
+++ b/mobile/plugins/user-location.user.js
@@ -94,13 +94,25 @@ window.plugin.userLocation.onLocationChange = function(lat, lng) {
 };
 
 window.plugin.userLocation.onOrientationChange = function(direction) {
-  $(".container", window.plugin.userLocation.marker._icon)
-    .removeClass("circle")
-    .addClass("arrow")
-    .css({
-      "transform": "rotate(" + direction + "deg)",
-      "webkitTransform": "rotate(" + direction + "deg)"
-    });
+  var container = $(".container", window.plugin.userLocation.marker._icon);
+
+  if(direction === null) {
+    container
+      .removeClass("arrow")
+      .addClass("circle")
+      .css({
+        "transform": "",
+        "webkitTransform": ""
+      });
+  } else {
+    container
+      .removeClass("circle")
+      .addClass("arrow")
+      .css({
+        "transform": "rotate(" + direction + "deg)",
+        "webkitTransform": "rotate(" + direction + "deg)"
+      });
+  }
 }
 
 var setup = window.plugin.userLocation.setup;
diff --git a/mobile/res/values/strings.xml b/mobile/res/values/strings.xml
index e6fb797d..3b5c6a26 100644
--- a/mobile/res/values/strings.xml
+++ b/mobile/res/values/strings.xml
@@ -83,11 +83,7 @@
     <string name="pref_tweaks_cat">Tweaks</string>
     <string name="pref_plugins">IITC Plugins</string>
     <string name="pref_plugins_title">Available plugins</string>
-    <string name="pref_user_loc_screen">User location</string>
-    <string name="pref_user_loc">Display user location</string>
-    <string name="pref_user_loc_sum">Show users position on map</string>
-    <string name="pref_user_loc_sensor">Use sensor orientation</string>
-    <string name="pref_user_loc_sensor_sum">Experimental! Fancier but eats battery packs for breakfast</string>
+    <string name="pref_user_location">Display user location on map</string>
     <string name="pref_user_zoom">Show zoom control</string>
     <string name="pref_user_zoom_sum">Shows +/- buttons even on multitouch capable devices.</string>
     <string name="pref_fullscreen">Hide in fullscreen mode</string>
@@ -128,6 +124,16 @@
         <item>8</item>
         <item>16</item>
     </string-array>
+    <string-array name="pref_user_location_titles">
+        <item>Don\'t show user position</item>
+        <item>Show user position on map</item>
+        <item>Show user position and orientation on map</item>
+    </string-array>
+    <string-array name="pref_user_location_values">
+        <item>0</item>
+        <item>1</item>
+        <item>2</item>
+    </string-array>
 
     <string name="menu_clear_cookies">Clear Cookies</string>
     <string name="menu_search">Search</string>
diff --git a/mobile/res/xml/preferences.xml b/mobile/res/xml/preferences.xml
index 089dcebc..7e6d0a8d 100644
--- a/mobile/res/xml/preferences.xml
+++ b/mobile/res/xml/preferences.xml
@@ -14,33 +14,26 @@
     <PreferenceCategory
         android:key="pref_ui_cat"
         android:title="@string/pref_ui_cat">
-        <PreferenceScreen
-            android:key="pref_user_loc_screen"
-            android:persistent="false"
-            android:title="@string/pref_user_loc_screen">
-            <CheckBoxPreference
-                android:defaultValue="false"
-                android:key="pref_user_loc"
-                android:summary="@string/pref_user_loc_sum"
-                android:title="@string/pref_user_loc"/>
-            <CheckBoxPreference
-                android:defaultValue="true"
-                android:key="pref_user_loc_sensor"
-                android:dependency="pref_user_loc"
-                android:summary="@string/pref_user_loc_sensor_sum"
-                android:title="@string/pref_user_loc_sensor"/>
-        </PreferenceScreen>
+        <ListPreference
+            android:defaultValue="0"
+            android:entries="@array/pref_user_location_titles"
+            android:entryValues="@array/pref_user_location_values"
+            android:key="pref_user_location_mode"
+            android:title="@string/pref_user_location"/>
+
         <CheckBoxPreference
             android:defaultValue="false"
             android:key="pref_user_zoom"
             android:summary="@string/pref_user_zoom_sum"
             android:title="@string/pref_user_zoom"/>
+
         <MultiSelectListPreference
+            android:entries="@array/pref_hide_fullscreen"
+            android:entryValues="@array/pref_hide_fullscreen_vals"
             android:key="pref_fullscreen"
             android:summary="@string/pref_fullscreen_sum"
-            android:title="@string/pref_fullscreen"
-            android:entries="@array/pref_hide_fullscreen"
-            android:entryValues="@array/pref_hide_fullscreen_vals"/>
+            android:title="@string/pref_fullscreen"/>
+
         <CheckBoxPreference
             android:defaultValue="false"
             android:key="pref_force_desktop"
@@ -50,7 +43,6 @@
     <PreferenceCategory
         android:key="pref_mics"
         android:title="@string/pref_misc_cat">
-
         <PreferenceScreen
             android:fragment="com.cradle.iitc_mobile.fragments.PluginsFragment"
             android:key="pref_plugins"
@@ -67,15 +59,15 @@
             android:summary="@string/pref_force_https_sum"
             android:title="@string/pref_force_https"/>
         <CheckBoxPreference
+            android:defaultValue="false"
             android:key="pref_external_storage"
-            android:title="@string/pref_external_storage"
             android:summary="@string/pref_external_storage_sum"
-            android:defaultValue="false" />
+            android:title="@string/pref_external_storage"/>
         <CheckBoxPreference
-                android:key="pref_press_twice_to_exit"
-                android:title="@string/pref_press_twice_to_exit"
-                android:summary="@string/pref_press_twice_to_exit_sum"
-                android:defaultValue="false" />
+            android:defaultValue="false"
+            android:key="pref_press_twice_to_exit"
+            android:summary="@string/pref_press_twice_to_exit_sum"
+            android:title="@string/pref_press_twice_to_exit"/>
     </PreferenceCategory>
     <PreferenceCategory
         android:key="pref_developer_options"
@@ -121,4 +113,4 @@
         </PreferenceScreen>
     </PreferenceCategory>
 
-</PreferenceScreen>
+</PreferenceScreen>
\ No newline at end of file
diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java b/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
index 97293ed7..c6d8d07f 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
@@ -88,8 +88,7 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
         mIitcWebView.updateFullscreenStatus();
 
         mUserLocation = new IITC_UserLocation(this);
-        mUserLocation.setLocationEnabled(mSharedPrefs.getBoolean("pref_user_loc", false));
-        mUserLocation.setSensorEnabled(mSharedPrefs.getBoolean("pref_user_loc_sensor", true));
+        mUserLocation.setLocationMode(Integer.parseInt(mSharedPrefs.getString("pref_user_location_mode", "0")));
 
         // pass ActionBar to helper because we deprecated getActionBar
         mNavigationHelper = new IITC_NavigationHelper(this, super.getActionBar());
@@ -111,10 +110,11 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
         if (key.equals("pref_force_desktop")) {
             mDesktopMode = sharedPreferences.getBoolean("pref_force_desktop", false);
             mNavigationHelper.onPrefChanged();
-        } else if (key.equals("pref_user_loc")) {
-            mUserLocation.setLocationEnabled(sharedPreferences.getBoolean("pref_user_loc", false));
-        } else if (key.equals("pref_user_loc_sensor")) {
-            mUserLocation.setSensorEnabled(sharedPreferences.getBoolean("pref_user_loc_sensor", true));
+        } else if (key.equals("pref_user_location_mode")) {
+            int mode = Integer.parseInt(mSharedPrefs.getString("pref_user_location_mode", "0"));
+            if (mUserLocation.setLocationMode(mode))
+                mReloadNeeded = true;
+            return;
         } else if (key.equals("pref_fullscreen")) {
             mIitcWebView.updateFullscreenStatus();
             mNavigationHelper.onPrefChanged();
@@ -247,43 +247,41 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
     protected void onStart() {
         super.onStart();
 
-        // enough idle...let's do some work
-        Log.d("iitcm", "resuming...reset idleTimer");
-        mUserLocation.onStart();
-
         if (mReloadNeeded) {
             Log.d("iitcm", "preference had changed...reload needed");
             reloadIITC();
         } else {
             // iitc is not fully booted...timer will be reset by the script itself
             if (findViewById(R.id.imageLoading).getVisibility() == View.GONE) {
+                // enough idle...let's do some work
+                Log.d("iitcm", "resuming...reset idleTimer");
                 mIitcWebView.loadUrl("javascript: window.idleReset();");
             }
         }
+
+        mUserLocation.onStart();
     }
 
     @Override
     protected void onResume() {
+        super.onResume();
         mIitcWebView.resumeTimers();
         mIitcWebView.onResume();
-        super.onResume();
     }
 
     @Override
     protected void onPause() {
+        super.onPause();
         mIitcWebView.pauseTimers();
         mIitcWebView.onPause();
-        super.onPause();
     }
 
     @Override
     protected void onStop() {
+        super.onStop();
         Log.d("iitcm", "stopping iitcm");
         mIitcWebView.loadUrl("javascript: window.idleSet();");
-
         mUserLocation.onStop();
-
-        super.onStop();
     }
 
     @Override
diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
index de6834a8..dfa81c67 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
@@ -12,15 +12,15 @@ import android.os.Bundle;
 import android.view.Surface;
 
 public class IITC_UserLocation implements LocationListener, SensorEventListener {
-    private static final double SENSOR_DELAY_USER = 100*1e6;
-    private boolean mLocationEnabled = false;
-    private boolean mSensorEnabled = true;
+    private static final double SENSOR_DELAY_USER = 100 * 1e6; // 100 milliseconds
+    private int mMode = 0;
+    private boolean mRunning = false;
+    private boolean mLocationRegistered = false;
+    private boolean mOrientationRegistered = false;
     private long mLastUpdate = 0;
     private IITC_Mobile mIitc;
     private Location mLastLocation = null;
     private LocationManager mLocationManager;
-    private boolean mRegistered = false;
-    private boolean mRunning = false;
     private Sensor mSensorAccelerometer, mSensorMagnetometer;
     private SensorManager mSensorManager = null;
     float[] mValuesGravity = null, mValuesGeomagnetic = null;
@@ -35,51 +35,44 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         mSensorMagnetometer = mSensorManager.getDefaultSensor(Sensor.TYPE_MAGNETIC_FIELD);
     }
 
-    private void registerListeners() {
-        if (mRegistered) return;
-        mRegistered = true;
+    private void updateListeners() {
+        boolean useLocation = mRunning && mMode != 0;
+        boolean useOrientation = mRunning && mMode == 2;
 
-        try {
-            mLocationManager.requestLocationUpdates(LocationManager.NETWORK_PROVIDER, 0, 0, this);
-        } catch (IllegalArgumentException e) {
-            // if the given provider doesn't exist
-            e.printStackTrace();
+        if (useLocation && !mLocationRegistered) {
+            try {
+                mLocationManager.requestLocationUpdates(LocationManager.NETWORK_PROVIDER, 0, 0, this);
+            } catch (IllegalArgumentException e) {
+                // if the given provider doesn't exist
+                e.printStackTrace();
+            }
+            try {
+                mLocationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 0, 0, this);
+            } catch (IllegalArgumentException e) {
+                // if the given provider doesn't exist
+                e.printStackTrace();
+            }
+            mLocationRegistered = true;
         }
-        try {
-            mLocationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 0, 0, this);
-        } catch (IllegalArgumentException e) {
-            // if the given provider doesn't exist
-            e.printStackTrace();
+        if (!useLocation && mLocationRegistered) {
+            mLocationManager.removeUpdates(this);
+            mLocationRegistered = false;
         }
 
-        if (mSensorAccelerometer != null && mSensorMagnetometer != null && mSensorEnabled) {
+        if (useOrientation && !mOrientationRegistered && mSensorAccelerometer != null && mSensorMagnetometer != null) {
             mSensorManager.registerListener(this, mSensorAccelerometer, SensorManager.SENSOR_DELAY_NORMAL);
             mSensorManager.registerListener(this, mSensorMagnetometer, SensorManager.SENSOR_DELAY_NORMAL);
+            mOrientationRegistered = true;
         }
-    }
-
-    private void unregisterListeners() {
-        if (!mRegistered) return;
-        mRegistered = false;
-
-        mLocationManager.removeUpdates(this);
-
-        if (mSensorAccelerometer != null && mSensorMagnetometer != null && mSensorEnabled) {
+        if (!useOrientation && mOrientationRegistered && mSensorAccelerometer != null && mSensorMagnetometer != null) {
             mSensorManager.unregisterListener(this, mSensorAccelerometer);
             mSensorManager.unregisterListener(this, mSensorMagnetometer);
+            mOrientationRegistered = false;
         }
-
-    }
-
-    private void updateListeners() {
-        if (mRunning && mLocationEnabled)
-            registerListeners();
-        else
-            unregisterListeners();
     }
 
     public boolean hasCurrentLocation() {
-        if (!mRegistered) return false;
+        if (!mLocationRegistered) return false;
         return mLastLocation != null;
     }
 
@@ -98,6 +91,12 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
     public void onStart() {
         mRunning = true;
         updateListeners();
+
+        // in case we just switched from loc+sensor to loc-only, let javascript know
+        if (mMode == 1) {
+            mIitc.getWebView().loadJS("if(window.plugin && window.plugin.userLocation)"
+                    + "window.plugin.userLocation.onOrientationChange(null);");
+        }
     }
 
     public void onStop() {
@@ -105,18 +104,19 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         updateListeners();
     }
 
-    public void setLocationEnabled(boolean enabled) {
-        if (enabled == mLocationEnabled) return;
+    /**
+     * set the location mode to use. Available modes:
+     * 0: don't show user's position
+     * 1: show user's position
+     * 2: show user's position and orientation
+     * 
+     * @return whether a reload is needed to reflect the changes made to the preferences
+     */
+    public boolean setLocationMode(int mode) {
+        boolean needsReload = (mode == 0 && mMode != 0) || (mode != 0 && mMode == 0);
+        mMode = mode;
 
-        mLocationEnabled = enabled;
-        updateListeners();
-    }
-
-    public void setSensorEnabled(boolean enabled) {
-        if (enabled == mSensorEnabled) return;
-
-        mSensorEnabled = enabled;
-        updateListeners();
+        return needsReload;
     }
 
     // ------------------------------------------------------------------------
@@ -140,12 +140,10 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
 
     @Override
     public void onProviderDisabled(String provider) {
-
     }
 
     @Override
     public void onProviderEnabled(String provider) {
-
     }
 
     @Override
@@ -165,15 +163,15 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
 
     @Override
     public void onSensorChanged(SensorEvent event) {
-        // save some battery 10 updates per second should be enough
-        if ((event.timestamp - mLastUpdate) < SENSOR_DELAY_USER) return;
-        mLastUpdate = event.timestamp;
-
         if (event.sensor.getType() == Sensor.TYPE_ACCELEROMETER)
             mValuesGravity = event.values;
         if (event.sensor.getType() == Sensor.TYPE_MAGNETIC_FIELD)
             mValuesGeomagnetic = event.values;
 
+        // save some battery, 10 updates per second should be enough
+        if ((event.timestamp - mLastUpdate) < SENSOR_DELAY_USER) return;
+        mLastUpdate = event.timestamp;
+
         // do not touch the javascript while iitc boots
         if (mIitc.isLoading()) return;
 
diff --git a/mobile/src/com/cradle/iitc_mobile/fragments/MainSettings.java b/mobile/src/com/cradle/iitc_mobile/fragments/MainSettings.java
index 77ccac2b..461e034b 100644
--- a/mobile/src/com/cradle/iitc_mobile/fragments/MainSettings.java
+++ b/mobile/src/com/cradle/iitc_mobile/fragments/MainSettings.java
@@ -6,6 +6,7 @@ import android.content.pm.PackageManager;
 import android.content.pm.PackageManager.NameNotFoundException;
 import android.os.Bundle;
 import android.preference.EditTextPreference;
+import android.preference.ListPreference;
 import android.preference.Preference;
 import android.preference.Preference.OnPreferenceChangeListener;
 import android.preference.PreferenceFragment;
@@ -54,6 +55,20 @@ public class MainSettings extends PreferenceFragment {
         // first init of summary
         String pref_iitc_source_sum = getString(R.string.pref_select_iitc_sum) + " " + pref_iitc_source.getText();
         pref_iitc_source.setSummary(pref_iitc_source_sum);
+
+        final ListPreference pref_user_location_mode = (ListPreference) findPreference("pref_user_location_mode");
+        pref_user_location_mode.setOnPreferenceChangeListener(new OnPreferenceChangeListener() {
+            @Override
+            public boolean onPreferenceChange(Preference preference, Object newValue) {
+                int mode = Integer.parseInt((String) newValue);
+                preference.setSummary(getResources().getStringArray(R.array.pref_user_location_titles)[mode]);
+                return true;
+            }
+        });
+
+        String value = getPreferenceManager().getSharedPreferences().getString("pref_user_location_mode", "0");
+        int mode = Integer.parseInt(value);
+        pref_user_location_mode.setSummary(getResources().getStringArray(R.array.pref_user_location_titles)[mode]);
     }
 
     // we want a home button + HomeAsUpEnabled in nested preferences
@@ -63,8 +78,7 @@ public class MainSettings extends PreferenceFragment {
     @Override
     public boolean onPreferenceTreeClick(PreferenceScreen preferenceScreen, Preference preference) {
         if (preference.getTitle().toString().equals(getString(R.string.pref_advanced_options))
-                || preference.getTitle().toString().equals(getString(R.string.pref_about_title))
-                || preference.getTitle().toString().equals(getString(R.string.pref_user_loc_screen))) {
+                || preference.getTitle().toString().equals(getString(R.string.pref_about_title))) {
             initializeActionBar((PreferenceScreen) preference);
         }
         return super.onPreferenceTreeClick(preferenceScreen, preference);

From 51064a29c43edd1e0179a1bb3704f7c8da3d6bae Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Mon, 23 Dec 2013 21:16:34 +0100
Subject: [PATCH 3/7] Use CSS transition for arrow rotation

---
 mobile/plugins/user-location.css              |  5 +++-
 .../cradle/iitc_mobile/IITC_UserLocation.java | 27 +++++++++++++++----
 2 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/mobile/plugins/user-location.css b/mobile/plugins/user-location.css
index 52d79f88..8890bdd8 100644
--- a/mobile/plugins/user-location.css
+++ b/mobile/plugins/user-location.css
@@ -5,8 +5,11 @@
 .user-location .container {
   height: 32px;
   width: 32px;
-  transform-origin: center;
+          transform-origin: center;
   -webkit-transform-origin: center;
+          transition: all 200ms linear;
+     -moz-transition: all 200ms linear;
+  -webkit-transition: all 200ms linear;
 }
 
 .user-location .container .inner,
diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
index dfa81c67..516f1589 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
@@ -23,7 +23,8 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
     private LocationManager mLocationManager;
     private Sensor mSensorAccelerometer, mSensorMagnetometer;
     private SensorManager mSensorManager = null;
-    float[] mValuesGravity = null, mValuesGeomagnetic = null;
+    private float[] mValuesGravity = null, mValuesGeomagnetic = null;
+    private double mOrientation = 0;
 
     public IITC_UserLocation(IITC_Mobile iitc) {
         mIitc = iitc;
@@ -35,6 +36,24 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         mSensorMagnetometer = mSensorManager.getDefaultSensor(Sensor.TYPE_MAGNETIC_FIELD);
     }
 
+    private void setOrientation(Double orientation) {
+        // we have a transition defined for the rotation
+        // changes to the orientation should always be less than 180°
+
+        if (orientation != null) {
+            while (orientation < mOrientation - 180)
+                orientation += 360;
+            while (orientation > mOrientation + 180)
+                orientation -= 360;
+            mOrientation = orientation;
+        } else {
+            mOrientation = 0;
+        }
+
+        mIitc.getWebView().loadJS("if(window.plugin && window.plugin.userLocation)"
+                + "window.plugin.userLocation.onOrientationChange(" + String.valueOf(orientation) + ");");
+    }
+
     private void updateListeners() {
         boolean useLocation = mRunning && mMode != 0;
         boolean useOrientation = mRunning && mMode == 2;
@@ -94,8 +113,7 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
 
         // in case we just switched from loc+sensor to loc-only, let javascript know
         if (mMode == 1) {
-            mIitc.getWebView().loadJS("if(window.plugin && window.plugin.userLocation)"
-                    + "window.plugin.userLocation.onOrientationChange(null);");
+            setOrientation(null);
         }
     }
 
@@ -200,8 +218,7 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
                 break;
         }
 
-        mIitc.getWebView().loadJS("if(window.plugin && window.plugin.userLocation)"
-                + "window.plugin.userLocation.onOrientationChange(" + direction + ");");
+        setOrientation(direction);
     }
 
     // </interface SensorEventListener>

From 78f6e07457624f7b5cde6557ef681945072d3d8f Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Mon, 23 Dec 2013 21:53:25 +0100
Subject: [PATCH 4/7] Mobile: implement follow mode

---
 mobile/plugins/user-location.user.js          |  28 ++++++++++++
 .../ic_action_location_follow.png             | Bin 0 -> 646 bytes
 .../ic_action_location_follow.png             | Bin 0 -> 451 bytes
 .../ic_action_location_follow.png             | Bin 0 -> 820 bytes
 .../ic_action_location_follow.png             | Bin 0 -> 1236 bytes
 .../cradle/iitc_mobile/IITC_JSInterface.java  |  10 +++++
 .../com/cradle/iitc_mobile/IITC_Mobile.java   |  41 +++++++++++++-----
 .../cradle/iitc_mobile/IITC_UserLocation.java |  14 ++++++
 8 files changed, 82 insertions(+), 11 deletions(-)
 create mode 100644 mobile/res/drawable-hdpi/ic_action_location_follow.png
 create mode 100644 mobile/res/drawable-mdpi/ic_action_location_follow.png
 create mode 100644 mobile/res/drawable-xhdpi/ic_action_location_follow.png
 create mode 100644 mobile/res/drawable-xxhdpi/ic_action_location_follow.png

diff --git a/mobile/plugins/user-location.user.js b/mobile/plugins/user-location.user.js
index 24a76147..c11ddf94 100644
--- a/mobile/plugins/user-location.user.js
+++ b/mobile/plugins/user-location.user.js
@@ -21,6 +21,7 @@
 window.plugin.userLocation = function() {};
 
 window.plugin.userLocation.locationLayer = new L.LayerGroup();
+window.plugin.userLocation.follow = false;
 
 window.plugin.userLocation.setup = function() {
   $('<style>').prop('type', 'text/css').html('@@INCLUDESTRING:mobile/plugins/user-location.css@@').appendTo('head');
@@ -59,10 +60,20 @@ window.plugin.userLocation.setup = function() {
   window.plugin.userLocation.circle = circle;
   window.plugin.userLocation.icon = icon;
 
+  window.map.on('movestart', window.plugin.userLocation.onMoveStart);
   window.map.on('zoomend', window.plugin.userLocation.onZoomEnd);
   window.plugin.userLocation.onZoomEnd();
 };
 
+window.plugin.userLocation.onMoveStart = function(e) {
+  if(window.plugin.userLocation.moving)
+    return;
+
+  window.plugin.userLocation.follow = false;
+  if(typeof android !== 'undefined' && android && android.setFollowMode)
+    android.setFollowMode(window.plugin.userLocation.follow);
+};
+
 window.plugin.userLocation.onZoomEnd = function() {
   if(window.map.getZoom() < 16)
     window.plugin.userLocation.locationLayer.removeLayer(window.plugin.userLocation.circle);
@@ -71,6 +82,13 @@ window.plugin.userLocation.onZoomEnd = function() {
 };
 
 window.plugin.userLocation.locate = function(lat, lng, accuracy) {
+  if(window.plugin.userLocation.follow) {
+    window.plugin.userLocation.follow = false;
+    if(typeof android !== 'undefined' && android && android.setFollowMode)
+      android.setFollowMode(window.plugin.userLocation.follow);
+    return;
+  }
+
   var latlng = new L.LatLng(lat, lng);
 
   var latAccuracy = 180 * accuracy / 40075017;
@@ -85,12 +103,22 @@ window.plugin.userLocation.locate = function(lat, lng, accuracy) {
   zoom = Math.min(zoom,17);
 
   window.map.setView(latlng, zoom);
+
+  window.plugin.userLocation.follow = true;
+  if(typeof android !== 'undefined' && android && android.setFollowMode)
+    android.setFollowMode(window.plugin.userLocation.follow);
 }
 
 window.plugin.userLocation.onLocationChange = function(lat, lng) {
   var latlng = new L.LatLng(lat, lng);
   window.plugin.userLocation.marker.setLatLng(latlng);
   window.plugin.userLocation.circle.setLatLng(latlng);
+
+  if(window.plugin.userLocation.follow) {
+    window.plugin.userLocation.moving = true;
+    window.map.setView(latlng);
+    window.plugin.userLocation.moving = false;
+  }
 };
 
 window.plugin.userLocation.onOrientationChange = function(direction) {
diff --git a/mobile/res/drawable-hdpi/ic_action_location_follow.png b/mobile/res/drawable-hdpi/ic_action_location_follow.png
new file mode 100644
index 0000000000000000000000000000000000000000..aa706a81e844bd6d2b8e0ad0214fe235e325be11
GIT binary patch
literal 646
zcmV;10(t$3P)<h;3K|Lk000e1NJLTq001xm001xu1^@s6R|5Hm0006_Nkl<ZXx{Bu
zv30^Q5dGoNGC-o91++kVIqA&+VSuCu`Y-?tVER0}XMo%S?hONYfi&m$?kTc^92p4F
zufR__>7;*m{{%!tL_}N?cS9N0MKuNR3IJr(e5iHC10K!<fFAk=Mt_|`qZk9<x&Oe_
z(Et+uyajLsKxEY6>KxETHQ~+90NnpD{0JZekdsmM#U0Q^^$1|WLEE4PP+CK$jxj4T
zYJV{YbWtq@O#mPVkdaYkM{1qfbI7RW*&WbDmGP+p0P$%aJmO>_(riga&Cce4b@IJW
zFoc~^PtGSHuAk@T{5N6S!`%R9n-2~~a!CYTR2o1Ez~0Orz>bU}$D9q*sRr=ulbw8T
zAQ72l<=Dt8fQ@7G;BOlqV`C-cM=~UZW`a5g$L7-$Y^1gmY%90c2<rf6j;2UuEwnim
zDbJB;Y~J%)!Ec~huy*A6v_}l5)-9AA1MpF?K61J_{j!DMzHrSc8y%&KN>a3LD~{t1
z%52V|=R1;x3}yl_zezdWMWMANzMgzj+%K>tqqe_tz~&rhhD7KeyGZ%QflGx>h2##N
zg_;UFVO1%y>v76Ehf<wbYlU-wi_OthY*-%nIN_2Mjm@J^*XA3<nGjwNmJ{m}o!IeM
z7d^if$<PkuqS!1Px3uPTw{PM6+}^xps}_=Cn0|mI?|TS2?dS>Kd|OQKI=~A^4ZjSu
z`Mghmd@U^{^?iQ)zfJv3JgeNq!&VEwH9PVms71Ker^OCuKKbX|)y#)Ld|toiejo%$
gWSoeIh=?1+7tRa+-uUY^5dZ)H07*qoM6N<$f<XBxQ2+n{

literal 0
HcmV?d00001

diff --git a/mobile/res/drawable-mdpi/ic_action_location_follow.png b/mobile/res/drawable-mdpi/ic_action_location_follow.png
new file mode 100644
index 0000000000000000000000000000000000000000..64e9f660d04fe0f254559aa38b0c6a72886c8175
GIT binary patch
literal 451
zcmV;!0X+VRP)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F80004sNkl<ZSi|j?
zF>b;@5Jg|2v_3#^&jt1X(3_+uIY9c59=Q+n901XixHo-#0rw^ca1M~>ue376#$F;4
z$x|84-<jFnKYteJtv3%dNij1?G4qLW<5|PdcN5o_KE<n+M;-tZU<{awI|EYSJ&VHf
z>;;&l7z1-407_s7?ByN+BgYqKQJ9{s0FxAVN?roXEDGOhaL+0v{t4Hr0PXx0$bpwE
z3il28f$&4&)6NCx&bRvcXp<|+)aim8*i`&$`R?mD(bNSK9p};sey5_QBBIgvRz8GX
zYVE|Q4aY!f?@~LP*Wl;Epf#mpbS}V%?bfCzaMs1_j<jpN5=vTKh1<7OzJs+UDOQ#%
zSrjh*loB4CfLozSiVukl4%<yb%-s35e0b{0flLZ#ZPS1+E0~dDbS7a61eNxRmFQUV
zYP$xCaWNtK0v_sw=;Bh`*aZp@E8yB*N6j!=3wzQ;Y&(u;!0nN7JXCiCz$dUc>p1?+
t2Z`^abmger|9lhrzKZky&q;5-_6@2Ov`p`IX-NP8002ovPDHLkV1le<&N%=8

literal 0
HcmV?d00001

diff --git a/mobile/res/drawable-xhdpi/ic_action_location_follow.png b/mobile/res/drawable-xhdpi/ic_action_location_follow.png
new file mode 100644
index 0000000000000000000000000000000000000000..c058a2b4f350ab6740b474bd4355924f388ebf15
GIT binary patch
literal 820
zcmV-41Izr0P)<h;3K|Lk000e1NJLTq002M$002M;1^@s6s%dfF0008}Nkl<Zc-rk+
z%aNln5Pi6OB><K?2`B;Xl=nUf-~h%6?0qHxlfc}U;!Xmc0C$D}oWLBmU8U6^5J<pK
z=vBFD?7{jx_2+fVfQN^Nhlhv9Q4wr4v8Y-BK=50lET%2iaoUi40U+m>*lH1{qq~47
zz!TuJz-m!H0g!UQO8|t*BM$%#00K}`7B{;mKvZoA;03@1Ks52|9e@(R2W9bNcLa#4
zec==rY94u>I5ZX^KUYu|f9!$)KJ~Yr+};7G*}Q2$9HsnHErtNxTw6o1fS=!$)9m;J
z%ZX!M*n%R(8m=zmL$L7X3(Dfca@?m{T!e(cq1peHHveFAB}d~6b}M=O-!9z2l@3rB
zcXe6Z@K{$K8$8$BkFWra_OH@Mqnj61`-^>BNCB>+EdDgGguC?7yKQy|9Gm{2&x@)}
z*`E{5BMpyH8^#1{R|p&isE)GuGBGu3t$rxwiK;E7>|hmWZDWAE;bLI(_lY&X0C4B;
z_My<MVLbvk?M2e*mjeMZnK4D83v@hJ$8RHE%jy=8N`Kb_K5x_Q5j;j8YRy_!BS4H1
zpa__`$PEY(TM&>adAcvvB@PHkc&t8c@NF$qaW+(8eGG@D1UWziU`Qi`oRPlqG^R<;
zhNhm6q^zY05Sn>NI-FdSEocoXyjz%90BLf3J`#XjwzS9~5Em1^XHEe75NT?Fp*Y}@
zfNtdfkGX2l#swNBP^!dzIwGOzE0UBsXU}tTWR|IcWSxt`K#0*61dJ*%uIg<10`)*;
zII2V!C4gH}vL4qJCAXO-+5q^Y(IywA4%x&7fPa1aLCMM`?RDZQ)!bNl7|7$&W&^$<
zVlB~*+!B)(CO15s_wJ?sgWPr}%Hpf-xUQt|O$vx1cXNcp%?oAo9nWbEe=)LvnZD!y
z>nxX7V;z9kA9*#VWQJp2j`n1Se57Eak&5zZ^82|S@3Dqs{y;(#A5t9YCo&VFyLGnM
yS_kYFo&Zn4!ExSjZn@h`@bK{P@bK{PSQ%fTRcI!uV6=n)0000<MNUMnLSTXn6nGy1

literal 0
HcmV?d00001

diff --git a/mobile/res/drawable-xxhdpi/ic_action_location_follow.png b/mobile/res/drawable-xxhdpi/ic_action_location_follow.png
new file mode 100644
index 0000000000000000000000000000000000000000..423a873f5a025ec0c8404c9f80cfb79c2201845d
GIT binary patch
literal 1236
zcmV;_1S|WAP)<h;3K|Lk000e1NJLTq003YB003YJ1^@s6;+S_h000D=Nkl<Zc-rk;
zF;eR=5PdMTIY4j^2QWS9P3V&YBt6h4>BH~I0g|2%ebRp&0QV#Z@CAOsGCyC)k?dHq
z4e!k`VUn@5yU*HPNh<*aK@bE%5ClOG1VIqQPr*qWg^pj}I{+^Bzf(!1+NjG}OJAG|
z08iR?S8Y_}lum#IkN|=NkN|=NkN|=cvAl^_3?P>Lza#&fpa5h5QUIzAle3cs8@&(U
z8bI&Jrq%#9Dv5rR0JkNEf&6tUZaxFB0<cm^l#zh1#vgeyypaf%L<<t|VfYIGQ)r88
z2JFPgZ{^e29DsBA>{{AO0ZdgAr9YN{`1O4N;ND{NrTFk|!M4vOh72uaJgOvG{Wt>R
z*Y_!an*-U*qN*%LvZ&F4?CC`%(ey_UAc1zCV~h;IOeN9UQ{O!RlbkrC06c^a&~;42
z@{GR*aO=zX03dDON?)e<G5^*h;DFl+?uQS=pP=sOG_b-Z<Im~{g3{;A@Y9XR`h7-#
zq?*ed<JUSvl^OTM@Y66viW+JJOmo~$?aKoq!=t0j?Hd6y!yV}XS0&LyiR`H-x&5Uf
zx8DJH1F(_yvx4o<@pav4;d{n>;@S8e0MA<5mr1sh6=<XA`cUC@pDJW+q6;z^ZC1c2
zJu%7%#ny;A^31ce;o^!i0KW|Fcn!!Q2>4G=*F0Z>;~d8=&UFCHr7hQZpe#DHg8)%l
zH{PPavy%$NnYfZ*kg-$X*+z|kYduK1tnKVA7gUXji#rk-cNSe*L4dC8(lHQDJjyhY
zK5ST8bZIsLvihiFQ!;2*3D=;yanz}?BOo@opRHrM=^F#vC=gk97M&VRfWEusvaa8Q
z|Lw@nQQ+cIltrgT6QCQ?WjE^@;o#EO)}m7-@O3^QE-?Zw0}^mqVg$qvaUd-^Yc5QH
z2=U~>DBXYrl<mu=_t^1e3pNubNB{{SDh|S___||!r3t%$abQ>GA(cc|wG$vwUReUQ
z(lVjgAvnDSB;d_cCf67jq^Dx%Wv2fg;nLT+MW;p+kd~OnuYD<LFyy+#2uRyRfIeKi
zQmbimZ6V52tlN|#Yqzy7C}-W@W%|GJJPKS%zIN29(P^kY%!iHve&$go*V@maMVD3(
zurauy*nxz#ClY1_cI3LD5(IAq$OgCmwv7rrJFD3Eu0>$_o<!!efXw$ySogPa!3u?k
zBV$4aVC-r@kuA39&^iKi1Kv6A%kGzxFA5To$uqaE!Bb?qD;cy)gQbc8+W&r~iF?W|
z@pVhD*0Wife5SJksY;@u@4oA-j=uL<@KnI~eV7kwOhfb^gaE0tH<f@->LlSn1>R$*
z<Ez2;1H|3D<d86^!T5t5<G+M-jdh%od_w*qy_xS*ihI>}J=1%ll4u;#9(C-9J;1!M
z)x$?QdEcY)`}in#HyU4L3yLNm)LvC}WL1_I2H-($r}R(H?Riy!XA-KqtmE@GU0&94
z#EhBO)znM^FKlWM8N9NrL1gHc_FW+V#uHxJ7l@QU+E?Q?USZGDL=r#(2ogX72ogX7
yh#GL#V%hbLlazE41VIo4K@bE%5ClOGO~zm3W8~qDH(jg%0000<MNUMnLSTYlV@l!x

literal 0
HcmV?d00001

diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_JSInterface.java b/mobile/src/com/cradle/iitc_mobile/IITC_JSInterface.java
index 93372fd1..d51f0e59 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_JSInterface.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_JSInterface.java
@@ -189,4 +189,14 @@ public class IITC_JSInterface {
             }
         });
     }
+
+    @JavascriptInterface
+    public void setFollowMode(final boolean follow) {
+        mIitc.runOnUiThread(new Runnable() {
+            @Override
+            public void run() {
+                mIitc.getUserLocation().setFollowMode(follow);
+            }
+        });
+    }
 }
diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java b/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
index c6d8d07f..711be309 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_Mobile.java
@@ -400,18 +400,32 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
 
     @Override
     public boolean onPrepareOptionsMenu(Menu menu) {
-        if (mNavigationHelper != null) {
-            boolean visible = !mNavigationHelper.isDrawerOpened();
+        boolean visible = false;
+        if (mNavigationHelper != null)
+            visible = !mNavigationHelper.isDrawerOpened();
 
-            for (int i = 0; i < menu.size(); i++)
-                if (menu.getItem(i).getItemId() != R.id.action_settings) {
-                    // clear cookies is part of the advanced menu
-                    if (menu.getItem(i).getItemId() == R.id.menu_clear_cookies) {
-                        menu.getItem(i).setVisible(mAdvancedMenu & visible);
-                    } else {
-                        menu.getItem(i).setVisible(visible);
-                    }
-                }
+        for (int i = 0; i < menu.size(); i++) {
+            MenuItem item = menu.getItem(i);
+
+            switch (item.getItemId()) {
+                case R.id.action_settings:
+                    item.setVisible(true);
+                    break;
+
+                case R.id.menu_clear_cookies:
+                    item.setVisible(mAdvancedMenu && visible);
+                    break;
+
+                case R.id.locate:
+                    item.setVisible(visible);
+                    item.setIcon(mUserLocation.isFollowing()
+                            ? R.drawable.ic_action_location_follow
+                            : R.drawable.ic_action_location_found);
+                    break;
+
+                default:
+                    item.setVisible(visible);
+            }
         }
 
         return super.onPrepareOptionsMenu(menu);
@@ -473,6 +487,7 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
     public void reloadIITC() {
         mNavigationHelper.reset();
         mMapSettings.reset();
+        mUserLocation.reset();
         mBackStack.clear();
         // iitc starts on map after reload
         mCurrentPane = Pane.MAP;
@@ -633,4 +648,8 @@ public class IITC_Mobile extends Activity implements OnSharedPreferenceChangeLis
     public IITC_MapSettings getMapSettings() {
         return mMapSettings;
     }
+
+    public IITC_UserLocation getUserLocation() {
+        return mUserLocation;
+    }
 }
diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
index 516f1589..551ee404 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
@@ -25,6 +25,7 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
     private SensorManager mSensorManager = null;
     private float[] mValuesGravity = null, mValuesGeomagnetic = null;
     private double mOrientation = 0;
+    private boolean mFollowing = false;
 
     public IITC_UserLocation(IITC_Mobile iitc) {
         mIitc = iitc;
@@ -95,6 +96,10 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         return mLastLocation != null;
     }
 
+    public boolean isFollowing() {
+        return mFollowing;
+    }
+
     public void locate() {
         // do not touch the javascript while iitc boots
         if (mIitc.isLoading()) return;
@@ -122,6 +127,15 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         updateListeners();
     }
 
+    public void reset() {
+        setFollowMode(false);
+    }
+
+    public void setFollowMode(boolean follow) {
+        mFollowing = follow;
+        mIitc.invalidateOptionsMenu();
+    }
+
     /**
      * set the location mode to use. Available modes:
      * 0: don't show user's position

From 4b415dba597443f3a1341d7e1c1a3dfd5e175fbb Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Mon, 23 Dec 2013 23:15:20 +0100
Subject: [PATCH 5/7] Only move map when marker reaches edge

---
 mobile/plugins/user-location.user.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/mobile/plugins/user-location.user.js b/mobile/plugins/user-location.user.js
index c11ddf94..43b14d93 100644
--- a/mobile/plugins/user-location.user.js
+++ b/mobile/plugins/user-location.user.js
@@ -115,6 +115,11 @@ window.plugin.userLocation.onLocationChange = function(lat, lng) {
   window.plugin.userLocation.circle.setLatLng(latlng);
 
   if(window.plugin.userLocation.follow) {
+    // move map if marker moves more than 35% from the center
+    // 100% - 2*15% = 70% → 35% from center in either direction
+    if(map.getBounds().pad(-0.15).contains(latlng))
+      return;
+
     window.plugin.userLocation.moving = true;
     window.map.setView(latlng);
     window.plugin.userLocation.moving = false;

From 7ff4c79910dd05a050c4ad83aa39b884d81f8d99 Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Tue, 24 Dec 2013 11:48:01 +0100
Subject: [PATCH 6/7] Don't start follow mode immediately

---
 mobile/plugins/user-location.user.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/mobile/plugins/user-location.user.js b/mobile/plugins/user-location.user.js
index 43b14d93..d4523322 100644
--- a/mobile/plugins/user-location.user.js
+++ b/mobile/plugins/user-location.user.js
@@ -102,11 +102,13 @@ window.plugin.userLocation.locate = function(lat, lng, accuracy) {
   // so limit to 17 (enough to see all portals)
   zoom = Math.min(zoom,17);
 
-  window.map.setView(latlng, zoom);
+  if(window.map.getCenter().distanceTo(latlng) < 10) {
+    window.plugin.userLocation.follow = true;
+    if(typeof android !== 'undefined' && android && android.setFollowMode)
+      android.setFollowMode(window.plugin.userLocation.follow);
+  }
 
-  window.plugin.userLocation.follow = true;
-  if(typeof android !== 'undefined' && android && android.setFollowMode)
-    android.setFollowMode(window.plugin.userLocation.follow);
+  window.map.setView(latlng, zoom);
 }
 
 window.plugin.userLocation.onLocationChange = function(lat, lng) {

From 3fe9b91a92098a998c74752fc5ef99ae2cb18e87 Mon Sep 17 00:00:00 2001
From: Philipp Schaefer <trashzopf@gmail.com>
Date: Fri, 27 Dec 2013 17:09:33 +0100
Subject: [PATCH 7/7] use current best location estimation from the android
 docs:
 http://developer.android.com/guide/topics/location/strategies.html#BestEstimate

---
 .../cradle/iitc_mobile/IITC_UserLocation.java | 64 ++++++++++++++++++-
 1 file changed, 61 insertions(+), 3 deletions(-)

diff --git a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
index 551ee404..da47e1e6 100644
--- a/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
+++ b/mobile/src/com/cradle/iitc_mobile/IITC_UserLocation.java
@@ -136,6 +136,65 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
         mIitc.invalidateOptionsMenu();
     }
 
+    private static final int TWO_MINUTES = 1000 * 60 * 2;
+
+    /**
+      * Determines whether one Location reading is better than the current Location fix
+      * @param location  The new Location that you want to evaluate
+      * @param currentBestLocation  The current Location fix, to which you want to compare the new one
+      *
+      * code copied from http://developer.android.com/guide/topics/location/strategies.html#BestEstimate
+      */
+    protected boolean isBetterLocation(Location location, Location currentBestLocation) {
+        if (currentBestLocation == null) {
+            // A new location is always better than no location
+            return true;
+        }
+
+        // Check whether the new location fix is newer or older
+        long timeDelta = location.getTime() - currentBestLocation.getTime();
+        boolean isSignificantlyNewer = timeDelta > TWO_MINUTES;
+        boolean isSignificantlyOlder = timeDelta < -TWO_MINUTES;
+        boolean isNewer = timeDelta > 0;
+
+        // If it's been more than two minutes since the current location, use the new location
+        // because the user has likely moved
+        if (isSignificantlyNewer) {
+            return true;
+        // If the new location is more than two minutes older, it must be worse
+        } else if (isSignificantlyOlder) {
+            return false;
+        }
+
+        // Check whether the new location fix is more or less accurate
+        int accuracyDelta = (int) (location.getAccuracy() - currentBestLocation.getAccuracy());
+        boolean isLessAccurate = accuracyDelta > 0;
+        boolean isMoreAccurate = accuracyDelta < 0;
+        boolean isSignificantlyLessAccurate = accuracyDelta > 100;
+
+        // Check if the old and new location are from the same provider
+        boolean isFromSameProvider = isSameProvider(location.getProvider(),
+                currentBestLocation.getProvider());
+
+        // Determine location quality using a combination of timeliness and accuracy
+        if (isMoreAccurate) {
+            return true;
+        } else if (isNewer && !isLessAccurate) {
+            return true;
+        } else if (isNewer && !isSignificantlyLessAccurate && isFromSameProvider) {
+            return true;
+        }
+        return false;
+    }
+
+    // Checks whether two providers are the same
+    private boolean isSameProvider(String provider1, String provider2) {
+        if (provider1 == null) {
+          return provider2 == null;
+        }
+        return provider1.equals(provider2);
+    }
+
     /**
      * set the location mode to use. Available modes:
      * 0: don't show user's position
@@ -157,10 +216,9 @@ public class IITC_UserLocation implements LocationListener, SensorEventListener
 
     @Override
     public void onLocationChanged(Location location) {
-        mLastLocation = location;
+        if (!isBetterLocation(location, mLastLocation)) return;
 
-        // throw away all positions with accuracy > 100 meters should avoid gps glitches
-        if (location.getAccuracy() > 100) return;
+        mLastLocation = location;
 
         // do not touch the javascript while iitc boots
         if (mIitc.isLoading()) return;