fix: mobile sheet ui fix (#20)
This commit is contained in:
		
							parent
							
								
									4c32248d86
								
							
						
					
					
						commit
						1939ff4153
					
				@ -29,6 +29,7 @@ export function Navbar() {
 | 
			
		||||
  const { organization } = useOrganization();
 | 
			
		||||
  const _isDesktop = useMediaQuery("(min-width: 1024px)");
 | 
			
		||||
  const [isDesktop, setIsDesktop] = useState(true);
 | 
			
		||||
  const [isSheetOpen, setSheetOpen] = useState(false);
 | 
			
		||||
  useEffect(() => {
 | 
			
		||||
    setIsDesktop(_isDesktop);
 | 
			
		||||
  }, [_isDesktop]);
 | 
			
		||||
@ -36,7 +37,7 @@ export function Navbar() {
 | 
			
		||||
    <>
 | 
			
		||||
      <div className="flex flex-row items-center gap-4">
 | 
			
		||||
        {!isDesktop && (
 | 
			
		||||
          <Sheet>
 | 
			
		||||
          <Sheet open={isSheetOpen} onOpenChange={(open) => setSheetOpen(open)}>
 | 
			
		||||
            <SheetTrigger asChild>
 | 
			
		||||
              <button className="flex items-center justify-center w-8 h-8 p-2">
 | 
			
		||||
                <Menu />
 | 
			
		||||
@ -47,7 +48,10 @@ export function Navbar() {
 | 
			
		||||
                <SheetTitle className="text-start">Comfy Deploy</SheetTitle>
 | 
			
		||||
              </SheetHeader>
 | 
			
		||||
              <div className="grid h-full grid-rows-[1fr_auto]">
 | 
			
		||||
                <NavbarMenu className=" h-full" />
 | 
			
		||||
                <NavbarMenu
 | 
			
		||||
                  className=" h-full"
 | 
			
		||||
                  closeSheet={() => setSheetOpen(false)}
 | 
			
		||||
                />
 | 
			
		||||
                {/* <OrganizationSwitcher
 | 
			
		||||
                  appearance={{
 | 
			
		||||
                    elements: {
 | 
			
		||||
 | 
			
		||||
@ -9,7 +9,13 @@ import { useRouter } from "next/navigation";
 | 
			
		||||
import { useEffect, useState } from "react";
 | 
			
		||||
import { useMediaQuery } from "usehooks-ts";
 | 
			
		||||
 | 
			
		||||
export function NavbarMenu({ className }: { className?: string }) {
 | 
			
		||||
export function NavbarMenu({
 | 
			
		||||
  className,
 | 
			
		||||
  closeSheet,
 | 
			
		||||
}: {
 | 
			
		||||
  className?: string;
 | 
			
		||||
  closeSheet?: () => void;
 | 
			
		||||
}) {
 | 
			
		||||
  const _isDesktop = useMediaQuery("(min-width: 1024px)");
 | 
			
		||||
  const [isDesktop, setIsDesktop] = useState(true);
 | 
			
		||||
  useEffect(() => {
 | 
			
		||||
@ -68,6 +74,9 @@ export function NavbarMenu({ className }: { className?: string }) {
 | 
			
		||||
              <Link
 | 
			
		||||
                key={page.name}
 | 
			
		||||
                href={page.path}
 | 
			
		||||
                onClick={() => {
 | 
			
		||||
                  if (!!closeSheet) closeSheet();
 | 
			
		||||
                }}
 | 
			
		||||
                className="p-2 hover:bg-gray-100/20 hover:underline"
 | 
			
		||||
              >
 | 
			
		||||
                {page.name}
 | 
			
		||||
 | 
			
		||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user