소스 검색

merged part of the fixes from material buttons branch to fix bugs reported in PR #1076

Andy Scherzinger 9 년 전
부모
커밋
92af92ee3b
26개의 변경된 파일65개의 추가작업 그리고 41개의 파일을 삭제
  1. 2 2
      res/layout/drawer.xml
  2. 3 2
      res/layout/drawer_list_item.xml
  3. 0 1
      res/values-v11/versioned_styles.xml
  4. 1 1
      res/values/attrs.xml
  5. 3 1
      res/values/colors.xml
  6. 11 1
      res/values/dims.xml
  7. 13 2
      res/values/styles.xml
  8. 0 1
      src/com/owncloud/android/authentication/AccountAuthenticatorActivity.java
  9. 2 2
      src/com/owncloud/android/ui/activity/ErrorsWhileCopyingHandlerActivity.java
  10. 1 1
      src/com/owncloud/android/ui/activity/FileDisplayActivity.java
  11. 2 3
      src/com/owncloud/android/ui/activity/GenericExplanationActivity.java
  12. 2 3
      src/com/owncloud/android/ui/activity/LogHistoryActivity.java
  13. 2 5
      src/com/owncloud/android/ui/activity/PassCodeActivity.java
  14. 2 2
      src/com/owncloud/android/ui/activity/Uploader.java
  15. 1 1
      src/com/owncloud/android/ui/dialog/ChangelogDialog.java
  16. 1 1
      src/com/owncloud/android/ui/dialog/ConflictsResolveDialog.java
  17. 1 1
      src/com/owncloud/android/ui/dialog/CreateFolderDialogFragment.java
  18. 2 2
      src/com/owncloud/android/ui/dialog/CredentialsDialogFragment.java
  19. 7 0
      src/com/owncloud/android/ui/dialog/LoadingDialog.java
  20. 1 1
      src/com/owncloud/android/ui/dialog/RenameFileDialogFragment.java
  21. 1 1
      src/com/owncloud/android/ui/dialog/ShareLinkToDialog.java
  22. 1 1
      src/com/owncloud/android/ui/dialog/SharePasswordDialogFragment.java
  23. 1 1
      src/com/owncloud/android/ui/dialog/UploadSourceDialogFragment.java
  24. 3 3
      src/com/owncloud/android/ui/fragment/ExtendedListFragment.java
  25. 1 1
      src/com/owncloud/android/ui/preview/PreviewMediaFragment.java
  26. 1 1
      src/com/owncloud/android/ui/preview/PreviewVideoActivity.java

+ 2 - 2
res/layout/drawer.xml

@@ -19,7 +19,7 @@
 <merge xmlns:android="http://schemas.android.com/apk/res/android">
     <RelativeLayout
         android:id="@+id/left_drawer"
-        android:layout_width="match_parent"
+        android:layout_width="240dp"
         android:layout_height="match_parent"
         android:layout_gravity="start"
         android:gravity="center"
@@ -92,7 +92,7 @@
                 android:orientation="vertical"
                 android:text="@string/app_name"
                 android:textColor="#FFF"
-                android:paddingLeft="16dp"
+                android:paddingLeft="22dp"
                 android:paddingRight="16dp"
                 android:textSize="24sp"
                 android:ellipsize="end"

+ 3 - 2
res/layout/drawer_list_item.xml

@@ -40,12 +40,13 @@
         android:id="@+id/itemTitle"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:paddingLeft="16dp"
+        android:paddingLeft="22dp"
         android:paddingRight="16dp"
         android:textColor="@color/drawerMenuTextColor"
         android:text="@string/app_name"
+        android:textStyle="bold"
         android:layout_gravity="center_vertical"
-        android:textAppearance="?android:attr/textAppearanceListItem"
+        android:textSize="14sp"
         />
 </LinearLayout>
 

+ 0 - 1
res/values-v11/versioned_styles.xml

@@ -6,7 +6,6 @@
 		<item name="android:textColor">@color/button_text_color</item>
 	</style>
 
-	<!-- Notifications -->
 	<!-- Notifications -->
 	<style 	name="Theme.ownCloud.NotificationText.Title" 
 	    	parent="@android:style/TextAppearance.StatusBar.EventContent.Title" > 

+ 1 - 1
res/values/attrs.xml

@@ -5,7 +5,7 @@
 
 		<attr name="optionOneString" format="string" />
 		<attr name="optionOneColor" format="color" />
-    <attr name="optionTwoString" format="string" />
+    	<attr name="optionTwoString" format="string" />
 		<attr name="optionTwoColor" format="color" />
 		<attr name="onBadgeClick" format="string" />
 

+ 3 - 1
res/values/colors.xml

@@ -27,12 +27,14 @@
     <color name="black">#000000</color>
     <color name="white">#FFFFFF</color>
     <color name="textColor">#303030</color>
