Skip to content

Commit 84b4d03

Browse files
committed
Fix -> video click event, overflow menu item hide and theming add menu
Signed-off-by: Surinder Kumar <[email protected]>
1 parent 3ebefc8 commit 84b4d03

File tree

8 files changed

+36
-10
lines changed

8 files changed

+36
-10
lines changed

app/src/main/java/com/owncloud/android/ui/activity/AlbumsPickerActivity.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/*
22
* Nextcloud - Android Client
33
*
4-
* SPDX-FileCopyrightText: 2025 Your Name <your@email.com>
4+
* SPDX-FileCopyrightText: 2025 TSI-mc <surinder.kumar@t-systems.com>
55
* SPDX-License-Identifier: AGPL-3.0-or-later
66
*/
77

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -986,7 +986,7 @@ private void requestUploadOfFilesFromFileSystem(String localBasePath, String[] f
986986

987987
connectivityService.isNetworkAndServerAvailable(result -> {
988988
if (result) {
989-
boolean isValidFolderPath = FileNameValidator.INSTANCE.checkFolderPath(remotePathBase, getCapabilities(), this);
989+
boolean isValidFolderPath = FileNameValidator.INSTANCE.checkFolderPath(remotePathBase,getCapabilities(),this);
990990
if (!isValidFolderPath) {
991991
DisplayUtils.showSnackMessage(this, R.string.file_name_validator_error_contains_reserved_names_or_invalid_characters);
992992
return;
@@ -1231,7 +1231,6 @@ protected void onResume() {
12311231

12321232
Log_OC.v(TAG, "onResume() end");
12331233
}
1234-
12351234
private void setDrawerAllFiles() {
12361235
if (MainApp.isOnlyPersonFiles()) {
12371236
menuItemId = R.id.nav_personal_files;
@@ -1725,7 +1724,7 @@ private void observeWorkerState() {
17251724
previewFile(finishedState);
17261725
} else if (state instanceof WorkerState.UploadFinished) {
17271726
refreshList();
1728-
} else if (state instanceof WorkerState.OfflineOperationsCompleted) {
1727+
} else if (state instanceof WorkerState.OfflineOperationsCompleted) {
17291728
refreshCurrentDirectory();
17301729
}
17311730
});

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/*
22
* Nextcloud - Android Client
33
*
4-
* SPDX-FileCopyrightText: 2025 Your Name <your@email.com>
4+
* SPDX-FileCopyrightText: 2025 TSI-mc <surinder.kumar@t-systems.com>
55
* SPDX-License-Identifier: AGPL-3.0-or-later
66
*/
77

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/*
22
* Nextcloud - Android Client
33
*
4-
* SPDX-FileCopyrightText: 2025 Your Name <your@email.com>
4+
* SPDX-FileCopyrightText: 2025 TSI-mc <surinder.kumar@t-systems.com>
55
* SPDX-License-Identifier: AGPL-3.0-or-later
66
*/
77

app/src/main/java/com/owncloud/android/ui/fragment/ExtendedListFragment.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
/*
22
* Nextcloud - Android Client
33
*
4+
* SPDX-FileCopyrightText: 2025 TSI-mc <[email protected]>
45
* SPDX-FileCopyrightText: 2025 Alper Ozturk <[email protected]>
56
* SPDX-FileCopyrightText: 2022 Álvaro Brey <[email protected]>
67
* SPDX-FileCopyrightText: 2018-2021 Tobias Kaminsky <[email protected]>

app/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@
8080
import com.owncloud.android.lib.resources.files.ToggleFavoriteRemoteOperation;
8181
import com.owncloud.android.lib.resources.status.E2EVersion;
8282
import com.owncloud.android.lib.resources.status.OCCapability;
83+
import com.owncloud.android.ui.activity.AlbumsPickerActivity;
8384
import com.owncloud.android.ui.activity.DrawerActivity;
8485
import com.owncloud.android.ui.activity.FileActivity;
8586
import com.owncloud.android.ui.activity.FileDisplayActivity;
@@ -849,7 +850,13 @@ public boolean onCreateActionMode(ActionMode mode, Menu menu) {
849850

850851
if (OCFileListFragment.this instanceof GalleryFragment) {
851852
final MenuItem addAlbumItem = menu.findItem(R.id.add_to_album);
853+
// show add to album button for gallery to add media to Album
852854
addAlbumItem.setVisible(true);
855+
856+
// hide the 3 dot menu icon while picking media for Albums
857+
if (requireActivity() instanceof AlbumsPickerActivity) {
858+
item.setVisible(false);
859+
}
853860
}
854861

855862
getCommonAdapter().setMultiSelect(true);

app/src/main/java/com/owncloud/android/ui/fragment/albums/AlbumItemsFragment.kt

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,16 +61,16 @@ import com.owncloud.android.lib.resources.albums.ReadAlbumItemsRemoteOperation
6161
import com.owncloud.android.lib.resources.albums.RemoveAlbumFileRemoteOperation
6262
import com.owncloud.android.lib.resources.albums.ToggleAlbumFavoriteRemoteOperation
6363
import com.owncloud.android.ui.activity.AlbumsPickerActivity
64+
import com.owncloud.android.ui.activity.AlbumsPickerActivity.Companion.intentForPickingMediaFiles
6465
import com.owncloud.android.ui.activity.FileActivity
6566
import com.owncloud.android.ui.activity.FileDisplayActivity
6667
import com.owncloud.android.ui.adapter.GalleryAdapter
6768
import com.owncloud.android.ui.dialog.CreateAlbumDialogFragment
6869
import com.owncloud.android.ui.events.FavoriteEvent
6970
import com.owncloud.android.ui.fragment.FileFragment
70-
import com.owncloud.android.ui.activity.AlbumsPickerActivity.Companion.intentForPickingMediaFiles
7171
import com.owncloud.android.ui.interfaces.OCFileListFragmentInterface
7272
import com.owncloud.android.ui.preview.PreviewImageFragment
73-
import com.owncloud.android.ui.preview.PreviewMediaActivity
73+
import com.owncloud.android.ui.preview.PreviewMediaActivity.Companion.canBePreviewed
7474
import com.owncloud.android.utils.DisplayUtils
7575
import com.owncloud.android.utils.ErrorMessageAdapter
7676
import com.owncloud.android.utils.FileStorageUtils
@@ -484,13 +484,17 @@ class AlbumItemsFragment : Fragment(), OCFileListFragmentInterface, Injectable {
484484
if (PreviewImageFragment.canBePreviewed(file)) {
485485
(mContainerActivity as FileDisplayActivity).startImagePreview(file, !file.isDown)
486486
} else if (file.isDown) {
487-
if (PreviewMediaActivity.canBePreviewed(file)) {
487+
if (canBePreviewed(file)) {
488488
(mContainerActivity as FileDisplayActivity).startMediaPreview(file, 0, true, true, false, true)
489489
} else {
490490
mContainerActivity?.getFileOperationsHelper()?.openFile(file)
491491
}
492492
} else {
493-
Log_OC.d(TAG, "Couldn't handle item click")
493+
if (canBePreviewed(file) && !file.isEncrypted) {
494+
(mContainerActivity as FileDisplayActivity).startMediaPreview(file, 0, true, true, true, true)
495+
} else {
496+
Log_OC.d(TAG, "Couldn't handle item click")
497+
}
494498
}
495499
}
496500
}

app/src/main/java/com/owncloud/android/ui/fragment/albums/AlbumsFragment.kt

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,17 @@ import android.content.Context
1212
import android.content.Intent
1313
import android.content.res.Configuration
1414
import android.os.Bundle
15+
import android.text.Spannable
16+
import android.text.SpannableString
17+
import android.text.style.ForegroundColorSpan
1518
import android.view.LayoutInflater
1619
import android.view.Menu
1720
import android.view.MenuInflater
1821
import android.view.MenuItem
1922
import android.view.View
2023
import android.view.ViewGroup
2124
import androidx.annotation.VisibleForTesting
25+
import androidx.core.content.ContextCompat
2226
import androidx.core.content.res.ResourcesCompat
2327
import androidx.core.view.MenuHost
2428
import androidx.core.view.MenuProvider
@@ -137,6 +141,17 @@ class AlbumsFragment : Fragment(), AlbumFragmentInterface, Injectable {
137141
override fun onCreateMenu(menu: Menu, menuInflater: MenuInflater) {
138142
menu.clear() // important: clears any existing activity menu
139143
menuInflater.inflate(R.menu.fragment_create_album, menu)
144+
145+
val addItem = menu.findItem(R.id.action_create_new_album)
146+
val coloredTitle = SpannableString(addItem.title).apply {
147+
setSpan(
148+
ForegroundColorSpan(ContextCompat.getColor(requireContext(), R.color.primary)),
149+
0,
150+
length,
151+
Spannable.SPAN_INCLUSIVE_INCLUSIVE
152+
)
153+
}
154+
addItem.title = coloredTitle
140155
}
141156

142157
override fun onMenuItemSelected(menuItem: MenuItem): Boolean {

0 commit comments

Comments
 (0)