Skip to content

Commit dedcca3

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

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
@@ -16,6 +16,7 @@ import android.graphics.Bitmap
1616
import android.graphics.Canvas
1717
import android.graphics.drawable.BitmapDrawable
1818
import android.graphics.drawable.Drawable
19+
import androidx.core.content.ContextCompat
1920
import androidx.core.content.pm.ShortcutInfoCompat
2021
import androidx.core.content.pm.ShortcutManagerCompat
2122
import androidx.core.graphics.drawable.IconCompat
@@ -59,10 +60,13 @@ class ShortcutUtil @Inject constructor(private val mContext: Context) {
5960
icon = IconCompat.createWithAdaptiveBitmap(thumbnail)
6061
} else if (file.isFolder) {
6162
val isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user)
62-
val isDarkModeActive = syncedFolderProvider.preferences.isDarkModeEnabled
6363

6464
val overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder)
65-
val drawable = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, mContext, viewThemeUtils)
65+
// NMC Customization: No overlay icon will be used. Directly using folder icons
66+
val drawable = ContextCompat.getDrawable(mContext, overlayIconId) ?: MimeTypeUtil.getDefaultFolderIcon(
67+
mContext,
68+
viewThemeUtils
69+
)
6670
val bitmapIcon = drawable.toBitmap()
6771
icon = IconCompat.createWithBitmap(bitmapIcon)
6872
} 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
@@ -39,6 +39,8 @@
3939

4040
import javax.inject.Inject;
4141

