Various bug fixes

Author: micwallace

reddinator/src/main/AndroidManifest.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="au.com.wallaceit.reddinator"
-    android:versionCode="35"
+    android:versionCode="36"
     android:versionName="3.04">
 
     <uses-permission android:name="android.permission.INTERNET" />

reddinator/src/main/java/au/com/wallaceit/reddinator/Reddinator.java

@@ -447,6 +447,6 @@ public static boolean isImageUrl(String url) {
     }
 
     public static boolean hasImageExtension(String url){
-        return url.toLowerCase().matches("([^\\s]+(\\.(?i)(jpe?g|png|gif|bmp))$)");
+        return url.toLowerCase().matches("([^\\s]+(\\.(?i)(jpe?g|png|gif?v|bmp))$)");
     }
 }

reddinator/src/main/java/au/com/wallaceit/reddinator/activity/ViewImageDialogActivity.java

@@ -17,6 +17,7 @@
  */
 package au.com.wallaceit.reddinator.activity;
 
+import android.annotation.SuppressLint;
 import android.app.Activity;
 import android.content.Intent;
 import android.content.pm.PackageManager;
@@ -37,6 +38,7 @@ public class ViewImageDialogActivity extends Activity {
     WebView webView;
     String imageUrl;
 
+    @SuppressLint("SetJavaScriptEnabled")
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
@@ -53,6 +55,7 @@ protected void onCreate(Bundle savedInstanceState) {
         webView.setBackgroundColor(Color.TRANSPARENT);
         webView.setWebViewClient(new ImageWebViewClient());
         webView.getSettings().setDomStorageEnabled(true);
+        webView.getSettings().setJavaScriptEnabled(true);
         webView.getSettings().setLoadWithOverviewMode(true);
         webView.getSettings().setUseWideViewPort(true);
         webView.getSettings().setSupportZoom(true);
@@ -82,5 +85,9 @@ class ImageWebViewClient extends WebViewClient {
         public void onPageFinished(WebView webView, String url) {
             findViewById(R.id.loadingPanel).setVisibility(View.GONE);
         }
+
+        public boolean shouldOverrideUrlLoading(WebView view, String url) {
+            return true;
+        }
     }
 }

reddinator/src/main/res/layout/applistrowbigthumb.xml

@@ -25,6 +25,24 @@
             android:scaleType="centerCrop"
             android:visibility="gone" />
 
+        <ImageView
+            android:id="@+id/thumbnail_expand"
+            android:layout_width="18dp"
+            android:layout_height="18dp"
+            android:maxHeight="18dp"
+            android:maxWidth="18dp"
+            android:minHeight="18dp"
+            android:minWidth="18dp"
+            android:layout_alignParentLeft="true"
+            android:layout_alignParentStart="true"
+            android:layout_alignParentTop="true"
+            android:layout_marginLeft="8dp"
+            android:layout_marginStart="8dp"
+            android:layout_marginTop="68dp"
+            android:contentDescription="@string/img"
+            android:scaleType="centerCrop"
+            android:visibility="gone" />
+
         <TextView
             android:id="@+id/listheading"
             android:layout_width="fill_parent"
@@ -53,6 +71,7 @@
             android:layout_height="wrap_content"
             android:layout_alignParentRight="true"
             android:layout_alignParentEnd="true"
+            android:layout_alignParentTop="true"
             android:orientation="vertical"
             android:layout_marginRight="6dp"
             android:layout_marginEnd="6dp">

reddinator/src/main/res/layout/listrowbigthumb.xml

@@ -16,6 +16,7 @@
             android:layout_height="wrap_content"
             android:layout_alignParentRight="true"
             android:layout_alignParentEnd="true"
+            android:layout_alignParentTop="true"
             android:orientation="vertical"
             android:gravity="center_horizontal"
             android:layout_marginRight="8dp"
@@ -62,8 +63,6 @@
             android:layout_height="80dp"
             android:maxHeight="80dp"
             android:maxWidth="80dp"
-            android:layout_toLeftOf="@+id/widget_votecontainer"
-            android:layout_toStartOf="@+id/widget_votecontainer"
             android:layout_centerHorizontal="true"
             android:layout_marginLeft="6dp"
             android:layout_marginRight="6dp"
@@ -72,6 +71,24 @@
             android:scaleType="centerCrop"
             android:visibility="gone" />
 
+        <ImageView
+            android:id="@+id/thumbnail_expand"
+            android:layout_width="18dp"
+            android:layout_height="18dp"
+            android:maxHeight="18dp"
+            android:maxWidth="18dp"
+            android:minHeight="18dp"
+            android:minWidth="18dp"
+            android:layout_alignParentLeft="true"
+            android:layout_alignParentStart="true"
+            android:layout_alignParentTop="true"
+            android:layout_marginLeft="8dp"
+            android:layout_marginStart="8dp"
+            android:layout_marginTop="68dp"
+            android:contentDescription="@string/img"
+            android:scaleType="centerCrop"
+            android:visibility="gone" />
+
         <TextView
             android:id="@+id/listheading"
             android:layout_width="fill_parent"