fix: should remove app skeleton in error page

Signed-off-by: Innei <tukon479@gmail.com>

Author: Innei

apps/renderer/src/App.tsx

@@ -53,15 +53,15 @@ const AppLayer = () => {
   const appIsReady = useAppIsReady()
 
   useEffect(() => {
+    document.querySelector("#app-skeleton")?.remove()
+
     const doneTime = Math.trunc(performance.now())
     window.analytics?.capture("ui_render_init", {
       time: doneTime,
     })
     appLog("App is ready", `${doneTime}ms`)
 
     applyAfterReadyCallbacks()
-
-    document.querySelector("#app-skeleton")?.remove()
   }, [appIsReady])
 
   return appIsReady ? <Outlet /> : <AppSkeleton />

apps/renderer/src/components/common/ErrorElement.tsx

@@ -20,6 +20,10 @@ export function ErrorElement() {
       : JSON.stringify(error)
   const stack = error instanceof Error ? error.stack : null
 
+  useEffect(() => {
+    document.querySelector("#app-skeleton")?.remove()
+  }, [])
+
   useEffect(() => {
     console.error("Error handled by React Router default ErrorBoundary:", error)