浏览代码

ProfileBottomSheetDialog: fix icon colors and detekt issues

Signed-off-by: Álvaro Brey <alvaro.brey@nextcloud.com>
Álvaro Brey 2 年之前
父节点
当前提交
131bd9e62c

+ 1 - 2
app/src/androidTest/java/com/owncloud/android/ui/dialog/DialogFragmentIT.java

@@ -438,8 +438,7 @@ public class DialogFragmentIT extends AbstractIT {
         ProfileBottomSheetDialog sut = new ProfileBottomSheetDialog(fda,
         ProfileBottomSheetDialog sut = new ProfileBottomSheetDialog(fda,
                                                                     user,
                                                                     user,
                                                                     hoverCard,
                                                                     hoverCard,
-                                                                    fda.themeColorUtils,
-                                                                    fda.themeDrawableUtils);
+                                                                    fda.viewThemeUtils);
 
 
         fda.runOnUiThread(sut::show);
         fda.runOnUiThread(sut::show);
 
 

+ 0 - 3
app/src/main/java/com/owncloud/android/ui/activity/DrawerActivity.java

@@ -209,9 +209,6 @@ public abstract class DrawerActivity extends ToolbarActivity
     @Inject
     @Inject
     ClientFactory clientFactory;
     ClientFactory clientFactory;
 
 
-    @Inject
-    ViewThemeUtils viewThemeUtils;
-
     /**
     /**
      * Initializes the drawer, its content and highlights the menu item with the given id. This method needs to be
      * Initializes the drawer, its content and highlights the menu item with the given id. This method needs to be
      * called after the content view has been set.
      * called after the content view has been set.

+ 0 - 2
app/src/main/java/com/owncloud/android/ui/asynctasks/RetrieveHoverCardAsyncTask.java

@@ -91,8 +91,6 @@ public class RetrieveHoverCardAsyncTask extends AsyncTask<Void, Void, HoverCard>
                 new ProfileBottomSheetDialog(activity,
                 new ProfileBottomSheetDialog(activity,
                                              user,
                                              user,
                                              hoverCard,
                                              hoverCard,
-                                             themeColorUtils,
-                                             themeDrawableUtils,
                                              viewThemeUtils)
                                              viewThemeUtils)
                     .show();
                     .show();
             } else {
             } else {

+ 1 - 7
app/src/main/java/com/owncloud/android/ui/fragment/ProfileBottomSheetDialog.kt

@@ -39,8 +39,6 @@ import com.owncloud.android.R
 import com.owncloud.android.databinding.ProfileBottomSheetActionBinding
 import com.owncloud.android.databinding.ProfileBottomSheetActionBinding
 import com.owncloud.android.databinding.ProfileBottomSheetFragmentBinding
 import com.owncloud.android.databinding.ProfileBottomSheetFragmentBinding
 import com.owncloud.android.utils.DisplayUtils
 import com.owncloud.android.utils.DisplayUtils
-import com.owncloud.android.utils.theme.ThemeColorUtils
-import com.owncloud.android.utils.theme.ThemeDrawableUtils
 import com.owncloud.android.utils.theme.newm3.ViewThemeUtils
 import com.owncloud.android.utils.theme.newm3.ViewThemeUtils
 
 
 /**
 /**
@@ -50,8 +48,6 @@ class ProfileBottomSheetDialog(
     private val fileActivity: FragmentActivity,
     private val fileActivity: FragmentActivity,
     private val user: User,
     private val user: User,
     private val hoverCard: HoverCard,
     private val hoverCard: HoverCard,
-    private val themeColorUtils: ThemeColorUtils,
-    private val themeDrawableUtils: ThemeDrawableUtils,
     private val viewThemeUtils: ViewThemeUtils
     private val viewThemeUtils: ViewThemeUtils
 ) : BottomSheetDialog(fileActivity), DisplayUtils.AvatarGenerationListener {
 ) : BottomSheetDialog(fileActivity), DisplayUtils.AvatarGenerationListener {
     private var _binding: ProfileBottomSheetFragmentBinding? = null
     private var _binding: ProfileBottomSheetFragmentBinding? = null
@@ -69,8 +65,6 @@ class ProfileBottomSheetDialog(
 
 
         viewThemeUtils.platform.themeDialog(binding.root)
         viewThemeUtils.platform.themeDialog(binding.root)
 
 
-        val primaryColor = themeColorUtils.primaryColor(context, true)
-
         binding.icon.tag = hoverCard.userId
         binding.icon.tag = hoverCard.userId
         DisplayUtils.setAvatar(
         DisplayUtils.setAvatar(
             user,
             user,
@@ -111,7 +105,7 @@ class ProfileBottomSheetDialog(
                     null
                     null
                 )
                 )
             )
             )
-            themeDrawableUtils.tintDrawable(actionBinding.icon.drawable, primaryColor)
+            viewThemeUtils.platform.tintPrimaryDrawable(context, actionBinding.icon.drawable)
 
 
             creatorView.setOnClickListener { v: View? ->
             creatorView.setOnClickListener { v: View? ->
                 send(hoverCard.userId, action)
                 send(hoverCard.userId, action)