From 03f032998454b31702b2d5dd96d10ae582497f73 Mon Sep 17 00:00:00 2001
From: keyan <keyan.kousha+huumn@gmail.com>
Date: Sun, 13 Aug 2023 14:12:18 -0500
Subject: [PATCH] fix image uploading/edit

---
 components/avatar.js   | 30 ++++++++++++++++--------------
 components/job-form.js |  2 +-
 pages/settings.js      |  6 +++---
 3 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/components/avatar.js b/components/avatar.js
index 2e71b330..e9d08c32 100644
--- a/components/avatar.js
+++ b/components/avatar.js
@@ -9,13 +9,14 @@ import { useShowModal } from './modal'
 
 export default function Avatar ({ onSuccess }) {
   const [uploading, setUploading] = useState()
-  const ref = useRef()
-  const [scale, setScale] = useState(1)
   const showModal = useShowModal()
 
   const Body = ({ onClose, file, upload }) => {
+    const [scale, setScale] = useState(1)
+    const ref = useRef()
+
     return (
-      <div className='text-right mt-1 p-4'>
+      <div className='text-end mt-1 p-4'>
         <AvatarEditor
           ref={ref} width={200} height={200}
           image={file}
@@ -26,20 +27,21 @@ export default function Avatar ({ onSuccess }) {
           }}
         />
         <BootstrapForm.Group controlId='formBasicRange'>
-          <BootstrapForm.Control
-            type='range' onChange={e => setScale(parseFloat(e.target.value))}
+          <BootstrapForm.Range
+            onChange={e => setScale(parseFloat(e.target.value))}
             min={1} max={2} step='0.05'
-            defaultValue={scale}
+            // defaultValue={scale}
           />
         </BootstrapForm.Group>
-        <Button onClick={() => {
-          ref.current.getImageScaledToCanvas().toBlob(blob => {
-            if (blob) {
-              upload(blob)
-              onClose()
-            }
-          }, 'image/jpeg')
-        }}
+        <Button
+          onClick={() => {
+            ref.current.getImageScaledToCanvas().toBlob(blob => {
+              if (blob) {
+                upload(blob)
+                onClose()
+              }
+            }, 'image/jpeg')
+          }}
         >save
         </Button>
       </div>
diff --git a/components/job-form.js b/components/job-form.js
index c484e0a5..d710a2fb 100644
--- a/components/job-form.js
+++ b/components/job-form.js
@@ -80,7 +80,7 @@ export default function JobForm ({ item, sub }) {
       } else {
         await router.push(`/~${sub.name}/recent`)
       }
-    }, [upsertJob, router])
+    }, [upsertJob, router, item?.id, sub?.name, logoId])
 
   const invoiceableUpsertJob = useInvoiceable(submitUpsertJob, { requireSession: true })
 
diff --git a/pages/settings.js b/pages/settings.js
index ce2b1c40..2ed3b14e 100644
--- a/pages/settings.js
+++ b/pages/settings.js
@@ -52,7 +52,7 @@ export default function Settings ({ ssrData }) {
   return (
     <CenterLayout>
       <div className='py-3 w-100'>
-        <h2 className='mb-2 text-left'>settings</h2>
+        <h2 className='mb-2 text-start'>settings</h2>
         <Form
           initial={{
             tipDefault: settings?.tipDefault || 21,
@@ -248,7 +248,7 @@ export default function Settings ({ ssrData }) {
           <AccordianItem
             headerColor='var(--bs-body-color)'
             show={settings?.nostrPubkey}
-            header={<h4 className='text-left'>nostr <small><a href='https://github.com/nostr-protocol/nips/blob/master/05.md' target='_blank' rel='noreferrer'>NIP-05</a></small></h4>}
+            header={<h4 className='text-start'>nostr <small><a href='https://github.com/nostr-protocol/nips/blob/master/05.md' target='_blank' rel='noreferrer'>NIP-05</a></small></h4>}
             body={
               <>
                 <Input
@@ -270,7 +270,7 @@ export default function Settings ({ ssrData }) {
             <SubmitButton variant='info' className='ms-auto mt-1 px-4'>save</SubmitButton>
           </div>
         </Form>
-        <div className='text-left w-100'>
+        <div className='text-start w-100'>
           <div className='form-label'>saturday newsletter</div>
           <Button href='https://mail.stacker.news/subscription/form' target='_blank'>(re)subscribe</Button>
           {settings?.authMethods && <AuthMethods methods={settings.authMethods} />}