fix: design parity (#2873)

* fix: review comments * fix: remove border
100mslive · May 6, 2024 · 279ed73 · 279ed73
1 parent cce83da
commit 279ed73
Showing 1 changed file with 6 additions and 9 deletions.
diff --git a/packages/roomkit-react/src/Prebuilt/components/RoleChangeModal.tsx b/packages/roomkit-react/src/Prebuilt/components/RoleChangeModal.tsx
@@ -56,7 +56,7 @@ const RoleChangeContent = ({
 
       <Box>
         <Text
-          variant="body2"
+          variant="sm"
           css={{
             mt: '$4',
             mb: '$8',
@@ -68,7 +68,7 @@ const RoleChangeContent = ({
         >
           Switch the role of
           <HighlightTerm value={peer.name} />
-          from <HighlightTerm value={peer.roleName} />
+          from <HighlightTerm value={peer.roleName} /> to
         </Text>
       </Box>
       <Flex
@@ -92,7 +92,6 @@ const RoleChangeContent = ({
               data-testid="open_role_selection_dropdown"
               asChild
               css={{
-                border: '1px solid $border_bright',
                 bg: '$surface_default',
                 r: '$1',
                 p: '$6 $9',
@@ -176,12 +175,10 @@ export const RoleChangeModal = ({
 
   return (
     <Dialog.Root defaultOpen onOpenChange={onOpenChange}>
-      <Dialog.Portal>
-        <Dialog.Overlay />
-        <Dialog.Content css={{ width: 'min(400px,80%)', p: '$10' }}>
-          <RoleChangeContent peer={peer} onOpenChange={onOpenChange} />
-        </Dialog.Content>
-      </Dialog.Portal>
+      <Dialog.Overlay />
+      <Dialog.Content css={{ width: 'min(400px,80%)', p: '$10', overflow: 'visible' }}>
+        <RoleChangeContent peer={peer} onOpenChange={onOpenChange} />
+      </Dialog.Content>
     </Dialog.Root>
   );
 };