Skip to content

Commit c85d340

Browse files
committed
Tinting removed for folder and file icons and not overlaying icon for folders.
1 parent 2e55cc5 commit c85d340

File tree

6 files changed

+26
-26
lines changed

6 files changed

+26
-26
lines changed

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
@@ -52,6 +52,8 @@
5252

5353
import javax.inject.Inject;
5454

55+
import androidx.core.content.ContextCompat;
56+
5557
public abstract class EditorWebView extends ExternalSiteWebView {
5658
public static final int REQUEST_LOCAL_FILE = 101;
5759
public ValueCallback<Uri[]> uploadMessage;
@@ -225,8 +227,8 @@ protected void setThumbnailView(final User user) {
225227
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
226228

227229
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
228-
LayerDrawable drawable = MimeTypeUtil.getFileIcon(preferences.isDarkModeEnabled(), overlayIconId, this, viewThemeUtils);
229-
binding.thumbnail.setImageDrawable(drawable);
230+
// NMC Customization: No overlay icon will be used. Directly using folder icons
231+
binding.thumbnail.setImageDrawable(ContextCompat.getDrawable(this, overlayIconId));
230232
} else {
231233
if ((MimeTypeUtil.isImage(file) || MimeTypeUtil.isVideo(file)) && file.getRemoteId() != null) {
232234
// 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
@@ -48,6 +48,7 @@
4848

4949
import javax.inject.Inject;
5050

51+
import androidx.core.content.ContextCompat;
5152
import androidx.fragment.app.Fragment;
5253
import androidx.fragment.app.FragmentTransaction;
5354

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

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

app/src/main/java/com/owncloud/android/ui/adapter/UploaderAdapter.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import android.content.Context;
2424
import android.graphics.Bitmap;
2525
import android.graphics.drawable.Drawable;
26-
import android.graphics.drawable.LayerDrawable;
2726
import android.view.LayoutInflater;
2827
import android.view.View;
2928
import android.view.ViewGroup;
@@ -32,7 +31,6 @@
3231
import android.widget.TextView;
3332

3433
import com.nextcloud.client.account.User;
35-
import com.nextcloud.client.preferences.DarkMode;
3634
import com.owncloud.android.R;
3735
import com.owncloud.android.datamodel.FileDataStorageManager;
3836
import com.owncloud.android.datamodel.OCFile;
@@ -47,6 +45,8 @@
4745
import java.util.List;
4846
import java.util.Map;
4947

48+
import androidx.core.content.ContextCompat;
49+
5050
public class UploaderAdapter extends SimpleAdapter {
5151

5252
private final Context mContext;
@@ -107,11 +107,10 @@ public View getView(int position, View convertView, ViewGroup parent) {
107107

108108
if (file.isFolder()) {
109109
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
110-
boolean isDarkModeActive = syncedFolderProvider.getPreferences().isDarkModeEnabled();
111110

112111
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
113-
final LayerDrawable icon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, mContext, viewThemeUtils);
114-
fileIcon.setImageDrawable(icon);
112+
// NMC Customization: No overlay icon will be used. Directly using folder icons
113+
fileIcon.setImageDrawable(ContextCompat.getDrawable(mContext, overlayIconId));
115114
} else {
116115
// get Thumbnail if file is image
117116
if (MimeTypeUtil.isImage(file) && file.getRemoteId() != null) {

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

Lines changed: 3 additions & 6 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;
@@ -67,7 +66,6 @@
6766
import com.nextcloud.client.account.User;
6867
import com.nextcloud.client.network.ClientFactory;
6968
import com.nextcloud.client.preferences.AppPreferences;
70-
import com.nextcloud.client.preferences.DarkMode;
7169
import com.owncloud.android.MainApp;
7270
import com.owncloud.android.R;
7371
import com.owncloud.android.datamodel.ArbitraryDataProvider;
@@ -112,6 +110,7 @@
112110
import androidx.annotation.Nullable;
113111
import androidx.annotation.StringRes;
114112
import androidx.appcompat.widget.AppCompatDrawableManager;
113+
import androidx.core.content.ContextCompat;
115114
import androidx.core.content.res.ResourcesCompat;
116115
import androidx.fragment.app.FragmentManager;
117116
import androidx.fragment.app.FragmentTransaction;
@@ -856,11 +855,9 @@ public static void setThumbnail(OCFile file,
856855
stopShimmer(shimmerThumbnail, thumbnailView);
857856

858857
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
859-
boolean isDarkModeActive = preferences.isDarkModeEnabled();
860-
861858
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
862-
LayerDrawable fileIcon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, context, viewThemeUtils);
863-
thumbnailView.setImageDrawable(fileIcon);
859+
// NMC Customization: No overlay icon will be used. Directly using folder icons
860+
thumbnailView.setImageDrawable(ContextCompat.getDrawable(context, overlayIconId));
864861
} else {
865862
if (file.getRemoteId() != null && file.isPreviewAvailable()) {
866863
// 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
@@ -108,13 +108,8 @@ public static Drawable getFileTypeIcon(String mimetype,
108108
ViewThemeUtils viewThemeUtils) {
109109
if (context != null) {
110110
int iconId = MimeTypeUtil.getFileTypeIconId(mimetype, filename);
111-
Drawable icon = ContextCompat.getDrawable(context, iconId);
112-
113-
if (R.drawable.file_zip == iconId) {
114-
viewThemeUtils.platform.tintPrimaryDrawable(context, icon);
115-
}
116-
117-
return icon;
111+
//NMC Customization
112+
return ContextCompat.getDrawable(context, iconId);
118113
} else {
119114
return null;
120115
}
@@ -142,11 +137,13 @@ public static Drawable getDefaultFolderIcon(Context context, ViewThemeUtils view
142137
Drawable drawable = ContextCompat.getDrawable(context, R.drawable.folder);
143138
assert(drawable != null);
144139

145-
viewThemeUtils.platform.tintDrawable(context, drawable, ColorRole.PRIMARY);
146140
return drawable;
147141
}
148142

149-
public static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
143+
// 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.
144+
// 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
145+
// so that UI won't be impacted.
146+
private static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
150147
Drawable folderDrawable = getDefaultFolderIcon(context, viewThemeUtils);
151148
assert(folderDrawable != null);
152149

0 commit comments

Comments
 (0)