Skip to content

Commit e05a14e

Browse files
committed
Tinting removed for folder and file icons and not overlaying icon for folders.
1 parent f59caa7 commit e05a14e

6 files changed

Lines changed: 25 additions & 23 deletions

File tree

app/src/main/java/com/nextcloud/utils/ShortcutUtil.kt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ import android.graphics.Bitmap
2929
import android.graphics.Canvas
3030
import android.graphics.drawable.BitmapDrawable
3131
import android.graphics.drawable.Drawable
32+
import androidx.core.content.ContextCompat
3233
import androidx.core.content.pm.ShortcutInfoCompat
3334
import androidx.core.content.pm.ShortcutManagerCompat
3435
import androidx.core.graphics.drawable.IconCompat
@@ -72,10 +73,13 @@ class ShortcutUtil @Inject constructor(private val mContext: Context) {
7273
icon = IconCompat.createWithAdaptiveBitmap(thumbnail)
7374
} else if (file.isFolder) {
7475
val isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user)
75-
val isDarkModeActive = syncedFolderProvider.preferences.isDarkModeEnabled
7676

7777
val overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder)
78-
val drawable = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, mContext, viewThemeUtils)
78+
// NMC Customization: No overlay icon will be used. Directly using folder icons
79+
val drawable = ContextCompat.getDrawable(mContext, overlayIconId) ?: MimeTypeUtil.getDefaultFolderIcon(
80+
mContext,
81+
viewThemeUtils
82+
)
7983
val bitmapIcon = drawable.toBitmap()
8084
icon = IconCompat.createWithBitmap(bitmapIcon)
8185
} else {

app/src/main/java/com/owncloud/android/ui/activity/EditorWebView.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@
5353

5454
import javax.inject.Inject;
5555

56+
import androidx.core.content.ContextCompat;
57+
5658
public abstract class EditorWebView extends ExternalSiteWebView {
5759
public static final int REQUEST_LOCAL_FILE = 101;
5860
public ValueCallback<Uri[]> uploadMessage;
@@ -231,8 +233,8 @@ protected void setThumbnailView(final User user) {
231233
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
232234

233235
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
234-
LayerDrawable drawable = MimeTypeUtil.getFileIcon(preferences.isDarkModeEnabled(), overlayIconId, this, viewThemeUtils);
235-
binding.thumbnail.setImageDrawable(drawable);
236+
// NMC Customization: No overlay icon will be used. Directly using folder icons
237+
binding.thumbnail.setImageDrawable(ContextCompat.getDrawable(this, overlayIconId));
236238
} else {
237239
if ((MimeTypeUtil.isImage(file) || MimeTypeUtil.isVideo(file)) && file.getRemoteId() != null) {
238240
// Thumbnail in cache?

app/src/main/java/com/owncloud/android/ui/activity/ShareActivity.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@
4747

4848
import javax.inject.Inject;
4949

50+
import androidx.core.content.ContextCompat;
5051
import androidx.fragment.app.Fragment;
5152
import androidx.fragment.app.FragmentTransaction;
5253

@@ -81,8 +82,8 @@ protected void onCreate(Bundle savedInstanceState) {
8182
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, optionalUser.get());
8283

8384
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
84-
LayerDrawable drawable = MimeTypeUtil.getFileIcon(preferences.isDarkModeEnabled(), overlayIconId, this, viewThemeUtils);
85-
binding.shareFileIcon.setImageDrawable(drawable);
85+
// NMC Customization: No overlay icon will be used. Directly using folder icons
86+
binding.shareFileIcon.setImageDrawable(ContextCompat.getDrawable(this, overlayIconId));
8687
} else {
8788
binding.shareFileIcon.setImageDrawable(MimeTypeUtil.getFileTypeIcon(file.getMimeType(),
8889
file.getFileName(),

app/src/main/java/com/owncloud/android/ui/adapter/ReceiveExternalFilesAdapter.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import android.view.LayoutInflater
2727
import android.view.View
2828
import android.view.ViewGroup
2929
import android.widget.ImageView
30+
import androidx.core.content.ContextCompat
3031
import androidx.recyclerview.widget.RecyclerView
3132
import com.nextcloud.client.account.User
3233
import com.owncloud.android.databinding.UploaderListItemLayoutBinding
@@ -127,10 +128,9 @@ class ReceiveExternalFilesAdapter(
127128

128129
private fun setupThumbnailForFolder(thumbnailImageView: ImageView, file: OCFile) {
129130
val isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user)
130-
val isDarkModeActive = syncedFolderProvider.preferences.isDarkModeEnabled
131131
val overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder)
132-
val icon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, context, viewThemeUtils)
133-
thumbnailImageView.setImageDrawable(icon)
132+
// NMC Customization: No overlay icon will be used. Directly using folder icons
133+
thumbnailImageView.setImageDrawable(ContextCompat.getDrawable(context, overlayIconId))
134134
}
135135

136136
@Suppress("NestedBlockDepth")

app/src/main/java/com/owncloud/android/utils/DisplayUtils.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@
3838
import android.graphics.Point;
3939
import android.graphics.drawable.ColorDrawable;
4040
import android.graphics.drawable.Drawable;
41-
import android.graphics.drawable.LayerDrawable;
4241
import android.net.Uri;
4342
import android.os.AsyncTask;
4443
import android.text.Spannable;
@@ -111,6 +110,7 @@
111110
import androidx.annotation.Nullable;
112111
import androidx.annotation.StringRes;
113112
import androidx.appcompat.widget.AppCompatDrawableManager;
113+
import androidx.core.content.ContextCompat;
114114
import androidx.core.content.res.ResourcesCompat;
115115
import androidx.fragment.app.FragmentManager;
116116
import androidx.fragment.app.FragmentTransaction;
@@ -855,11 +855,9 @@ public static void setThumbnail(OCFile file,
855855
stopShimmer(shimmerThumbnail, thumbnailView);
856856

857857
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
858-
boolean isDarkModeActive = preferences.isDarkModeEnabled();
859-
860858
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
861-
LayerDrawable fileIcon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, context, viewThemeUtils);
862-
thumbnailView.setImageDrawable(fileIcon);
859+
// NMC Customization: No overlay icon will be used. Directly using folder icons
860+
thumbnailView.setImageDrawable(ContextCompat.getDrawable(context, overlayIconId));
863861
} else {
864862
if (file.getRemoteId() != null && file.isPreviewAvailable()) {
865863
// Thumbnail in cache?

app/src/main/java/com/owncloud/android/utils/MimeTypeUtil.java

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -104,13 +104,8 @@ public static Drawable getFileTypeIcon(String mimetype,
104104
ViewThemeUtils viewThemeUtils) {
105105
if (context != null) {
106106
int iconId = MimeTypeUtil.getFileTypeIconId(mimetype, filename);
107-
Drawable icon = ContextCompat.getDrawable(context, iconId);
108-
109-
if (R.drawable.file_zip == iconId) {
110-
viewThemeUtils.platform.tintPrimaryDrawable(context, icon);
111-
}
112-
113-
return icon;
107+
//NMC Customization
108+
return ContextCompat.getDrawable(context, iconId);
114109
} else {
115110
return null;
116111
}
@@ -138,11 +133,13 @@ public static Drawable getDefaultFolderIcon(Context context, ViewThemeUtils view
138133
Drawable drawable = ContextCompat.getDrawable(context, R.drawable.folder);
139134
assert(drawable != null);
140135

141-
viewThemeUtils.platform.tintDrawable(context, drawable, ColorRole.PRIMARY);
142136
return drawable;
143137
}
144138

145-
public static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
139+
// NMC Note: This funtion won't be used in NMC as we are using different folder icons with inbuilt overlay. So this function is of no use for us.
140+
// changed access to PRIVATE, in case if NC will use this function in more areas then we will get compile error which can be fixed by us
141+
// so that UI won't be impacted.
142+
private static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
146143
Drawable folderDrawable = getDefaultFolderIcon(context, viewThemeUtils);
147144
assert(folderDrawable != null);
148145

0 commit comments

Comments
 (0)