-    <color name="drawerMenuTextColor">#757575</color>
+    <color name="drawerMenuTextColor">#000000</color>
     <color name="list_divider_background">#fff0f0f0</color>
     <color name="filelist_icon_backgorund">#DDDDDD</color>
     <color name="dark_background_text_color">#EEEEEE</color>
     <color name="login_text_color">@color/dark_background_text_color</color>
     <color name="login_text_hint_color">#777777</color>
+    <color name="primary_button_color">@color/owncloud_blue_accent</color>
+    <color name="secondary_button_color">#D6D7D7</color>
 
     <!-- Colors -->
     <color name="color_accent">@color/owncloud_blue_accent</color>

+ 11 - 1
res/values/dims.xml

@@ -17,6 +17,16 @@
   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 -->
 <resources>
-	<dimen name="file_icon_size">32dp</dimen>
+    <!-- Default screen margins, per the Android Design guidelines. -->
+    <dimen name="activity_horizontal_margin">@dimen/standard_padding</dimen>
+    <dimen name="activity_vertical_margin">@dimen/standard_padding</dimen>
+    <dimen name="nav_drawer_width">260dp</dimen>
+
+    <dimen name="file_icon_size">32dp</dimen>
     <dimen name="file_icon_size_grid">128dp</dimen>
+    <dimen name="standard_padding">16dp</dimen>
+    <dimen name="standard_margin">16dp</dimen>
+
+    <dimen name="two_line_primary_text_size">16sp</dimen>
+    <dimen name="two_line_secondary_text_size">14sp</dimen>
 </resources>

+ 13 - 2
res/values/styles.xml

@@ -21,7 +21,7 @@
 	<style name="Animations" />
 
 	<!-- General ownCloud app style -->
-	<style name="Theme.ownCloud" parent="style/Theme.AppCompat.Light.DarkActionBar">
+	<style name="Theme.ownCloud" parent="Theme.AppCompat.Light.DarkActionBar">
     	<item name="android:actionBarStyle">@style/Theme.ownCloud.Widget.ActionBar</item>
     	<item name="actionBarStyle">@style/Theme.ownCloud.Widget.ActionBar</item>
     	<item name="android:buttonStyle">@style/Theme.ownCloud.ButtonStyle</item>
@@ -29,13 +29,14 @@
 		<item name="colorPrimaryDark">@color/primary_dark</item>
 		<item name="colorAccent">@color/color_accent</item>
 		<item name="android:alertDialogTheme">@style/Theme.ownCloud.Dialog</item>
+		<item name="alertDialogTheme">@style/ownCloud.AlertDialog</item>
 		<item name="colorButtonNormal">@color/background_color</item>
 		<item name="colorControlActivated">@color/primary_dark</item>
 		<item name="colorControlHighlight">@color/color_accent</item>
 	</style>
 
 	<!-- seperate action bar style for activities without an action bar -->
-	<style name="Theme.ownCloud.noActionBar" parent="style/Theme.AppCompat.Light.DarkActionBar">
+	<style name="Theme.ownCloud.noActionBar" parent="Theme.AppCompat.Light.DarkActionBar">
 		<item name="android:buttonStyle">@style/Theme.ownCloud.ButtonStyle</item>
 		<item name="windowNoTitle">true</item>
 		<item name="windowActionBar">false</item>
@@ -43,6 +44,7 @@
 		<item name="colorPrimaryDark">@color/primary_dark</item>
 		<item name="colorAccent">@color/color_accent</item>
 		<item name="android:alertDialogTheme">@style/Theme.ownCloud.Dialog</item>
+		<item name="alertDialogTheme">@style/ownCloud.AlertDialog</item>
 		<item name="colorButtonNormal">@color/background_color</item>
 		<item name="colorControlActivated">@color/primary_dark</item>
 		<item name="colorControlHighlight">@color/color_accent</item>
@@ -52,6 +54,15 @@
 		<item name="colorAccent">@color/white</item>
 	</style>
 
+	<style name="ownCloud.AlertDialog" parent="Theme.AppCompat.Light.Dialog.Alert">
+		<item name="colorAccent">@color/color_accent</item>
+		<item name="android:textColorPrimary">@color/primary</item>
+	</style>
+
+	<style name="ownCloud.Dialog" parent="Theme.AppCompat.Light.Dialog">
+		
+	</style>
+
 	<style name="ownCloud.Button" parent="Widget.AppCompat.Button">
 		<item name="colorButtonNormal">@color/primary</item>
 		<item name="colorControlActivated">@color/primary_dark</item>

+ 0 - 1
src/com/owncloud/android/authentication/AccountAuthenticatorActivity.java

@@ -19,7 +19,6 @@ package com.owncloud.android.authentication;
 import android.accounts.AccountAuthenticatorResponse;
 import android.accounts.AccountManager;
 import android.os.Bundle;
