From 12f25c49a7c2e8af3b2d0e957f3aa841d6efbff4 Mon Sep 17 00:00:00 2001
From: Max Leiter <maxwell.leiter@gmail.com>
Date: Thu, 24 Mar 2022 19:25:02 -0700
Subject: [PATCH] client: misc style improvements

---
 client/components/header/header.tsx           |  2 +-
 .../new-post/drag-and-drop/index.tsx          | 11 +++++-
 client/components/post-list/list-item.tsx     | 38 +++++++++++--------
 .../components/post-list/post-list.module.css |  1 -
 client/components/visibility-badge/index.tsx  |  2 +-
 client/pages/mine.tsx                         |  2 +-
 6 files changed, 34 insertions(+), 22 deletions(-)

diff --git a/client/components/header/header.tsx b/client/components/header/header.tsx
index 5608906..b83f69b 100644
--- a/client/components/header/header.tsx
+++ b/client/components/header/header.tsx
@@ -129,7 +129,7 @@ const Header = () => {
 
 
     return (
-        <Page.Header height={'var(--page-nav-height)'} margin={0} paddingBottom={'var(--gap-double)'} paddingTop={"var(--gap)"}>
+        <Page.Header height={'var(--page-nav-height)'} marginBottom={2}>
             <div className={styles.tabs}>
                 <Tabs
                     value={selectedTab}
diff --git a/client/components/new-post/drag-and-drop/index.tsx b/client/components/new-post/drag-and-drop/index.tsx
index 611e8dc..a20ca0e 100644
--- a/client/components/new-post/drag-and-drop/index.tsx
+++ b/client/components/new-post/drag-and-drop/index.tsx
@@ -35,8 +35,6 @@ const allowedFileTypes = [
     'text/markdown',
 ]
 
-// Files with no extension can't be easily detected as plain-text,
-// so instead of allowing all of them we'll just allow common ones
 const allowedFileNames = [
     'Makefile',
     'README',
@@ -53,6 +51,15 @@ const allowedFileNames = [
     '.env.staging',
     '.env.development.local',
     'yarn.lock',
+    '.bash',
+    '.bashrc',
+    '.bash_profile',
+    '.bash_logout',
+    '.profile',
+    '.fish_prompt',
+    '.zshrc',
+    '.zsh',
+    '.zprofile',
 ]
 
 const allowedFileExtensions = [
diff --git a/client/components/post-list/list-item.tsx b/client/components/post-list/list-item.tsx
index f9f3f40..a8d6b26 100644
--- a/client/components/post-list/list-item.tsx
+++ b/client/components/post-list/list-item.tsx
@@ -2,10 +2,9 @@
 import NextLink from "next/link"
 import { useEffect, useMemo, useState } from "react"
 import timeAgo from "@lib/time-ago"
-import ShiftBy from "../shift-by"
 import VisibilityBadge from "../visibility-badge"
 import getPostPath from "@lib/get-post-path"
-import { Input, Link, Text, Card, Spacer, Grid, Tooltip, Divider } from "@geist-ui/core"
+import { Input, Link, Text, Card, Spacer, Grid, Tooltip, Divider, Badge } from "@geist-ui/core"
 
 const FilenameInput = ({ title }: { title: string }) => <Input
     value={title || 'No title'}
@@ -32,22 +31,29 @@ const ListItem = ({ post }: { post: any }) => {
     const formattedTime = `${createdDate.toLocaleDateString()} ${createdDate.toLocaleTimeString()}`
     return (<li key={post.id}>
         <Card style={{ overflowY: 'scroll' }}>
-            <Spacer height={1 / 2} />
-            <Grid.Container>
-                <Grid md={14} xs={14}>
-                    <Text h3 paddingLeft={1 / 2} >
-                        <NextLink passHref={true} href={getPostPath(post.visibility, post.id)}>
-                            <Link color>{post.title}
-                                <ShiftBy y={-1}><VisibilityBadge visibility={post.visibility} /></ShiftBy>
-                            </Link>
-                        </NextLink>
-                    </Text></Grid>
-                <Grid paddingLeft={1 / 2} md={5} xs={9}><Text type="secondary" h5><Tooltip text={formattedTime}>{time}</Tooltip></Text></Grid>
-                <Grid paddingLeft={1 / 2} md={5} xs={4}><Text type="secondary" h5>{post.files.length === 1 ? "1 file" : `${post.files.length} files`}</Text></Grid>
-            </Grid.Container>
+            <Card.Body>
+                <Text h3>
+                    <NextLink passHref={true} href={getPostPath(post.visibility, post.id)}>
+                        <Link color marginRight={'var(--gap)'}>
+                            {post.title}
+                        </Link>
+                    </NextLink>
+                    <div style={{ display: 'inline-flex' }}>
+                        <span>
+                            <VisibilityBadge visibility={post.visibility} />
+                        </span>
+                        <span style={{ marginLeft: 'var(--gap)' }}>
+                            <Badge type="secondary"><Tooltip text={formattedTime}>{time}</Tooltip></Badge>
+                        </span>
+                        <span style={{ marginLeft: 'var(--gap)' }}>
+                            <Badge type="secondary">{post.files.length === 1 ? "1 file" : `${post.files.length} files`}</Badge>
+                        </span>
 
+                    </div>
+                </Text>
+
+            </Card.Body>
             <Divider h="1px" my={0} />
-
             <Card.Content>
                 {post.files.map((file: any) => {
                     return <FilenameInput key={file.id} title={file.title} />
diff --git a/client/components/post-list/post-list.module.css b/client/components/post-list/post-list.module.css
index d883c22..2c9fdc7 100644
--- a/client/components/post-list/post-list.module.css
+++ b/client/components/post-list/post-list.module.css
@@ -31,5 +31,4 @@
 	flex-direction: column-reverse;
 	justify-content: center;
 	margin-bottom: var(--gap-double);
-	margin-top: var(--gap-double);
 }
diff --git a/client/components/visibility-badge/index.tsx b/client/components/visibility-badge/index.tsx
index d907fda..b640b57 100644
--- a/client/components/visibility-badge/index.tsx
+++ b/client/components/visibility-badge/index.tsx
@@ -17,7 +17,7 @@ const VisibilityBadge = ({ visibility }: Props) => {
         }
     }
 
-    return (<Badge marginLeft={'var(--gap)'} type={getBadgeType()}>{visibility}</Badge>)
+    return (<Badge type={getBadgeType()}>{visibility}</Badge>)
 }
 
 export default VisibilityBadge
diff --git a/client/pages/mine.tsx b/client/pages/mine.tsx
index 363f920..6d487cb 100644
--- a/client/pages/mine.tsx
+++ b/client/pages/mine.tsx
@@ -40,7 +40,7 @@ export const getServerSideProps: GetServerSideProps = async ({ req }) => {
     }
   })
 
-  if (!posts.ok || posts.status !== 200) {
+  if (!posts.ok) {
     return {
       redirect: {
         destination: '/',