foss/excalidraw
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Small refactor. Fixes library dropdown.

Browse Source
This commit is contained in:
barnabasmolnar 2023-04-17 17:12:42 +02:00
parent e094b8b539
commit 79cf71cccb
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/components/dropdownMenu/DropdownMenu.tsx
View File

@ -13,6 +13,8 @@ import {
import "./DropdownMenu.scss"; import "./DropdownMenu.scss";
import * as DropdownMenuRadix from "@radix-ui/react-dropdown-menu";
const DropdownMenu = ({ const DropdownMenu = ({
children, children,
open, open,
@ -23,10 +25,10 @@ const DropdownMenu = ({
const MenuTriggerComp = getMenuTriggerComponent(children); const MenuTriggerComp = getMenuTriggerComponent(children);
const MenuContentComp = getMenuContentComponent(children); const MenuContentComp = getMenuContentComponent(children);
return ( return (
<> <DropdownMenuRadix.Root open={open}>
{MenuTriggerComp} {MenuTriggerComp}
{open && MenuContentComp} {open && MenuContentComp}
</> </DropdownMenuRadix.Root>
); );
}; };

6
src/components/main-menu/MainMenu.tsx
View File

@ -15,8 +15,6 @@ import { withInternalFallback } from "../hoc/withInternalFallback";
import { composeEventHandlers } from "../../utils"; import { composeEventHandlers } from "../../utils";
import { useTunnels } from "../context/tunnels"; import { useTunnels } from "../context/tunnels";
import * as DropdownMenuRadix from "@radix-ui/react-dropdown-menu";
const MainMenu = Object.assign( const MainMenu = Object.assign(
withInternalFallback( withInternalFallback(
"MainMenu", "MainMenu",
@ -40,7 +38,7 @@ const MainMenu = Object.assign(
return ( return (
<mainMenuTunnel.In> <mainMenuTunnel.In>
<DropdownMenuRadix.Root open={appState.openMenu === "canvas"}> <DropdownMenu open={appState.openMenu === "canvas"}>
<DropdownMenu.Trigger <DropdownMenu.Trigger
onToggle={() => { onToggle={() => {
setAppState({ setAppState({
@ -67,7 +65,7 @@ const MainMenu = Object.assign(
</fieldset> </fieldset>
)} )}
</DropdownMenu.Content> </DropdownMenu.Content>
</DropdownMenuRadix.Root> </DropdownMenu>
</mainMenuTunnel.In> </mainMenuTunnel.In>
); );
}, },