fix list

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>

Author: alperozturk96

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

@@ -549,8 +549,10 @@ private void setFilenameAndExtension(ListGridItemViewHolder holder, OCFile file)
 
         if (holder instanceof OCFileListGridItemViewHolder gridItemViewHolder) {
             handleGridMode(filename, gridItemViewHolder, pair, file);
+        } else if (holder instanceof OCFileListItemViewHolder itemViewHolder) {
+            handleListMode(itemViewHolder, filename, pair);
         } else {
-            handleListMode(holder, filename, pair, isFolder);
+            handleRecommendedMode(holder, pair, isFolder);
         }
     }
 
@@ -575,23 +577,34 @@ private void handleGridMode(String filename, OCFileListGridItemViewHolder holder
         }
     }
 
-    private void handleListMode(ListGridItemViewHolder holder,
+    private void handleListMode(OCFileListItemViewHolder holder,
                                 String filename,
-                                Pair<String, String> filenamePair,
-                                boolean isFolder) {
-        final var extension = holder.getExtension();
-        final boolean useSeparateExtension = !isFolder && FileStorageUtils.containsBidiControlCharacters(filename);
+                                Pair<String, String> filenamePair) {
+        final boolean containsBidiControlCharacters = FileStorageUtils.containsBidiControlCharacters(filename);
+        ViewExtensionsKt.setVisibleIf(holder.getFileName(), !containsBidiControlCharacters);
+        ViewExtensionsKt.setVisibleIf(holder.getBidiFilenameContainer(), containsBidiControlCharacters);
 
-        if (!useSeparateExtension) {
+        if (containsBidiControlCharacters) {
+            holder.getBidiFilename().setText(filenamePair.getFirst());
+            holder.getExtension().setText(filenamePair.getSecond());
+        } else {
             holder.getFileName().setText(filename);
-            if (extension != null) {
-                extension.setVisibility(View.GONE);
-            }
-            return;
         }
+    }
 
+    private void handleRecommendedMode(ListGridItemViewHolder holder,
+                                       Pair<String, String> filenamePair,
+                                       boolean isFolder) {
         holder.getFileName().setText(filenamePair.getFirst());
-        if (extension != null) {
+
+        final var extension = holder.getExtension();
+        if (extension == null) {
+            return;
+        }
+
+        if (isFolder) {
+            extension.setVisibility(View.GONE);
+        } else {
             extension.setVisibility(View.VISIBLE);
             extension.setText(filenamePair.getSecond());
         }

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

@@ -47,6 +47,10 @@ class OCFileListItemViewHolder(private var binding: ListItemBinding) :
         get() = binding.Filename
     override val extension: TextView
         get() = binding.extension
+    val bidiFilename: TextView
+        get() = binding.bidiFilename
+    val bidiFilenameContainer: LinearLayout
+        get() = binding.bidiFilenameContainer
     override val thumbnail: ImageView
         get() = binding.thumbnailLayout.thumbnail
     override val tagsGroup: ChipGroup

app/src/main/res/layout/list_item.xml

@@ -77,13 +77,26 @@
         android:orientation="vertical"
         android:paddingTop="@dimen/standard_half_padding">
 
+        <TextView
+            android:id="@+id/Filename"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:ellipsize="middle"
+            android:singleLine="true"
+            android:textColor="@color/text_color"
+            android:textSize="@dimen/two_line_primary_text_size"
+            tools:text="@string/placeholder_filename" />
+
         <LinearLayout
-            android:layout_width="wrap_content"
+            android:id="@+id/bidi_filename_container"
+            android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:orientation="horizontal">
+            android:orientation="horizontal"
+            android:visibility="gone"
+            tools:visibility="visible">
 
             <TextView
-                android:id="@+id/Filename"
+                android:id="@+id/bidi_filename"
                 android:layout_width="0dp"
                 android:layout_height="wrap_content"
                 android:layout_weight="1"
@@ -97,7 +110,6 @@
                 android:id="@+id/extension"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:ellipsize="middle"
                 android:singleLine="true"
                 android:textColor="@color/text_color"
                 android:textSize="@dimen/two_line_primary_text_size"