瀏覽代碼

FileDetailSharingFragment: remove misleading "Open in..." option

This only triggered the share file dialog, which should be opened elsewhere.

Signed-off-by: Álvaro Brey Vilas <alvaro.brey@nextcloud.com>
Álvaro Brey Vilas 3 年之前
父節點
當前提交
038ac7b3c5

+ 0 - 5
src/main/java/com/owncloud/android/ui/fragment/FileDetailSharingFragment.java

@@ -462,11 +462,6 @@ public class FileDetailSharingFragment extends Fragment implements ShareeListAda
         return file;
     }
 
-    @Override
-    public void openIn(OCShare share) {
-        fileOperationsHelper.sendShareFile(file);
-    }
-
     @Override
     public void advancedPermissions(OCShare share) {
         modifyExistingShare(share, FileDetailsSharingProcessFragment.SCREEN_TYPE_PERMISSION);

+ 0 - 11
src/main/java/com/owncloud/android/ui/fragment/FileDetailSharingMenuBottomSheetDialog.java

@@ -69,12 +69,6 @@ public class FileDetailSharingMenuBottomSheetDialog extends BottomSheetDialog {
     }
 
     private void updateUI() {
-        if (ocShare.isFolder()) {
-            binding.menuShareOpenIn.setVisibility(View.GONE);
-        } else {
-            binding.menuShareOpenIn.setVisibility(View.VISIBLE);
-        }
-
         if (ocShare.getShareType() == ShareType.PUBLIC_LINK) {
             binding.menuShareAddAnotherLink.setVisibility(View.VISIBLE);
             binding.menuShareSendLink.setVisibility(View.VISIBLE);
@@ -85,11 +79,6 @@ public class FileDetailSharingMenuBottomSheetDialog extends BottomSheetDialog {
     }
 
     private void setupClickListener() {
-        binding.menuShareOpenIn.setOnClickListener(v -> {
-            actions.openIn(ocShare);
-            dismiss();
-        });
-
         binding.menuShareAdvancedPermissions.setOnClickListener(v -> {
             actions.advancedPermissions(ocShare);
             dismiss();

+ 0 - 4
src/main/java/com/owncloud/android/ui/fragment/FileDetailsSharingMenuBottomSheetActions.java

@@ -29,10 +29,6 @@ import com.owncloud.android.lib.resources.shares.ShareType;
  * Actions interface to be implemented by any class that makes use of {@link FileDetailSharingMenuBottomSheetDialog}.
  */
 public interface FileDetailsSharingMenuBottomSheetActions {
-    /**
-     * open sharing options only applicable for files
-     */
-    void openIn(OCShare share);
 
     /**
      * open advanced permission for selected share

+ 1 - 35
src/main/res/layout/file_details_sharing_menu_bottom_sheet_fragment.xml

@@ -25,41 +25,7 @@
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
     android:orientation="vertical"
-    android:background="@color/bg_default"
-    android:paddingTop="@dimen/standard_padding">
-
-    <LinearLayout
-        android:id="@+id/menu_share_open_in"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:minHeight="@dimen/minimum_size_for_touchable_area"
-        android:background="?android:attr/selectableItemBackground"
-        android:orientation="horizontal"
-        android:paddingLeft="@dimen/standard_padding"
-        android:paddingTop="@dimen/standard_half_padding"
-        android:paddingRight="@dimen/standard_padding"
-        android:paddingBottom="@dimen/standard_half_padding"
-        tools:ignore="UseCompoundDrawables">
-
-        <ImageView
-            android:id="@+id/menu_icon_open_in"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_vertical"
-            android:contentDescription="@null"
-            android:src="@drawable/ic_content_copy"
-            app:tint="@color/primary" />
-
-        <TextView
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_vertical"
-            android:layout_marginStart="@dimen/standard_margin"
-            android:text="@string/share_open_in"
-            android:textColor="@color/text_color"
-            android:textSize="@dimen/bottom_sheet_text_size" />
-
-    </LinearLayout>
+    android:background="@color/bg_default">
 
     <LinearLayout
         android:id="@+id/menu_share_advanced_permissions"