@visns-studio/visns-components 6.0.4 → 6.0.5

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

@@ -11,7 +11,11 @@
         padding: 0;
         margin: 0;
         line-height: 1.45;
-        margin-right: 1rem;
+        /* The mark and the navigation were butted together, reading as one
+           run of items. A hairline gives the logo its own territory. */
+        margin-right: var(--spacing-md);
+        padding-right: var(--spacing-md);
+        border-right: 1px solid rgba(255, 255, 255, 0.18);
 
         .logoButton {
             background: none;
@@ -101,19 +105,24 @@
         a,
         span,
         button {
-            font-size: 0.9rem;
+            font-size: var(--font-size-sm);
+            font-weight: var(--font-weight-medium);
             display: flex;
             flex-wrap: nowrap;
             align-items: center;
-            gap: 0.25rem;
+            gap: 0.35rem;
             box-sizing: border-box;
-            padding: 0.35em 0.5em;
+            /* Was 0.35em/0.5em — a tap target under the 44px guideline and
+               visually cramped against the taller header. */
+            padding: 0.55rem 0.7rem;
             margin: 0;
             line-height: 1;
             text-decoration: none;
-            outline: none;
-            border-radius: var(--br);
-            transition: all 0.15s cubic-bezier(0.25, 0.8, 0.25, 1);
+            border-radius: var(--radius-sm);
+            /* `all` also animated layout properties. Only what changes. */
+            transition:
+                background-color var(--speed) var(--ease),
+                color var(--speed) var(--ease);
             background: none;
             border: none;
             cursor: pointer;
@@ -124,7 +133,16 @@
             a,
             span,
             button {
-                color: var(--tertiary-color);
+                color: rgba(255, 255, 255, 0.82);
+            }
+
+            /* Nothing responded to the pointer unless it was already the
+               active page, so the bar felt inert. */
+            &:not(.active):hover a,
+            &:not(.active):hover span,
+            &:not(.active):hover button {
+                background: rgba(255, 255, 255, 0.12);
+                color: #fff;
             }
 
             svg {
@@ -133,19 +151,21 @@
                 visibility: hidden;
             }
 
-            &:hover a,
+            /* Only the current page wears the brand red. Hover shared it
+               before, so pointing at any item made it look selected and the
+               real active item was impossible to pick out while moving the
+               mouse across the bar. Hover is handled above, as a quiet lift. */
             &.active a,
-            &:hover span,
             &.active span,
-            &:hover button,
             &.active button {
-                background: rgba(var(--secondary-rgb), 1);
-                color: var(--tertiary-color);
+                background: var(--secondary-color);
+                color: #fff;
+                font-weight: var(--font-weight-semibold);
             }
 
-            &:hover svg,
-            &.active svg {
-                color: var(--tertiary-color);
+            &.active svg,
+            &:hover svg {
+                color: #fff;
             }
 
             > ul {
@@ -330,11 +350,34 @@
     justify-self: end;
     position: relative;
     z-index: 1000; // Higher z-index to stay above sticky header
+    /* Account actions are a different kind of thing from page navigation, so
+       a hairline separates them rather than leaving one undifferentiated row
+       of targets. */
+    padding-left: var(--spacing-md);
+    margin-left: var(--spacing-sm);
 
     > ul {
         list-style: none;
         display: flex;
-        padding: 0;
+        /* A faint inset holds the icons together as one group rather than
+           leaving them as loose targets floating in the bar.
+           Deliberately count-agnostic: projects mount a different number of
+           actions here, so this is padding and a pill radius rather than
+           anything sized to a fixed set. If a project mounts more than fits,
+           the group scrolls instead of pushing the navigation. */
+        background: rgba(255, 255, 255, 0.1);
+        border-radius: var(--radius-pill);
+        /* An even ring around the chips; any more and the group reads as a
+           separate control rather than a container for them. */
+        padding: 4px;
+        max-width: 100%;
+        overflow-x: auto;
+        scrollbar-width: none;
+
+        &::-webkit-scrollbar {
+            display: none;
+        }
+
         margin: 0;
         flex-wrap: nowrap;
         flex-direction: row;
@@ -356,39 +399,85 @@
             z-index: 1002;
         }
 
-        a {
+        a,
+        button {
             color: var(--tertiary-color);
             box-sizing: border-box;
-            padding: 0.35rem;
             text-decoration: none;
             background: none;
-            border: 1px solid var(--primary-color);
-            border-radius: 100%;
-            transition: all 0.25s cubic-bezier(0.25, 0.8, 0.25, 1); // Reduced duration
+            font: inherit;
+            padding: 0;
+            /* Was a 1px border in the header's own colour — invisible, but it
+               still consumed the ring of space that would have made these
+               read as buttons. */
+            border: none;
+            /* Circular, so each chip nests inside the pill-shaped group at any
+               position — a rounded square cut across the container's curve at
+               the two ends. Also stays correct however many actions a project
+               mounts, since no chip is styled by its position. */
+            border-radius: var(--radius-pill);
+            /* `all` animated layout too; and the hover scale made the icons
+               wobble against a static header. Colour alone is enough feedback
+               on a 36px target. */
+            transition: background-color var(--speed) var(--ease);
             display: flex;
             align-items: center;
             justify-content: center;
-            width: 32px; // Reduced from 36px
-            height: 32px; // Reduced from 36px
+            /* 32px was below the 44px guideline and awkward to hit. */
+            width: 36px;
+            height: 36px;
             position: relative;
             z-index: 1003;
-            transform: translateZ(0) translateX(0) translateY(0); // Force compositor layer
-            will-change: transform, background-color; // Optimize for animations
-            /* Remove contain: layout to prevent rendering optimization conflicts */
-            outline: none;
             appearance: none;
-            backface-visibility: hidden; // Prevent rendering glitches
-            -webkit-backface-visibility: hidden;
-            /* Force onto own layer to prevent culling */
-            isolation: isolate;
 
+            /* Brand red on hover, matching the rest of the bar. A neutral
+               white lift read as a system default rather than a Prime
+               control, and left this cluster looking unrelated to everything
+               around it. */
             &:hover,
             &.active {
-                background: var(--primary-color);
-                transform: translateZ(0) scale(1.05); // Slight scale on hover
+                background: var(--secondary-color);
 
                 svg {
-                    color: var(--secondary-color);
+                    color: #fff !important;
+                }
+            }
+
+            svg {
+                /* Components mounted here (notifications, switch account)
+                   colour their own icons, which is why they hovered
+                   differently from the plain links beside them. */
+                color: var(--tertiary-color);
+            }
+        }
+
+        /* Components mounted into this cluster ship their own trigger styles —
+           the notification bell is 40px with its own hover fill and a scale
+           transform, against 36px flat siblings. Rather than chase each one by
+           name, normalise any button nested inside a wrapper element so the
+           cluster stays uniform however a project fills it. */
+        li > div,
+        li > span {
+            display: flex;
+            align-items: center;
+        }
+
+        li button {
+            width: 36px;
+            height: 36px;
+            min-width: 36px;
+            padding: 0;
+            border-radius: var(--radius-pill) !important;
+            background: none;
+            transform: none !important;
+            color: var(--tertiary-color);
+
+            &:hover {
+                background: var(--secondary-color) !important;
+                color: #fff;
+
+                svg {
+                    color: #fff !important;
                 }
             }
         }

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

@@ -1,3 +1,8 @@
+@use 'controls' as *;
+
+.btn {
+    @include button-primary;
+}
 /* Profile.module.scss */
 
 /* Form field label animation */
@@ -295,33 +300,7 @@ select:focus + .fi__span {
     }
 }
 
-.btn {
-    width: max-content;
-    display: inline-block;
-    position: relative;
-    padding: 0.65rem 1rem;
-    cursor: pointer;
-    font-size: 1rem;
-    color: var(--secondary-color);
-    text-decoration: none;
-    overflow: hidden;
-    background: var(--primary-color);
-    border: 1px solid rgba(var(--primary-rgb), 1.1);
-    border-radius: var(--br);
-    outline: none;
-    transition: all 0.2s cubic-bezier(0.85, 0, 0.15, 1) 0s;
 
-    &:hover:not(:disabled) {
-        color: var(--tertiary-color);
-        background: var(--secondary-color);
-        border: 1px solid rgba(var(--secondary-color-rgb), 1.1);
-    }
-
-    &:disabled {
-        opacity: 0.6;
-        cursor: not-allowed;
-    }
-}
 
 .inputError {
     border-color: red !important;

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

@@ -1,14 +1,17 @@
 .quickActions {
     width: 100%;
-    background: rgba(var(--bg-rgb), 0.98);
-    padding: 0.75rem var(--padding);
-    border-top: solid 3px rgba(var(--primary-rgb), 0.1);
-    border-bottom: solid 1px rgba(var(--primary-rgb), 0.15);
+    background: var(--surface-color);
+    padding: var(--spacing-sm) clamp(var(--spacing-md), 3vw, var(--spacing-xl));
+    /* Was a 3px tinted top rule plus a 1px bottom rule plus a shadow — three
+       separators stacked on one strip, which made it read as a heavier
+       element than the header above it. One hairline is enough. */
+    border-top: none;
+    border-bottom: 1px solid var(--border-color);
     color: var(--paragraph-color);
     display: flex;
     flex-wrap: wrap;
     align-items: center;
-    box-shadow: 0 2px 8px rgba(0, 0, 0, 0.05);
+    box-shadow: none;
     overflow: visible; /* Ensure dropdowns are visible */
     position: relative; /* Create a stacking context */
     z-index: 100; /* Ensure proper stacking */
@@ -20,49 +23,51 @@
         display: flex;
         align-items: center;
         justify-content: center;
-        background-color: rgba(var(--primary-rgb), 0.08);
-        border-radius: 50%;
-        margin-right: 0.75rem;
+        /* A label, not a control — it does nothing when clicked, so a filled
+           circle that lights up on hover was promising an action it does not
+           have. */
+        background: none;
+        border-radius: var(--radius-sm);
+        margin-right: var(--spacing-sm);
+        padding-right: var(--spacing-sm);
         position: relative;
-        color: rgba(var(--primary-rgb), 0.8);
-        transition: all 0.2s ease;
-
-        &:hover {
-            background-color: rgba(var(--primary-rgb), 0.12);
-            transform: scale(1.05);
-        }
+        color: var(--muted-color);
+        flex: none;
 
         &::after {
             content: '';
             position: absolute;
-            right: -0.5rem;
+            right: 0;
             top: 50%;
             transform: translateY(-50%);
             height: 60%;
             width: 1px;
-            background-color: rgba(var(--primary-rgb), 0.15);
+            background-color: var(--border-color);
         }
     }
 
     > button {
         appearance: none;
-        border: 1px solid rgba(var(--primary-rgb), 0.2);
-        border-radius: var(--br, 4px);
-        background: rgba(var(--bg-rgb), 1);
+        border: 1px solid var(--border-color);
+        border-radius: var(--radius-sm);
+        background: var(--surface-color);
         width: max-content;
-        color: var(--paragraph-color);
-        padding: 0.4rem 0.8rem;
-        margin: 0 0.2rem;
-        font-size: 0.85rem;
-        font-weight: 500;
-        transition: all 0.2s ease;
-        box-shadow: 0 1px 3px rgba(0, 0, 0, 0.05);
+        color: var(--text-color);
+        padding: 0 var(--spacing-md);
+        margin: 0;
+        /* Matches the button mixin used everywhere else, so quick actions and
+           page buttons are the same size and weight. */
+        font-size: var(--font-size-sm);
+        font-weight: var(--font-weight-semibold);
+        transition:
+            background-color var(--speed) var(--ease),
+            border-color var(--speed) var(--ease);
+        box-shadow: none;
         display: flex;
         align-items: center;
         justify-content: center;
-        min-height: 32px;
+        min-height: 2.5rem;
         position: relative;
-        overflow: hidden;
 
         &::before {
             content: '';

package/src/components/styles/_controls.scss

@@ -0,0 +1,139 @@
+/**
+ * Shared control definitions.
+ *
+ * `.btn` was copy-pasted into ten stylesheets and had drifted into seven
+ * different definitions — different paddings, font sizes, borders and hover
+ * behaviour, all called the same thing. CSS modules hash each class per file,
+ * so a single global `.btn` cannot reach them; a mixin can, and each file
+ * keeps its own scoped class while the definition lives in one place.
+ *
+ * Every variant shares a height, radius, weight and focus treatment, so they
+ * line up when placed side by side.
+ */
+
+@mixin button-base {
+    display: inline-flex;
+    align-items: center;
+    justify-content: center;
+    gap: 0.4rem;
+    width: max-content;
+    min-height: 2.5rem;
+    padding: 0 var(--spacing-md);
+    border: 1px solid transparent;
+    border-radius: var(--radius-sm);
+    font-family: var(--font-family);
+    // Was 1rem against 15px body text, so buttons shouted slightly.
+    font-size: var(--font-size-sm);
+    font-weight: var(--font-weight-semibold);
+    line-height: 1;
+    text-decoration: none;
+    white-space: nowrap;
+    cursor: pointer;
+    // Was `all 0.2s`, which animates layout properties too and makes hover
+    // feel laggy. Only what actually changes.
+    transition:
+        background-color var(--speed) var(--ease),
+        border-color var(--speed) var(--ease),
+        color var(--speed) var(--ease);
+
+    &:disabled,
+    &[aria-disabled='true'] {
+        opacity: 0.55;
+        cursor: not-allowed;
+    }
+
+    // Focus is handled globally by the app's _focus.scss; nothing here removes
+    // the outline, which is what created the accessibility gap in the first
+    // place.
+}
+
+@mixin button-primary {
+    @include button-base;
+
+    background: var(--primary-color);
+    border-color: var(--primary-color);
+    color: #fff;
+
+    &:hover:not(:disabled) {
+        background: var(--primary-color-darker);
+        border-color: var(--primary-color-darker);
+        // Was a light-blue fill with navy text, which dropped below a legible
+        // contrast on hover — the one state where feedback matters most.
+        color: #fff;
+    }
+}
+
+@mixin button-secondary {
+    @include button-base;
+
+    background: var(--paper);
+    border-color: var(--border-color);
+    color: var(--text-color);
+
+    &:hover:not(:disabled) {
+        border-color: var(--border-color-dark);
+        background: var(--hover-color);
+    }
+}
+
+@mixin button-ghost {
+    @include button-base;
+
+    background: none;
+    border-color: transparent;
+    color: var(--primary-color);
+
+    &:hover:not(:disabled) {
+        background: var(--primary-color-light);
+    }
+}
+
+@mixin button-danger {
+    @include button-base;
+
+    background: none;
+    border-color: transparent;
+    color: var(--danger-color);
+
+    &:hover:not(:disabled) {
+        background: var(--danger-light);
+        border-color: var(--danger-color);
+    }
+}
+
+@mixin button-small {
+    min-height: 2rem;
+    padding: 0 var(--spacing-sm);
+    font-size: var(--font-size-xs);
+}
+
+/* --- Inputs -------------------------------------------------------------- */
+
+@mixin input-base {
+    width: 100%;
+    min-height: 2.5rem;
+    padding: 0 0.65rem;
+    border: 1px solid var(--border-color);
+    border-radius: var(--radius-sm);
+    background: var(--paper);
+    font-family: var(--font-family);
+    font-size: var(--font-size-md);
+    color: var(--text-color);
+    transition:
+        border-color var(--speed) var(--ease),
+        box-shadow var(--speed) var(--ease);
+
+    &::placeholder {
+        color: var(--muted-color);
+    }
+
+    &:hover:not(:disabled) {
+        border-color: var(--border-color-dark);
+    }
+
+    &:disabled {
+        background: var(--bg-color-secondary);
+        color: var(--disabled-color);
+        cursor: not-allowed;
+    }
+}

package/src/components/styles/global.css

@@ -10,24 +10,26 @@
 
 html,
 body {
-    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto,
-        Helvetica, Arial, sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji',
-        'Segoe UI Symbol';
+    /* Was a hardcoded system stack, which overrode the app's font because this
+       file is injected after the entry stylesheet. Through the token it
+       follows whatever the app defines. */
+    font-family: var(--font-family, system-ui, sans-serif);
     height: 100%;
-    text-rendering: geometricPrecision;
-    font-stretch: 75% 125%;
+    text-rendering: optimizeLegibility;
 }
 
 body {
     background: var(--bg-color);
     font-weight: var(--para-weight);
-    font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto,
-        Helvetica, Arial, sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji',
-        'Segoe UI Symbol';
+    font-family: var(--font-family, system-ui, sans-serif);
     font-style: normal;
     overflow-x: hidden;
     overscroll-behavior: none;
-    font-size: 0.85rem;
+    /* Was 0.85rem (13.6px) — small enough that every component had to size up
+       from it. Now the body step of the scale. */
+    font-size: var(--font-size-md, 0.9375rem);
+    line-height: var(--para-height, 1.6);
+    color: var(--text-color, #1f2a37);
 }
 
 input,
@@ -130,24 +132,61 @@ select {
     appearance: none;
 }
 
+/* A real scale. These carried colour and weight but no size, so every heading
+   fell back to the browser default — a hierarchy inherited rather than chosen,
+   and unrelated to the body size beneath it. Sizes come from the tokens, with
+   fallbacks so this file still stands alone. */
 h1,
 h2,
 h3,
 h4,
 h5,
 h6 {
-    color: var(--header-color);
-    font-weight: var(--title-weight);
-    line-height: var(--title-height);
-    letter-spacing: 0.015rem;
+    color: var(--heading-color, #10192a);
+    font-weight: var(--font-weight-bold, 700);
+    line-height: var(--title-height, 1.25);
+    /* Was +0.015rem. Barlow opens up as it scales, so the display sizes want
+       pulling in, not pushing apart. */
+    letter-spacing: -0.01em;
+    margin: 0 0 var(--spacing-sm, 0.5rem);
+}
+
+h1 {
+    font-size: var(--font-size-2xl, 1.75rem);
+    letter-spacing: -0.02em;
+}
+
+h2 {
+    font-size: var(--font-size-xl, 1.375rem);
+}
+
+h3 {
+    font-size: var(--font-size-lg, 1.125rem);
+}
+
+h4 {
+    font-size: var(--font-size-md, 0.9375rem);
+    font-weight: var(--font-weight-semibold, 600);
+    letter-spacing: 0;
+}
+
+h5,
+h6 {
+    font-size: var(--font-size-sm, 0.8125rem);
+    font-weight: var(--font-weight-semibold, 600);
+    letter-spacing: 0.02em;
+    text-transform: uppercase;
+    color: var(--muted-color, #6b7688);
 }
 
 p {
-    font-size: clamp(1rem, 2vw, 1.25rem);
+    /* Was clamp(1rem, 2vw, 1.25rem) — up to 20px, larger than the body text
+       around it, which inverted the hierarchy on every screen. */
+    font-size: inherit;
     color: var(--paragraph-color);
     font-weight: var(--para-weight);
     line-height: var(--para-height);
-    margin-bottom: 1rem;
+    margin-bottom: var(--spacing-md, 1rem);
 
     @media (max-width: 1024px) {
         font-size: clamp(0.85rem, 2vw, 0.95rem);