chore(dashboard): resolve copilot comments

awisniew207 · awisniew207 · commit 12eaf75f5236 · 2025-10-23T16:11:13.000-07:00
diff --git a/packages/apps/app-dashboard/public/fonts/encode-sans-semi-expanded/style.css b/packages/apps/app-dashboard/public/fonts/encode-sans-semi-expanded/style.css
@@ -1,8 +1,6 @@
 @font-face {
     font-family: 'Encode Sans Semi Expanded';
     src: local('Encode Sans Semi Expanded Regular'), local('Encode-Sans-Semi-Expanded-Regular'),
-        url('EncodeSansSemiExpanded-Regular.woff2') format('woff2'),
-        url('EncodeSansSemiExpanded-Regular.woff') format('woff'),
         url('EncodeSansSemiExpanded-Regular.ttf') format('truetype');
     font-weight: 400;
     font-style: normal;
diff --git a/packages/apps/app-dashboard/src/components/explorer/ui/AppFilter.tsx b/packages/apps/app-dashboard/src/components/explorer/ui/AppFilter.tsx
@@ -32,7 +32,7 @@ export function AppFilter({
               placeholder="Search applications..."
               value={searchQuery}
               onChange={(e) => setSearchQuery(e.target.value)}
-              className={`w-full pl-10 pr-4 py-3 rounded-xl ${theme.itemBg} border ${theme.cardBorder} ${theme.text} ${theme.textMuted} transition-all duration-300 focus:outline-none`}
+              className={`w-full pl-10 pr-4 py-3 rounded-xl ${theme.itemBg} border ${theme.cardBorder} ${theme.text} ${theme.textMuted} transition-all duration-300 focus:outline-none focus:ring-2 focus:ring-[#FF4205]`}
               style={fonts.body}
             />
           </div>
@@ -44,7 +44,7 @@ export function AppFilter({
               <select
                 value={statusFilter}
                 onChange={(e) => setStatusFilter(e.target.value)}
-                className={`px-3 py-2 rounded-lg bg-white dark:bg-gray-900 border ${theme.cardBorder} ${theme.text} text-sm transition-all duration-300 focus:outline-none hover:border-gray-300 dark:hover:border-white/20 cursor-pointer`}
+                className={`px-3 py-2 rounded-lg bg-white dark:bg-gray-900 border ${theme.cardBorder} ${theme.text} text-sm transition-all duration-300 focus:outline-none focus:ring-2 focus:ring-[#FF4205] hover:border-gray-300 dark:hover:border-white/20 cursor-pointer`}
                 style={fonts.heading}
               >
                 <option
@@ -71,7 +71,7 @@ export function AppFilter({
             <select
               value={sortBy}
               onChange={(e) => setSortBy(e.target.value)}
-              className={`px-3 py-2 rounded-lg bg-white dark:bg-gray-900 border ${theme.cardBorder} ${theme.text} text-sm transition-all duration-300 focus:outline-none hover:border-gray-300 dark:hover:border-white/20 cursor-pointer`}
+              className={`px-3 py-2 rounded-lg bg-white dark:bg-gray-900 border ${theme.cardBorder} ${theme.text} text-sm transition-all duration-300 focus:outline-none focus:ring-2 focus:ring-[#FF4205] hover:border-gray-300 dark:hover:border-white/20 cursor-pointer`}
               style={fonts.heading}
             >
               <option
diff --git a/packages/apps/app-dashboard/src/components/explorer/ui/ExplorerNav.tsx b/packages/apps/app-dashboard/src/components/explorer/ui/ExplorerNav.tsx
@@ -41,9 +41,7 @@ export function ExplorerNav({ onNavigate, sidebarTrigger }: ExplorerNavProps) {
           <div className="flex items-center gap-3 sm:gap-6">
             {/* Sidebar trigger for mobile */}
             {sidebarTrigger && (
-              <div className="md:hidden flex items-center relative" style={{ paddingTop: '10px' }}>
-                {sidebarTrigger}
-              </div>
+              <div className="md:hidden flex items-center relative pt-2.5">{sidebarTrigger}</div>
             )}
             {/* Logo */}
             <button
@@ -62,7 +60,7 @@ export function ExplorerNav({ onNavigate, sidebarTrigger }: ExplorerNavProps) {
             </button>
 
             {/* Desktop Nav Links */}
-            <div className="hidden md:flex items-center gap-1" style={{ paddingTop: '10px' }}>
+            <div className="hidden md:flex items-center gap-1 pt-2.5">
               <button
                 onClick={() => handleNavigation('/user/apps')}
                 className="px-3 py-1.5 rounded-lg text-sm transition-colors no-underline hover:bg-gray-100 dark:hover:bg-gray-800 leading-none"
@@ -97,7 +95,7 @@ export function ExplorerNav({ onNavigate, sidebarTrigger }: ExplorerNavProps) {
           </div>
 
           {/* Right side: Utility links and mobile menu */}
-          <div className="flex items-center gap-1 md:gap-2" style={{ paddingTop: '10px' }}>
+          <div className="flex items-center gap-1 md:gap-2 pt-2.5">
             {/* Desktop utility links */}
             <div className="hidden md:flex items-center gap-1 md:gap-2">
               <a
diff --git a/packages/apps/app-dashboard/src/components/shared/ui/sidebar.tsx b/packages/apps/app-dashboard/src/components/shared/ui/sidebar.tsx
@@ -178,7 +178,7 @@ function Sidebar({
 
   if (isMobile) {
     return (
-      <Sheet open={openMobile} onOpenChange={setOpenMobile} modal={false} {...props}>
+      <Sheet open={openMobile} onOpenChange={setOpenMobile} modal={true} {...props}>
         <SheetContent
           data-slot="sidebar"
           data-mobile="true"
diff --git a/packages/apps/app-dashboard/src/components/user-dashboard/auth/AuthMethods.tsx b/packages/apps/app-dashboard/src/components/user-dashboard/auth/AuthMethods.tsx
@@ -1,8 +1,9 @@
 import { Dispatch, SetStateAction, useEffect } from 'react';
 import { theme as themeImport, fonts } from '../connect/ui/theme';
+import { AuthView } from '../connect/Connect';
 
 interface AuthMethodsProps {
-  setView: Dispatch<SetStateAction<string>>;
+  setView: Dispatch<SetStateAction<AuthView>>;
   clearError?: () => void;
 }
 
diff --git a/packages/apps/app-dashboard/src/components/user-dashboard/auth/ConnectMethods.tsx b/packages/apps/app-dashboard/src/components/user-dashboard/auth/ConnectMethods.tsx
@@ -24,7 +24,7 @@ interface ConnectProps {
   clearError?: () => void;
   theme: ThemeType;
   view: AuthView;
-  setView: (view: AuthView) => void;
+  setView: Dispatch<SetStateAction<AuthView>>;
 }
 
 export default function ConnectMethods({
@@ -47,10 +47,7 @@ export default function ConnectMethods({
           exit={{ opacity: 0 }}
           transition={{ duration: 0.2, ease: 'easeInOut' }}
         >
-          <AuthMethods
-            setView={setView as Dispatch<SetStateAction<string>>}
-            clearError={clearError}
-          />
+          <AuthMethods setView={setView} clearError={clearError} />
         </motion.div>
       )}
       {view === 'email' && (
@@ -64,7 +61,7 @@ export default function ConnectMethods({
           <StytchOTP
             method="email"
             authWithStytch={authWithStytch}
-            setView={setView as Dispatch<SetStateAction<string>>}
+            setView={setView}
             theme={theme}
           />
         </motion.div>
@@ -80,7 +77,7 @@ export default function ConnectMethods({
           <StytchOTP
             method="phone"
             authWithStytch={authWithStytch}
-            setView={setView as Dispatch<SetStateAction<string>>}
+            setView={setView}
             theme={theme}
           />
         </motion.div>
@@ -93,11 +90,7 @@ export default function ConnectMethods({
           exit={{ opacity: 0 }}
           transition={{ duration: 0.2, ease: 'easeInOut' }}
         >
-          <EthWalletAuth
-            authWithEthWallet={authWithEthWallet}
-            setView={setView as Dispatch<SetStateAction<string>>}
-            theme={theme}
-          />
+          <EthWalletAuth authWithEthWallet={authWithEthWallet} setView={setView} theme={theme} />
         </motion.div>
       )}
       {view === 'webauthn' && (
@@ -111,7 +104,7 @@ export default function ConnectMethods({
           <WebAuthn
             authWithWebAuthn={authWithWebAuthn}
             registerWithWebAuthn={registerWithWebAuthn}
-            setView={setView as Dispatch<SetStateAction<string>>}
+            setView={setView}
             clearError={clearError}
             theme={theme}
           />
diff --git a/packages/apps/app-dashboard/src/components/user-dashboard/auth/EthWalletAuth.tsx b/packages/apps/app-dashboard/src/components/user-dashboard/auth/EthWalletAuth.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect, useRef } from 'react';
+import { useState, useEffect, useRef, Dispatch, SetStateAction } from 'react';
 import { createAppKit } from '@reown/appkit/react';
 import { WagmiAdapter } from '@reown/appkit-adapter-wagmi';
 import { mainnet, polygon, arbitrum, optimism, base, AppKitNetwork } from '@reown/appkit/networks';
@@ -7,13 +7,14 @@ import { Button } from '@/components/shared/ui/button';
 import { ThemeType, fonts } from '../connect/ui/theme';
 import StatusMessage from '../connect/StatusMessage';
 import { env } from '@/config/env';
+import { AuthView } from '../connect/Connect';
 
 interface WalletAuthProps {
   authWithEthWallet: (
     address: string,
     signMessage: (message: string) => Promise<string>,
   ) => Promise<void>;
-  setView: (view: string) => void;
+  setView: Dispatch<SetStateAction<AuthView>>;
   theme: ThemeType;
 }
 
@@ -258,4 +259,4 @@ export default function EthWalletAuth({ authWithEthWallet, theme }: WalletAuthPr
       </div>
     </>
   );
-}
+}
diff --git a/packages/apps/app-dashboard/src/components/user-dashboard/auth/StytchOTP.tsx b/packages/apps/app-dashboard/src/components/user-dashboard/auth/StytchOTP.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect } from 'react';
+import { useState, useEffect, Dispatch, SetStateAction } from 'react';
 import { useStytch } from '@stytch/react';
 import { z } from 'zod';
 import { Button } from '@/components/shared/ui/button';
@@ -7,11 +7,12 @@ import StatusMessage from '../connect/StatusMessage';
 import { countryCodes } from '@/utils/user-dashboard/countryCodes';
 import CountryCodeSelector from '@/components/shared/ui/CountryCodeSelector';
 import validator from 'validator';
+import { AuthView } from '../connect/Connect';
 
 interface StytchOTPProps {
   method: OtpMethod;
   authWithStytch: any;
-  setView: React.Dispatch<React.SetStateAction<string>>;
+  setView: Dispatch<SetStateAction<AuthView>>;
   theme: ThemeType;
 }
 
diff --git a/packages/apps/app-dashboard/src/components/user-dashboard/auth/WebAuthn.tsx b/packages/apps/app-dashboard/src/components/user-dashboard/auth/WebAuthn.tsx
@@ -1,12 +1,13 @@
-import { useState } from 'react';
+import { useState, Dispatch, SetStateAction } from 'react';
 import { Button } from '@/components/shared/ui/button';
 import { ThemeType, fonts } from '../connect/ui/theme';
 import StatusMessage from '../connect/StatusMessage';
 import { PasskeyNameInput } from '@/components/shared/ui/PasskeyNameInput';
+import { AuthView } from '../connect/Connect';
 
 interface WebAuthnProps {
   authWithWebAuthn: any;
-  setView: React.Dispatch<React.SetStateAction<string>>;
+  setView: Dispatch<SetStateAction<AuthView>>;
   registerWithWebAuthn?: (displayName: string) => Promise<void>;
   clearError?: () => void;
   theme: ThemeType;

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,9 @@`
`1`	`1`	`import { Dispatch, SetStateAction, useEffect } from 'react';`
`2`	`2`	`import { theme as themeImport, fonts } from '../connect/ui/theme';`
	`3`	`+import { AuthView } from '../connect/Connect';`
`3`	`4`
`4`	`5`	`interface AuthMethodsProps {`
`5`		`- setView: Dispatch<SetStateAction<string>>;`
	`6`	`+ setView: Dispatch<SetStateAction<AuthView>>;`
`6`	`7`	`clearError?: () => void;`
`7`	`8`	`}`
`8`	`9`