From a9445dfcca8d7770cea5272bb26a8fe30405b4ce Mon Sep 17 00:00:00 2001
From: Aman Harwara <amanharwara@protonmail.com>
Date: Sat, 24 Dec 2022 21:56:38 +0530
Subject: [PATCH] refactor(dev-only): file list item style

---
 .../ContentListView/ContentList.tsx           |   7 +-
 .../ContentListView/ContentListItem.tsx       |  13 +-
 .../ContentListView/FileListItem.tsx          |  62 +++------
 .../ContentListView/FileListItemCard.tsx      | 131 ++++++++++++++++++
 .../ContentListView/ListItemFlagIcons.tsx     |  14 +-
 5 files changed, 181 insertions(+), 46 deletions(-)
 create mode 100644 packages/web/src/javascripts/Components/ContentListView/FileListItemCard.tsx

diff --git a/packages/web/src/javascripts/Components/ContentListView/ContentList.tsx b/packages/web/src/javascripts/Components/ContentListView/ContentList.tsx
index 323d8606e..1b0773242 100644
--- a/packages/web/src/javascripts/Components/ContentListView/ContentList.tsx
+++ b/packages/web/src/javascripts/Components/ContentListView/ContentList.tsx
@@ -13,6 +13,7 @@ import { NotesController } from '@/Controllers/NotesController/NotesController'
 import { ElementIds } from '@/Constants/ElementIDs'
 import { classNames } from '@standardnotes/utils'
 import { ContentType, SNTag } from '@standardnotes/snjs'
