fix app layout dom order

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/renderer/components/__snapshots__/Sidebar.test.tsx.snap

@@ -24,14 +24,14 @@ export const LogoIcon: FC<ILogoIcon> = ({
   ...props
 }: ILogoIcon) => (
   <svg
+    aria-hidden="true"
     aria-label={`${APPLICATION.NAME} Logo`}
     className={cn(
       size === Size.SMALL && 'size-5',
       size === Size.MEDIUM && 'size-10',
       size === Size.LARGE && 'size-16',
     )}
     onClick={() => onClick?.()}
-    role="img"
     viewBox="0 0 500 500"
     xmlns="https://www.w3.org/2000/svg"
     xmlnsXlink="https://www.w3.org/1999/xlink"

src/renderer/components/icons/LogoIcon.tsx

@@ -13,9 +13,10 @@ interface IAppLayout {
 export const AppLayout: FC<IAppLayout> = ({ children }) => {
   return (
     <div className="flex flex-col min-h-screen bg-gitify-background">
-      <Sidebar />
-      {/* Content area with left padding to make space for the sidebar */}
+      {/* Content first in DOM so initial focus won't land on sidebar buttons */}
       <div className="flex-1 pl-sidebar">{children}</div>
+
+      <Sidebar />
     </div>
   );
 };