42+
import androidx.core.content.ContextCompat;
43+
4244
public abstract class EditorWebView extends ExternalSiteWebView {
4345
public static final int REQUEST_LOCAL_FILE = 101;
4446
public ValueCallback<Uri[]> uploadMessage;
@@ -217,8 +219,8 @@ protected void setThumbnailView(final User user) {
217219
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
218220

219221
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
220-
LayerDrawable drawable = MimeTypeUtil.getFileIcon(preferences.isDarkModeEnabled(), overlayIconId, this, viewThemeUtils);
221-
binding.thumbnail.setImageDrawable(drawable);
222+
// NMC Customization: No overlay icon will be used. Directly using folder icons
223+
binding.thumbnail.setImageDrawable(ContextCompat.getDrawable(this, overlayIconId));
222224
} else {
223225
if ((MimeTypeUtil.isImage(file) || MimeTypeUtil.isVideo(file)) && file.getRemoteId() != null) {
224226
// 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
@@ -39,6 +39,7 @@
3939

4040
import javax.inject.Inject;
4141

42+
import androidx.core.content.ContextCompat;
4243
import androidx.fragment.app.Fragment;
4344
import androidx.fragment.app.FragmentTransaction;
4445

@@ -73,8 +74,8 @@ protected void onCreate(Bundle savedInstanceState) {
7374
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, optionalUser.get());
7475

7576
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
76-
LayerDrawable drawable = MimeTypeUtil.getFileIcon(preferences.isDarkModeEnabled(), overlayIconId, this, viewThemeUtils);
77-
binding.shareFileIcon.setImageDrawable(drawable);
77+
// NMC Customization: No overlay icon will be used. Directly using folder icons
78+
binding.shareFileIcon.setImageDrawable(ContextCompat.getDrawable(this, overlayIconId));
7879
} else {
7980
binding.shareFileIcon.setImageDrawable(MimeTypeUtil.getFileTypeIcon(file.getMimeType(),
8081
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
@@ -13,6 +13,7 @@ import android.view.LayoutInflater
1313
import android.view.View
1414
import android.view.ViewGroup
1515
import android.widget.ImageView
16+
import androidx.core.content.ContextCompat
1617
import androidx.recyclerview.widget.RecyclerView
1718
import com.nextcloud.client.account.User
1819
import com.owncloud.android.databinding.UploaderListItemLayoutBinding
@@ -113,10 +114,9 @@ class ReceiveExternalFilesAdapter(
113114

114115
private fun setupThumbnailForFolder(thumbnailImageView: ImageView, file: OCFile) {
115116
val isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user)
116-
val isDarkModeActive = syncedFolderProvider.preferences.isDarkModeEnabled
117117
val overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder)
118-
val icon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, context, viewThemeUtils)
119-
thumbnailImageView.setImageDrawable(icon)
118+
// NMC Customization: No overlay icon will be used. Directly using folder icons
119+
thumbnailImageView.setImageDrawable(ContextCompat.getDrawable(context, overlayIconId))
120120
}
121121

122122
@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
@@ -32,7 +32,6 @@
3232
import android.graphics.Point;
3333
import android.graphics.drawable.ColorDrawable;
3434
import android.graphics.drawable.Drawable;
35-
import android.graphics.drawable.LayerDrawable;
3635
import android.net.Uri;
3736
import android.os.AsyncTask;
3837
import android.text.Spannable;
@@ -105,6 +104,7 @@
105104
import androidx.annotation.Nullable;
106105
import androidx.annotation.StringRes;
107106
import androidx.appcompat.widget.AppCompatDrawableManager;
107+
import androidx.core.content.ContextCompat;
108108
import androidx.core.content.res.ResourcesCompat;
109109
import androidx.fragment.app.FragmentManager;
110110
import androidx.fragment.app.FragmentTransaction;
@@ -849,11 +849,9 @@ public static void setThumbnail(OCFile file,
849849
stopShimmer(shimmerThumbnail, thumbnailView);
850850

851851
boolean isAutoUploadFolder = SyncedFolderProvider.isAutoUploadFolder(syncedFolderProvider, file, user);
852-
boolean isDarkModeActive = preferences.isDarkModeEnabled();
853-
854852
Integer overlayIconId = file.getFileOverlayIconId(isAutoUploadFolder);
855-
LayerDrawable fileIcon = MimeTypeUtil.getFileIcon(isDarkModeActive, overlayIconId, context, viewThemeUtils);
856-
thumbnailView.setImageDrawable(fileIcon);
853+
// NMC Customization: No overlay icon will be used. Directly using folder icons
854+
thumbnailView.setImageDrawable(ContextCompat.getDrawable(context, overlayIconId));
857855
} else {
858856
if (file.getRemoteId() != null && file.isPreviewAvailable()) {
859857
// 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
@@ -94,13 +94,8 @@ public static Drawable getFileTypeIcon(String mimetype,
9494
ViewThemeUtils viewThemeUtils) {
9595
if (context != null) {
9696
int iconId = MimeTypeUtil.getFileTypeIconId(mimetype, filename);
97-
Drawable icon = ContextCompat.getDrawable(context, iconId);
98-
99-
if (R.drawable.file_zip == iconId) {
100-
viewThemeUtils.platform.tintPrimaryDrawable(context, icon);
101-
}
102-
103-
return icon;
97+
//NMC Customization
98+
return ContextCompat.getDrawable(context, iconId);
10499
} else {
105100
return null;
106101
}
@@ -128,11 +123,13 @@ public static Drawable getDefaultFolderIcon(Context context, ViewThemeUtils view
128123
Drawable drawable = ContextCompat.getDrawable(context, R.drawable.folder);
129124
assert(drawable != null);
130125

131-
viewThemeUtils.platform.tintDrawable(context, drawable, ColorRole.PRIMARY);
132126
return drawable;
133127
}
134128

135-
public static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
129+
// 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.
130+
// 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
131+
// so that UI won't be impacted.
132+
private static LayerDrawable getFileIcon(Boolean isDarkModeActive, Integer overlayIconId, Context context, ViewThemeUtils viewThemeUtils) {
136133
Drawable folderDrawable = getDefaultFolderIcon(context, viewThemeUtils);
137134
assert(folderDrawable != null);
138135

0 commit comments

Comments
 (0)