From 882f7c38c5cb071c0ec49c344be6897e3898f8e8 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 21 Oct 2023 16:11:52 -0500
Subject: [PATCH] Fix `rows` prop in Textarea, use Textarea in
 SiteErrorBoundary

---
 src/components/site-error-boundary.tsx  | 8 ++++----
 src/components/ui/textarea/textarea.tsx | 5 +++--
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/components/site-error-boundary.tsx b/src/components/site-error-boundary.tsx
index 698cd6ea0..85ed50cac 100644
--- a/src/components/site-error-boundary.tsx
+++ b/src/components/site-error-boundary.tsx
@@ -3,7 +3,7 @@ import { ErrorBoundary } from 'react-error-boundary';
 import { FormattedMessage } from 'react-intl';
 
 import { NODE_ENV } from 'soapbox/build-config';
-import { HStack, Text, Stack } from 'soapbox/components/ui';
+import { HStack, Text, Stack, Textarea } from 'soapbox/components/ui';
 import { useSoapboxConfig } from 'soapbox/hooks';
 import { captureSentryException } from 'soapbox/sentry';
 import KVStore from 'soapbox/storage/kv-store';
@@ -121,12 +121,12 @@ const SiteErrorBoundary: React.FC<ISiteErrorBoundary> = ({ children }) => {
             ) : (
               <>
                 {errorText && (
-                  <textarea
+                  <Textarea
                     ref={textarea}
-                    className='block h-48 w-full rounded-md border-gray-300 bg-gray-100 p-4 font-mono text-gray-900 shadow-sm focus:border-primary-500 focus:ring-2 focus:ring-primary-500 dark:border-gray-700 dark:bg-gray-800 dark:text-gray-100 sm:text-sm'
                     value={errorText}
                     onClick={handleCopy}
-                    dir='ltr'
+                    isCodeEditor
+                    rows={12}
                     readOnly
                   />
                 )}
diff --git a/src/components/ui/textarea/textarea.tsx b/src/components/ui/textarea/textarea.tsx
index d1abaf2d6..0ba036a08 100644
--- a/src/components/ui/textarea/textarea.tsx
+++ b/src/components/ui/textarea/textarea.tsx
@@ -8,7 +8,7 @@ import { getTextDirection } from 'soapbox/utils/rtl';
 import Stack from '../stack/stack';
 import Text from '../text/text';
 
-interface ITextarea extends Pick<React.TextareaHTMLAttributes<HTMLTextAreaElement>, 'id' | 'maxLength' | 'onChange' | 'onKeyDown' | 'onPaste' | 'required' | 'disabled' | 'rows' | 'readOnly'> {
+interface ITextarea extends Pick<React.TextareaHTMLAttributes<HTMLTextAreaElement>, 'id' | 'maxLength' | 'onChange' | 'onClick' | 'onKeyDown' | 'onPaste' | 'required' | 'disabled' | 'rows' | 'readOnly'> {
   /** Put the cursor into the input on mount. */
   autoFocus?: boolean;
   /** Allows the textarea height to grow while typing */
@@ -48,13 +48,14 @@ const Textarea = React.forwardRef(({
   autoGrow = false,
   maxRows = 10,
   minRows = 1,
+  rows: initialRows = 4,
   theme = 'default',
   maxLength,
   value,
   ...props
 }: ITextarea, ref: React.ForwardedRef<HTMLTextAreaElement>) => {
   const length = value?.length || 0;
-  const [rows, setRows] = useState<number>(autoGrow ? 1 : 4);
+  const [rows, setRows] = useState<number>(autoGrow ? minRows : initialRows);
   const locale = useLocale();
 
   const handleChange = (event: React.ChangeEvent<HTMLTextAreaElement>) => {