Borders (onyx-dot-app#3388)

* remove double x * incorporate base default padding for modals
aronszanto · Dec 23, 2024 · cb0c049 · cb0c049
1 parent dd86913
commit cb0c049
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/web/src/app/admin/api-key/DanswerApiKeyForm.tsx b/web/src/app/admin/api-key/DanswerApiKeyForm.tsx
@@ -82,7 +82,7 @@ export const DanswerApiKeyForm = ({
           }}
         >
           {({ isSubmitting, values, setFieldValue }) => (
-            <Form>
+            <Form className="w-full overflow-visible">
               <Text className="mb-4 text-lg">
                 Choose a memorable name for your API key. This is optional and
                 can be added or changed later!

diff --git a/web/src/app/admin/api-key/page.tsx b/web/src/app/admin/api-key/page.tsx
@@ -45,9 +45,6 @@ function NewApiKeyModal({
       <div className="px-8 py-8">
         <div className="flex w-full border-b border-border mb-4 pb-4">
           <Title>New API Key</Title>
-          <div onClick={onClose} className="ml-auto p-1 rounded hover:bg-hover">
-            <FiX size={18} />
-          </div>
         </div>
         <div className="h-32">
           <Text className="mb-4">

diff --git a/web/src/components/Modal.tsx b/web/src/components/Modal.tsx
@@ -99,7 +99,7 @@ export function Modal({
             </button>
           </div>
         )}
-        <div className="w-full overflow-y-hidden flex flex-col h-full justify-stretch">
+        <div className="w-full overflow-y-auto overflow-x-visible p-1 flex flex-col h-full justify-stretch">
           {title && (
             <>
               <div className="flex mb-4">
@@ -117,7 +117,7 @@ export function Modal({
           )}
           <div
             className={cn(
-              noScroll ? "overflow-auto" : "overflow-x-hidden",
+              noScroll ? "overflow-auto" : "overflow-x-visible",
               height || "max-h-[60vh]"
             )}
           >