@visns-studio/visns-components 6.5.2 → 6.5.4

package/package.json

@@ -91,7 +91,7 @@
         "react-dom": "^17.0.0 || ^18.0.0"
     },
     "name": "@visns-studio/visns-components",
-    "version": "6.5.2",
+    "version": "6.5.4",
     "description": "Various packages to assist in the development of our Custom Applications.",
     "main": "src/index.js",
     "files": [

package/src/components/auth/Login.jsx

@@ -147,6 +147,11 @@ const Login = ({
                                 window.location.href = result.previous;
                             } else {
                                 setSystemAuth(true);
+                                // Move now, with the profile the server just
+                                // returned — waiting for the background
+                                // re-check to navigate leaves the login form
+                                // on screen for a beat after success.
+                                navigate('/', { replace: true });
                             }
                         }
                     } else {

package/src/components/generic/GenericAuth.jsx

@@ -121,12 +121,16 @@ const GenericAuth = ({
     const [isClientAuthenticated, setIsClientAuthenticated] = useState(false);
 
     const updateAuthStatus = async () => {
-        // Guards must not act while the profile check is in flight: after a
-        // login flips isAuthenticated this re-runs, and a route evaluated
-        // against the half-updated state bounced dashboard → /login →
-        // dashboard. Loading renders AuthLoading instead, so the user sees
-        // one spinner and one navigation.
-        setIsLoading(true);
+        // Guards must not act while the FIRST profile check is in flight:
+        // evaluated against empty state they bounced dashboard → /login →
+        // dashboard after sign-in. But only block when there is no usable
+        // profile — raising loading over an already-authenticated tree swaps
+        // the whole page for a spinner on every background re-check, which
+        // is its own flicker. With a profile in hand the re-check is
+        // invisible.
+        if (!userProfile.id) {
+            setIsLoading(true);
+        }
 
         try {
             const { data } = await CustomFetch('/ajax/user/profile', 'GET');
@@ -137,7 +141,12 @@ const GenericAuth = ({
             }
             setIsLoading(false);
 
-            if (location.pathname.split('/')[1] === 'login') {
+            // The live pathname, not the router hook's: this resolves after
+            // Login has already navigated, and the closure still holds
+            // '/login' from the render that created the effect — firing a
+            // second navigate('/') over the settled dashboard, whose route
+            // re-render is the sub-second flicker after sign-in.
+            if (window.location.pathname.split('/')[1] === 'login') {
                 navigate('/', { replace: true });
             }
         } catch (error) {

package/src/components/styles/Navigation.module.scss

@@ -888,6 +888,15 @@
     display: flex;
     flex-direction: column;
 
+    /* The rail is `position: fixed` with no z-index, so its notification /
+       switch-account panels — stacked inside it — lost to the content's own
+       contexts (grid dropdowns sit at 1000+) and painted underneath the
+       page. Desktop only: above the content chrome, below modal overlays.
+       The ≤1024px drawer keeps its own 600-under-the-header stacking. */
+    @media (min-width: 1025px) {
+        z-index: 1050;
+    }
+
     /* The rail clips itself so a long nav list can't spill; the panels that
        open from the action rows — the notification list, the account
        switcher — would be cut off by that. The nav list keeps its own