+import { featureTrunkEnabled, FeatureTrunkName } from '@/FeatureTrunk'
 
 type Props = {
   application: WebApplication
@@ -97,6 +98,8 @@ const ContentList: FunctionComponent<Props> = ({
 
   const hasNotes = items.some((item) => item.content_type === ContentType.Note)
 
+  const isFilesTableViewEnabled = featureTrunkEnabled(FeatureTrunkName.FilesTableView)
+
   return (
     <div
       className={classNames(
@@ -132,8 +135,8 @@ const ContentList: FunctionComponent<Props> = ({
             onSelect={selectItem}
             tags={getTagsForItem(item)}
             notesController={notesController}
-            isPreviousItemTiled={previousItem?.content_type === ContentType.File}
-            isNextItemTiled={nextItem?.content_type === ContentType.File}
+            isPreviousItemTiled={previousItem?.content_type === ContentType.File && !isFilesTableViewEnabled}
+            isNextItemTiled={nextItem?.content_type === ContentType.File && !isFilesTableViewEnabled}
           />
         )
       })}
diff --git a/packages/web/src/javascripts/Components/ContentListView/ContentListItem.tsx b/packages/web/src/javascripts/Components/ContentListView/ContentListItem.tsx
index 4221f67df..93ba67396 100644
--- a/packages/web/src/javascripts/Components/ContentListView/ContentListItem.tsx
+++ b/packages/web/src/javascripts/Components/ContentListView/ContentListItem.tsx
@@ -1,16 +1,25 @@
+import { featureTrunkEnabled, FeatureTrunkName } from '@/FeatureTrunk'
 import { ContentType, FileItem, SNNote } from '@standardnotes/snjs'
 import React, { FunctionComponent } from 'react'
 import FileListItem from './FileListItem'
+import FileListItemCard from './FileListItemCard'
 import NoteListItem from './NoteListItem'
 import { AbstractListItemProps, doListItemPropsMeritRerender } from './Types/AbstractListItemProps'
 import { ListableContentItem } from './Types/ListableContentItem'
 
 const ContentListItem: FunctionComponent<AbstractListItemProps<ListableContentItem>> = (props) => {
+  const isFilesTableViewEnabled = featureTrunkEnabled(FeatureTrunkName.FilesTableView)
+
   switch (props.item.content_type) {
     case ContentType.Note:
       return <NoteListItem {...props} item={props.item as SNNote} />
-    case ContentType.File:
-      return <FileListItem {...props} item={props.item as FileItem} />
+    case ContentType.File: {
+      if (isFilesTableViewEnabled) {
+        return <FileListItem {...props} item={props.item as FileItem} />
+      }
+
+      return <FileListItemCard {...props} item={props.item as FileItem} />
+    }
     default:
       return null
   }
diff --git a/packages/web/src/javascripts/Components/ContentListView/FileListItem.tsx b/packages/web/src/javascripts/Components/ContentListView/FileListItem.tsx
index 1b25b33ec..f02993e75 100644
--- a/packages/web/src/javascripts/Components/ContentListView/FileListItem.tsx
+++ b/packages/web/src/javascripts/Components/ContentListView/FileListItem.tsx
@@ -9,13 +9,12 @@ import { DisplayableListItemProps } from './Types/DisplayableListItemProps'
 import { useResponsiveAppPane } from '../Panes/ResponsivePaneProvider'
 import { useContextMenuEvent } from '@/Hooks/useContextMenuEvent'
 import { classNames } from '@standardnotes/utils'
-import { formatSizeToReadableString } from '@standardnotes/filepicker'
 import { getIconForFileType } from '@/Utils/Items/Icons/getIconForFileType'
 import { useApplication } from '../ApplicationProvider'
-import Icon from '../Icon/Icon'
 import { PaneLayout } from '@/Controllers/PaneController/PaneLayout'
+import ListItemFlagIcons from './ListItemFlagIcons'
 
-const FileListItem: FunctionComponent<DisplayableListItemProps<FileItem>> = ({
+const FileListItemCard: FunctionComponent<DisplayableListItemProps<FileItem>> = ({
   filesController,
   hideDate,
   hideIcon,
@@ -75,57 +74,38 @@ const FileListItem: FunctionComponent<DisplayableListItemProps<FileItem>> = ({
   }, [file, onSelect, setPaneLayout])
 
   const IconComponent = () =>
-    getFileIconComponent(getIconForFileType((file as FileItem).mimeType), 'w-10 h-10 flex-shrink-0')
+    getFileIconComponent(getIconForFileType((file as FileItem).mimeType), 'w-5 h-5 flex-shrink-0')
 
   useContextMenuEvent(listItemRef, openContextMenu)
 
   return (
     <div
       ref={listItemRef}
-      className={classNames('flex max-h-[300px] w-[190px] cursor-pointer px-1 pt-2 text-text md:w-[200px]')}
+      className={classNames(
+        'content-list-item flex w-full cursor-pointer items-stretch text-text',
+        selected && 'selected border-l-2px border-solid border-info',
+      )}
       id={file.uuid}
       onClick={onClick}
     >
-      <div
-        className={`flex flex-col justify-between overflow-hidden rounded bg-passive-5 pt-5 transition-all hover:bg-passive-4 ${
-          selected ? 'border-[1px] border-solid border-info' : 'border-[1px] border-solid border-border'
-        }`}
-      >
-        <div className={'px-5'}>
-          {!hideIcon ? (
-            <div className="mr-0">
-              <IconComponent />
-            </div>
-          ) : (
-            <div className="pr-4" />
-          )}
-          <div className="min-w-0 flex-grow py-4 px-0">
-            <div className="line-clamp-2 overflow-hidden text-editor font-semibold">
-              <div className="break-word line-clamp-2 mr-2 overflow-hidden">{file.title}</div>
-            </div>
-            <ListItemMetadata item={file} hideDate={hideDate} sortBy={sortBy} />
-            <ListItemTags hideTags={hideTags} tags={tags} />
-            <ListItemConflictIndicator item={file} />
-          </div>
+      {!hideIcon ? (
+        <div className="mr-0 flex flex-col items-center justify-between p-4.5 pr-3">
+          <IconComponent />
         </div>
-        <div
-          className={classNames(
-            'border-t-[1px] border-solid border-border p-3 text-xs font-bold',
-            selected ? 'bg-info text-info-contrast' : 'bg-passive-4 text-neutral',
-          )}
-        >
-          <div className="flex justify-between">
-            {formatSizeToReadableString(file.decryptedSize)}
-            {backupInfo && (
-              <div title="File is backed up locally">
-                <Icon type="check-circle" />
-              </div>
-            )}
-          </div>
+      ) : (
+        <div className="pr-4" />
+      )}
+      <div className="min-w-0 flex-grow border-b border-solid border-border py-4 px-0">
+        <div className="flex items-start justify-between overflow-hidden text-base font-semibold leading-[1.3]">
+          <div className="break-word mr-2">{file.title}</div>
         </div>
+        <ListItemMetadata item={file} hideDate={hideDate} sortBy={sortBy} />
+        <ListItemTags hideTags={hideTags} tags={tags} />
+        <ListItemConflictIndicator item={file} />
       </div>
+      <ListItemFlagIcons className="p-4" item={file} isFileBackedUp={!!backupInfo} />
     </div>
   )
 }
 
-export default observer(FileListItem)
+export default observer(FileListItemCard)
diff --git a/packages/web/src/javascripts/Components/ContentListView/FileListItemCard.tsx b/packages/web/src/javascripts/Components/ContentListView/FileListItemCard.tsx
new file mode 100644
index 000000000..c80bdcc0a
--- /dev/null
+++ b/packages/web/src/javascripts/Components/ContentListView/FileListItemCard.tsx
@@ -0,0 +1,131 @@
+import { FileItem, FileBackupRecord } from '@standardnotes/snjs'
+import { observer } from 'mobx-react-lite'
+import { FunctionComponent, useCallback, useEffect, useRef, useState } from 'react'
+import { getFileIconComponent } from '../FilePreview/getFileIconComponent'
+import ListItemConflictIndicator from './ListItemConflictIndicator'
+import ListItemTags from './ListItemTags'
+import ListItemMetadata from './ListItemMetadata'
+import { DisplayableListItemProps } from './Types/DisplayableListItemProps'
+import { useResponsiveAppPane } from '../Panes/ResponsivePaneProvider'
+import { useContextMenuEvent } from '@/Hooks/useContextMenuEvent'
+import { classNames } from '@standardnotes/utils'
+import { formatSizeToReadableString } from '@standardnotes/filepicker'
+import { getIconForFileType } from '@/Utils/Items/Icons/getIconForFileType'
+import { useApplication } from '../ApplicationProvider'
+import Icon from '../Icon/Icon'
+import { PaneLayout } from '@/Controllers/PaneController/PaneLayout'
+
+const FileListItemCard: FunctionComponent<DisplayableListItemProps<FileItem>> = ({
+  filesController,
+  hideDate,
+  hideIcon,
+  hideTags,
+  item: file,
+  onSelect,
+  selected,
+  sortBy,
+  tags,
+}) => {
+  const { setPaneLayout } = useResponsiveAppPane()
+  const application = useApplication()
+
+  const [backupInfo, setBackupInfo] = useState<FileBackupRecord | undefined>(undefined)
+
+  useEffect(() => {
+    void application.fileBackups?.getFileBackupInfo(file).then(setBackupInfo)
+  }, [application, file])
+
+  const listItemRef = useRef<HTMLDivElement>(null)
+
+  const openFileContextMenu = useCallback(
+    (posX: number, posY: number) => {
+      filesController.setShowFileContextMenu(false)
+      filesController.setFileContextMenuLocation({
+        x: posX,
+        y: posY,
+      })
+      filesController.setShowFileContextMenu(true)
+    },
+    [filesController],
+  )
+
+  const openContextMenu = useCallback(
+    async (posX: number, posY: number) => {
+      let shouldOpenContextMenu = selected
+
+      if (!selected) {
+        const { didSelect } = await onSelect(file)
+        if (didSelect) {
+          shouldOpenContextMenu = true
+        }
+      }
+
+      if (shouldOpenContextMenu) {
+        openFileContextMenu(posX, posY)
+      }
+    },
+    [selected, onSelect, file, openFileContextMenu],
+  )
+
+  const onClick = useCallback(async () => {
+    const { didSelect } = await onSelect(file, true)
+    if (didSelect) {
+      setPaneLayout(PaneLayout.Editing)
+    }
+  }, [file, onSelect, setPaneLayout])
+
+  const IconComponent = () =>
+    getFileIconComponent(getIconForFileType((file as FileItem).mimeType), 'w-10 h-10 flex-shrink-0')
+
+  useContextMenuEvent(listItemRef, openContextMenu)
+
+  return (
+    <div
+      ref={listItemRef}
+      className={classNames('flex max-h-[300px] w-[190px] cursor-pointer px-1 pt-2 text-text md:w-[200px]')}
+      id={file.uuid}
+      onClick={onClick}
+    >
+      <div
+        className={`flex flex-col justify-between overflow-hidden rounded bg-passive-5 pt-5 transition-all hover:bg-passive-4 ${
+          selected ? 'border-[1px] border-solid border-info' : 'border-[1px] border-solid border-border'
+        }`}
+      >
+        <div className={'px-5'}>
+          {!hideIcon ? (
+            <div className="mr-0">
+              <IconComponent />
+            </div>
+          ) : (
+            <div className="pr-4" />
+          )}
+          <div className="min-w-0 flex-grow py-4 px-0">
+            <div className="line-clamp-2 overflow-hidden text-editor font-semibold">
+              <div className="break-word line-clamp-2 mr-2 overflow-hidden">{file.title}</div>
+            </div>
+            <ListItemMetadata item={file} hideDate={hideDate} sortBy={sortBy} />
+            <ListItemTags hideTags={hideTags} tags={tags} />
+            <ListItemConflictIndicator item={file} />
+          </div>
+        </div>
+        <div
+          className={classNames(
+            'border-t-[1px] border-solid border-border p-3 text-xs font-bold',
+            selected ? 'bg-info text-info-contrast' : 'bg-passive-4 text-neutral',
+          )}
+        >
+          <div className="flex justify-between">
+            {formatSizeToReadableString(file.decryptedSize)}
+            {backupInfo && (
+              <div title="File is backed up locally">
+                <Icon type="check-circle" />
+              </div>
+            )}
+          </div>
+        </div>
+      </div>
+    </div>
+  )
+}
+
+export default observer(FileListItemCard)
diff --git a/packages/web/src/javascripts/Components/ContentListView/ListItemFlagIcons.tsx b/packages/web/src/javascripts/Components/ContentListView/ListItemFlagIcons.tsx
index 63beb562a..d8ba436ee 100644
--- a/packages/web/src/javascripts/Components/ContentListView/ListItemFlagIcons.tsx
+++ b/packages/web/src/javascripts/Components/ContentListView/ListItemFlagIcons.tsx
@@ -12,10 +12,17 @@ type Props = {
   }
   hasFiles?: boolean
   hasBorder?: boolean
+  isFileBackedUp?: boolean
   className?: string
 }
 
-const ListItemFlagIcons: FunctionComponent<Props> = ({ item, hasFiles = false, hasBorder = true, className }) => {
+const ListItemFlagIcons: FunctionComponent<Props> = ({
+  item,
+  hasFiles = false,
+  hasBorder = true,
+  isFileBackedUp = false,
+  className,
+}) => {
   return (
     <div className={`flex items-start ${hasBorder && 'border-b border-solid border-border'} ${className} pl-0`}>
       {item.locked && (
@@ -48,6 +55,11 @@ const ListItemFlagIcons: FunctionComponent<Props> = ({ item, hasFiles = false, h
           <Icon ariaLabel="Starred" type="star-filled" className="text-warning" size="medium" />
         </span>
       )}
+      {isFileBackedUp && (
+        <span className="ml-1.5 flex items-center" title="File is backed up locally">
+          <Icon ariaLabel="File is backed up locally" type="check-circle" className="text-info" size="medium" />
+        </span>
+      )}
     </div>
   )
 }