Fixed crash on horizontal swipe past end of search results.
authorkenstir <kenstir@gmail.com>
Tue, 31 Dec 2013 04:16:55 +0000 (23:16 -0500)
committerkenstir <kenstir@gmail.com>
Tue, 31 Dec 2013 04:16:55 +0000 (23:16 -0500)
Changed inexplicable "1 / 1" text to "1 of 1 copy available" in search results.

Open-ILS/src/Android/core/res/layout/record_details_basic_fragment.xml
Open-ILS/src/Android/core/res/layout/simple_underlines.xml
Open-ILS/src/Android/core/res/values/dimens.xml
Open-ILS/src/Android/core/res/values/strings.xml
Open-ILS/src/Android/core/res/values/styles.xml
Open-ILS/src/Android/core/src/org/evergreen_ils/searchCatalog/SampleUnderlinesNoFade.java
Open-ILS/src/Android/core/src/org/evergreen_ils/utils/ui/BaseSampleActivity.java
Open-ILS/src/Android/core/src/org/evergreen_ils/utils/ui/BasicDetailsFragment.java

index d02e8b5..aff3a10 100644 (file)
     <LinearLayout
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
-               style="@style/SearchDetailsInfoSeparator"
-         >
+        >
+        <!-- style="@style/SearchDetailsInfoSeparator" -->
 
         <TextView
             android:id="@+id/record_header_text"
             android:layout_width="fill_parent"
             android:layout_height="wrap_content"
-            android:textColor="@color/blue"
             android:gravity="center"
-            android:textSize="@dimen/search_details_rect_text_size"
             android:textStyle="bold"
             />
         
 
             <TextView
                 android:id="@+id/record_details_simple_copy_count"
-                style="@style/SeparatorInformation"
-               
-            />
+                android:textStyle="bold"
+                android:layout_height="wrap_content"
+                android:layout_width="match_parent"/>
+            <!-- style="@style/SeparatorInformation" -->
 
                        <LinearLayout
                            android:id="@+id/record_details_copy_information"
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
         android:gravity="center"
-               style="@style/SearchDetailsInfoSeparator"
                android:orientation="horizontal"
-         >
+        style="@style/SearchDetailsInfoSeparator"
+        >
 
-                   
                <ImageView 
                    android:layout_width="wrap_content"
                    android:layout_height="wrap_content"
             android:layout_height="wrap_content"
             android:textColor="@color/blue"
             android:text="@string/show_more_text"
-            android:textSize="@dimen/search_details_rect_text_size"
             android:textStyle="bold"
+            android:textSize="@dimen/search_details_rect_text_size"
         />
         
        </LinearLayout>
index bcad2dc..f1d6445 100644 (file)
@@ -22,8 +22,6 @@
     android:background="@color/background"
     >
 
-       <include android:id="@+id/header_actionbar" layout="@layout/simple_actionbar"/>
-
     <android.support.v4.view.ViewPager
         android:id="@+id/pager"
         android:layout_width="fill_parent"
@@ -35,7 +33,7 @@
         />
     <org.evergreen_ils.utils.ui.UnderlinePageIndicator
         android:id="@+id/indicator"
-        android:layout_height="2dp"
+        android:layout_height="3dp"
         android:layout_width="fill_parent"
         />
 </LinearLayout>
\ No newline at end of file
index d83619b..82c0900 100644 (file)
@@ -22,7 +22,7 @@
     <dimen name="text_size_medium">18sp</dimen>\r
     <dimen name="text_size_large">22sp</dimen>\r
     <dimen name="library_logo_text_size">16sp</dimen>\r
-    <dimen name="search_details_rect_text_size">16dip</dimen>\r
+    <dimen name="search_details_rect_text_size">12dip</dimen>\r
     <dimen name="activity_horizontal_margin">16dp</dimen>\r
     <dimen name="activity_vertical_margin">16dp</dimen>\r
     <dimen name="bigActionButton_marginVertical">18dp</dimen>\r