-import android.support.v7.app.ActionBarActivity;
 import android.support.v7.app.AppCompatActivity;
 
 /*

+ 2 - 2
src/com/owncloud/android/ui/activity/ErrorsWhileCopyingHandlerActivity.java

@@ -30,7 +30,7 @@ import android.os.AsyncTask;
 import android.os.Bundle;
 import android.os.Handler;
 import android.support.v4.app.DialogFragment;
-import android.support.v7.app.ActionBarActivity;
+import android.support.v7.app.AppCompatActivity;
 import android.text.method.ScrollingMovementMethod;
 import android.view.LayoutInflater;
 import android.view.View;
@@ -61,7 +61,7 @@ import com.owncloud.android.utils.FileStorageUtils;
  * 
  * Shown when the error notification summarizing the list of errors is clicked by the user.
  */
-public class ErrorsWhileCopyingHandlerActivity  extends ActionBarActivity
+public class ErrorsWhileCopyingHandlerActivity  extends AppCompatActivity
         implements OnClickListener {
 
     private static final String TAG = ErrorsWhileCopyingHandlerActivity.class.getSimpleName();

+ 1 - 1
src/com/owncloud/android/ui/activity/FileDisplayActivity.java

@@ -26,7 +26,7 @@ import android.accounts.Account;
 import android.accounts.AccountManager;
 import android.accounts.AuthenticatorException;
 import android.annotation.TargetApi;
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.content.BroadcastReceiver;
 import android.content.ComponentName;
 import android.content.ContentResolver;

+ 2 - 3
src/com/owncloud/android/ui/activity/GenericExplanationActivity.java

@@ -25,8 +25,7 @@ import java.util.ArrayList;
 import android.content.Context;
 import android.content.Intent;
 import android.os.Bundle;
-import android.support.v7.app.ActionBar;
-import android.support.v7.app.ActionBarActivity;
+import android.support.v7.app.AppCompatActivity;
 import android.text.method.ScrollingMovementMethod;
 import android.view.View;
 import android.view.ViewGroup;
@@ -45,7 +44,7 @@ import com.owncloud.android.utils.DisplayUtils;
  * Added to show explanations for notifications when the user clicks on them, and there no place
  * better to show them.
  */
-public class GenericExplanationActivity  extends ActionBarActivity {
+public class GenericExplanationActivity  extends AppCompatActivity {
 
     public static final String EXTRA_LIST = GenericExplanationActivity.class.getCanonicalName() +
             ".EXTRA_LIST";

+ 2 - 3
src/com/owncloud/android/ui/activity/LogHistoryActivity.java

@@ -35,8 +35,7 @@ import android.os.Bundle;
 import android.support.v4.app.Fragment;
 import android.support.v4.app.FragmentManager;
 import android.support.v4.app.FragmentTransaction;
-import android.support.v7.app.ActionBar;
-import android.support.v7.app.ActionBarActivity;
+import android.support.v7.app.AppCompatActivity;
 import android.view.MenuItem;
 import android.view.View;
 import android.view.View.OnClickListener;
@@ -51,7 +50,7 @@ import com.owncloud.android.utils.DisplayUtils;
 import com.owncloud.android.utils.FileStorageUtils;
 
 
-public class LogHistoryActivity extends ActionBarActivity {
+public class LogHistoryActivity extends AppCompatActivity {
 
     private static final String MAIL_ATTACHMENT_TYPE = "text/plain";
 

+ 2 - 5
src/com/owncloud/android/ui/activity/PassCodeActivity.java

@@ -27,8 +27,7 @@ import java.util.Arrays;
 import android.content.SharedPreferences;
 import android.os.Bundle;
 import android.preference.PreferenceManager;
-import android.support.v7.app.ActionBar;
-import android.support.v7.app.ActionBarActivity;
+import android.support.v7.app.AppCompatActivity;
 import android.text.Editable;
 import android.text.TextWatcher;
 import android.view.KeyEvent;
@@ -41,10 +40,8 @@ import android.widget.Toast;
 
 import com.owncloud.android.R;
 import com.owncloud.android.lib.common.utils.Log_OC;
-import com.owncloud.android.utils.DisplayUtils;
-
-public class PassCodeActivity extends ActionBarActivity {
 
+public class PassCodeActivity extends AppCompatActivity {
 
     private static final String TAG = PassCodeActivity.class.getSimpleName();
 

+ 2 - 2
src/com/owncloud/android/ui/activity/Uploader.java

@@ -33,8 +33,8 @@ import java.util.Vector;
 
 import android.accounts.Account;
 import android.accounts.AccountManager;
-import android.app.AlertDialog;
-import android.app.AlertDialog.Builder;
+import android.support.v7.app.AlertDialog;
+import android.support.v7.app.AlertDialog.Builder;
 import android.app.Dialog;
 import android.app.ProgressDialog;
 import android.content.Context;

+ 1 - 1
src/com/owncloud/android/ui/dialog/ChangelogDialog.java

@@ -19,7 +19,7 @@
 
 package com.owncloud.android.ui.dialog;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.os.Bundle;

+ 1 - 1
src/com/owncloud/android/ui/dialog/ConflictsResolveDialog.java

@@ -21,7 +21,7 @@
 
 package com.owncloud.android.ui.dialog;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.os.Bundle;

+ 1 - 1
src/com/owncloud/android/ui/dialog/CreateFolderDialogFragment.java

@@ -25,7 +25,7 @@ import com.owncloud.android.datamodel.OCFile;
 import com.owncloud.android.lib.resources.files.FileUtils;
 import com.owncloud.android.ui.activity.ComponentsGetter;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.os.Bundle;

+ 2 - 2
src/com/owncloud/android/ui/dialog/CredentialsDialogFragment.java

@@ -22,9 +22,9 @@ package com.owncloud.android.ui.dialog;
 import com.owncloud.android.R;
 import com.owncloud.android.authentication.AuthenticatorActivity;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
-import android.app.AlertDialog.Builder;
+import android.support.v7.app.AlertDialog.Builder;
 import android.content.DialogInterface;
 import android.os.Bundle;
 import android.support.v4.app.DialogFragment;

+ 7 - 0
src/com/owncloud/android/ui/dialog/LoadingDialog.java

@@ -21,12 +21,14 @@ package com.owncloud.android.ui.dialog;
 import com.owncloud.android.R;
 
 import android.app.Dialog;
+import android.graphics.PorterDuff;
 import android.os.Bundle;
 import android.support.v4.app.DialogFragment;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.Window;
+import android.widget.ProgressBar;
 import android.widget.TextView;
 
 public class LoadingDialog extends DialogFragment {
@@ -56,6 +58,11 @@ public class LoadingDialog extends DialogFragment {
         // set value
         TextView tv  = (TextView) v.findViewById(R.id.loadingText);
         tv.setText(mMessage);
+
+        // set progress wheel color
+        ProgressBar progressBar  = (ProgressBar) v.findViewById(R.id.loadingBar);
+        progressBar.getIndeterminateDrawable().setColorFilter(
+                getResources().getColor(R.color.color_accent), PorterDuff.Mode.SRC_IN);
         
         return v;
     }

+ 1 - 1
src/com/owncloud/android/ui/dialog/RenameFileDialogFragment.java

@@ -25,7 +25,7 @@ package com.owncloud.android.ui.dialog;
  * 
  *  Triggers the rename operation. 
  */
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.os.Bundle;

+ 1 - 1
src/com/owncloud/android/ui/dialog/ShareLinkToDialog.java

@@ -25,7 +25,7 @@ import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.ComponentName;
 import android.content.Context;

+ 1 - 1
src/com/owncloud/android/ui/dialog/SharePasswordDialogFragment.java

@@ -18,7 +18,7 @@
  */
 package com.owncloud.android.ui.dialog;
 
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.content.Intent;

+ 1 - 1
src/com/owncloud/android/ui/dialog/UploadSourceDialogFragment.java

@@ -20,7 +20,7 @@
 package com.owncloud.android.ui.dialog;
 
 import android.accounts.Account;
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.app.Dialog;
 import android.content.DialogInterface;
 import android.content.Intent;

+ 3 - 3
src/com/owncloud/android/ui/fragment/ExtendedListFragment.java

@@ -338,9 +338,9 @@ public class ExtendedListFragment extends Fragment
     }
 
     private void onCreateSwipeToRefresh(SwipeRefreshLayout refreshLayout) {
-        // Colors in animations: background
-        refreshLayout.setColorScheme(R.color.background_color, R.color.background_color,
-                R.color.background_color, R.color.background_color);
+        // Colors in animations
+        refreshLayout.setColorSchemeResources(R.color.color_accent, R.color.primary,
+                R.color.primary_dark);
 
         refreshLayout.setOnRefreshListener(this);
     }

+ 1 - 1
src/com/owncloud/android/ui/preview/PreviewMediaFragment.java

@@ -21,7 +21,7 @@ package com.owncloud.android.ui.preview;
 
 import android.accounts.Account;
 import android.app.Activity;
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.content.ComponentName;
 import android.content.Context;
 import android.content.DialogInterface;

+ 1 - 1
src/com/owncloud/android/ui/preview/PreviewVideoActivity.java

@@ -26,7 +26,7 @@ import com.owncloud.android.media.MediaService;
 import com.owncloud.android.ui.activity.FileActivity;
 
 import android.accounts.Account;
-import android.app.AlertDialog;
+import android.support.v7.app.AlertDialog;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.media.MediaPlayer;