index f464c74..a4c6afc 100644 (file)
@@ -58,7 +58,9 @@
     <string name="title_book_bags">Bookbags</string>
     <string name="renew_button">Renew</string>
     <string name="item_renewed">Item renewed</string>
-    <string name="record_of">Record %1$d of %2$d</string>
+    <string name="record_of">record %1$d of %2$d</string>
+    <string name="displaying_n_of_m_results">displaying %1$d of %2$d results</string>
+    <string name="n_of_m_available">%1$d of %2$s available</string>
 
     <!-- Search Details View -->
     <string name="show_more_text">Show More</string>
index 5683638..33b0c2d 100644 (file)
     <!-- Search Details -->
     <style name="SearchDetailsInfoSeparator">
         <item name="android:background">@drawable/shadow_rect</item>
-        <item name="android:padding">5dip</item>
-        <item name="android:layout_marginTop">10dip</item>
+        <item name="android:padding">3dip</item>
+        <item name="android:layout_marginTop">3dip</item>
         <item name="android:textColor">@color/dark</item>
     </style>
 
index f2b2903..3161cac 100644 (file)
@@ -43,7 +43,7 @@ import android.view.View.OnClickListener;
 import android.widget.Button;
 import android.widget.TextView;
 
-public class SampleUnderlinesNoFade extends ActionBarActivity {
+public class SampleUnderlinesNoFade extends BaseSampleActivity {
 
     private ArrayList<RecordInfo> records;
 
@@ -63,10 +63,6 @@ public class SampleUnderlinesNoFade extends ActionBarActivity {
     
     private Runnable searchRunnableWithOffset;
 
-    public TestFragmentAdapter mAdapter;
-    public ViewPager mPager;
-    public PageIndicator mIndicator;
-
     public static final int RETURN_DATA = 5;
     
     @Override
@@ -136,6 +132,7 @@ public class SampleUnderlinesNoFade extends ActionBarActivity {
                             // option button
 
                         }
+                        mAdapter.notifyDataSetChanged();
                         progressDialog.dismiss();
                     }
                 });
index 46cb48d..dfb9621 100644 (file)
@@ -21,6 +21,7 @@ package org.evergreen_ils.utils.ui;
 
 import java.util.Random;
 
+import android.support.v7.app.ActionBarActivity;
 import org.evergreen_ils.R;
 
 import android.support.v4.app.FragmentActivity;
@@ -29,7 +30,7 @@ import android.view.Menu;
 import android.view.MenuItem;
 import android.widget.Toast;
 
-public abstract class BaseSampleActivity extends FragmentActivity {
+public abstract class BaseSampleActivity extends ActionBarActivity {
     private static final Random RANDOM = new Random();
 
     public TestFragmentAdapter mAdapter;
index fb9175b..d2c5160 100644 (file)
@@ -76,7 +76,7 @@ public class BasicDetailsFragment extends Fragment {
     private TextView synopsisTextView;
     private TextView isbnTextView;
 
-    private TextView copyCountTestView;
+    private TextView copyCountTextView;
 
     private Button placeHoldButton;
 
@@ -144,7 +144,7 @@ public class BasicDetailsFragment extends Fragment {
                 R.layout.record_details_basic_fragment, null);
 
         record_header = (TextView) layout.findViewById(R.id.record_header_text);
-        copyCountTestView = (TextView) layout
+        copyCountTextView = (TextView) layout
                 .findViewById(R.id.record_details_simple_copy_count);
         showMore = (LinearLayout) layout
                 .findViewById(R.id.record_details_show_more);
@@ -299,7 +299,8 @@ public class BasicDetailsFragment extends Fragment {
             if (record.copyCountListInfo.get(i).org_id == current_org) {
                 int total = record.copyCountListInfo.get(i).count;
                 int available = record.copyCountListInfo.get(i).available;
-                copyCountTestView.setText(available + " / " + total);
+                String totalCopies = getResources().getQuantityString(R.plurals.number_of_copies, total, total);
+                copyCountTextView.setText(String.format(getString(R.string.n_of_m_available), available, totalCopies));
                 break;
             }
         }