@visns-studio/visns-components 6.3.15 → 6.3.17

package/package.json

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

package/src/components/Notification.jsx

@@ -239,7 +239,14 @@ function Notification(props) {
             </button>
 
             {show && (
-                <div className={styles.dropdownWrapper} ref={wrapperRef}>
+                <div
+                    className={styles.dropdownWrapper}
+                    ref={wrapperRef}
+                    /* Tells a scrollable host — the navigation's action pill —
+                       to stop clipping while this is open. See
+                       Navigation.module.scss `.hactions > ul`. */
+                    data-nav-overlay="open"
+                >
                     <div className={styles.notibox}>
                         <div className={styles.notiboxHeader}>
                             <h3>Notifications</h3>

package/src/components/SwitchAccount.jsx

@@ -87,7 +87,9 @@ function SwitchAccount({ setSystemAuth, setting, userProfile }) {
             </a>
 
             {show && (
-                <div ref={wrapperRef}>
+                /* `data-nav-overlay` stops the navigation's scrollable action
+                   pill from clipping this list. See Navigation.module.scss. */
+                <div ref={wrapperRef} data-nav-overlay="open">
                     {data.length > 0 ? (
                         <div className={styles.notibox}>
                             {data.map((item) => (

package/src/components/auth/Profile.jsx

@@ -480,9 +480,20 @@ const Profile = ({ userProfile, setUserProfile }) => {
                                     label="Password"
                                     error={inputClasses.password}
                                 >
-                                    <PasswordStrengthBar
-                                        password={profile.password}
-                                    />
+                                    {/* Only once there is something to
+                                        measure. Rendered unconditionally it
+                                        sat under an empty password box
+                                        reporting "too short" before the user
+                                        had typed a character. */}
+                                    {profile.password ? (
+                                        <PasswordStrengthBar
+                                            password={profile.password}
+                                            className={styles.strengthMeter}
+                                            scoreWordClassName={
+                                                styles.strengthWord
+                                            }
+                                        />
+                                    ) : null}
                                 </FormField>
 
                                 <FormField
@@ -494,10 +505,12 @@ const Profile = ({ userProfile, setUserProfile }) => {
                                     error={inputClasses.password_confirmation}
                                 />
 
-                                <div className={styles.formItem}>
-                                    <label className={styles.fi__label}>
+                                <div
+                                    className={`${styles.formItem} ${styles.formItemFull}`}
+                                >
+                                    <div className={styles.signatureField}>
                                         <span
-                                            className={`${styles.fi__span} ${styles.prefocus}`}
+                                            className={styles.signatureLabel}
                                         >
                                             Signature
                                         </span>
@@ -516,31 +529,38 @@ const Profile = ({ userProfile, setUserProfile }) => {
                                                 }}
                                                 redrawOnResize
                                                 canvasProps={{
+                                                    /* The frame lives on
+                                                       `.sigcanvas` now. Its
+                                                       geometry is unchanged —
+                                                       border and padding move
+                                                       where the ink lands. */
                                                     style: {
-                                                        border: '2px solid #ccc',
-                                                        borderRadius: '10px',
-                                                        padding: '10px',
                                                         width: '100%',
                                                     },
                                                     className: styles.sigcanvas,
                                                 }}
                                             />
-                                            <small
-                                                style={{
-                                                    display: 'block',
-                                                    marginTop: '5px',
-                                                    color: '#666',
-                                                }}
-                                            >
-                                                Draw your signature above
-                                            </small>
                                         </div>
-                                    </label>
+                                        <small
+                                            className={styles.signatureHint}
+                                        >
+                                            Draw your signature above
+                                        </small>
+                                    </div>
                                 </div>
                             </form>
                         </div>
 
+                        {/* The form card's footer. Secondary before primary. */}
                         <div className={styles.polActions}>
+                            <button
+                                className={`${styles.btn} ${styles.btnSecondary}`}
+                                onClick={handleClearSignature}
+                                disabled={isLoading}
+                                type="button"
+                            >
+                                Clear Signature
+                            </button>
                             <button
                                 className={`${styles.btn} ${
                                     isLoading ? styles.loading : ''
@@ -551,14 +571,6 @@ const Profile = ({ userProfile, setUserProfile }) => {
                             >
                                 {isLoading ? 'Saving...' : 'Save'}
                             </button>
-                            <button
-                                className={styles.btn}
-                                onClick={handleClearSignature}
-                                disabled={isLoading}
-                                type="button"
-                            >
-                                Clear Signature
-                            </button>
                         </div>
                     </div>
                 );

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

@@ -425,6 +425,21 @@
             display: none;
         }
 
+        /* A scroll container clips on BOTH axes: `overflow-x: auto` makes the
+           computed `overflow-y` `auto` as well, however `visible` it was
+           written. So the popovers that mounted components anchor to their own
+           chip — the notification dropdown, the account switcher list, both
+           `position: absolute; top: 100%` — were being cut off at the bottom of
+           this ~44px pill and never appeared on screen. The bell looked dead.
+
+           While one of those overlays is open the group stops scrolling and
+           lets it out. The overlays are only in the DOM while open, so they
+           advertise themselves with `data-nav-overlay` and nothing here has to
+           know which component owns them or track any open state. */
+        &:has([data-nav-overlay='open']) {
+            overflow: visible;
+        }
+
         margin: 0;
         flex-wrap: nowrap;
         flex-direction: row;

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

@@ -115,7 +115,10 @@ select:focus + .fi__span {
     border-radius: var(--br);
     border: 1px solid rgba(var(--primary-rgb), 0.05);
     box-sizing: border-box;
-    padding: 2px;
+    /* Was 2px, which left the section header's hairline floating a couple of
+       pixels short of the panel edge. The panel is a card; its header, body and
+       footer own their own padding, so the card itself has none. */
+    padding: 0;
     margin: 8px 0;
     height: auto;
     position: relative;
@@ -128,21 +131,45 @@ select:focus + .fi__span {
     margin: 0 0 30px 0;
 }
 
+/* The panel's section header. It follows the same spec as the unified page
+   header documented at the top of GenericIndex.module.scss — the panel's own
+   surface, one bottom hairline, the title left-aligned on the same padding as
+   the body beneath it — a step down in type size because this names a section
+   inside a page rather than the page.
+
+   It used to be a pale centred band with its own fill and radius: a second
+   card sitting on top of the card, and the only centred heading in the
+   library. */
 .gridtxt__header {
     width: 100%;
-    display: block;
-    background: rgba(var(--primary-rgb), 0.05);
+    display: flex;
+    align-items: center;
+    justify-content: space-between;
+    gap: var(--spacing-md, 1rem);
     box-sizing: border-box;
-    padding: 10px 20px;
-    border-radius: var(--br);
+    padding: var(--spacing-md, 1rem);
+    background: none;
+    border-bottom: 1px solid var(--border-color, #e1e1e1);
+    border-radius: 0;
     margin-bottom: 0;
-    font-weight: 700;
-    text-align: center;
+    text-align: left;
     color: var(--paragraph-color);
 }
 
 .gridtxt__header span {
     font-weight: 700;
+    /* One step under the page title above it, so the two read as a hierarchy
+       rather than as two competing headings. */
+    font-size: var(--font-size-lg, 1.125rem);
+    line-height: 1.25;
+}
+
+.gridtxt {
+    /* One column width, shared by the form body and the footer under it, so
+       Save lands beneath the right edge of the fields instead of out at the
+       card's edge. A project can retune it from `:root` like the rest of the
+       library's density hooks. */
+    --profile-form-width: 56rem;
 }
 
 .gridtxt > ul {
@@ -180,24 +207,42 @@ select:focus + .fi__span {
 
 .formcontainer {
     width: 100%;
+    /* Two columns across the full width of a 1600px panel gave every field a
+       ~600px line — far past the point where a name box reads as a box. The
+       form takes the width it needs and the rest of the panel stays quiet.
+       The 2FA tab is a document rather than a form and opts out with its own
+       `max-width: 100%`. */
+    max-width: var(--profile-form-width, 56rem);
+    box-sizing: border-box;
 }
 
 .modalForm {
     width: 100%;
-    display: flex;
-    align-items: flex-start;
-    flex-wrap: wrap;
+    box-sizing: border-box;
+    /* Was flex-wrap with 50%-wide children, so a field that needed the full
+       row had no way to say so and the row it started stayed half empty. */
+    display: grid;
+    grid-template-columns: repeat(2, minmax(0, 1fr));
+    align-items: start;
+    gap: var(--spacing-lg, 1.5rem) var(--spacing-md, 1rem);
+    padding: var(--spacing-md, 1rem);
+
+    @media (max-width: 768px) {
+        grid-template-columns: 1fr;
+    }
 }
 
 .formItem {
-    width: 50%;
-    padding: 0.35em;
+    width: auto;
+    padding: 0;
     position: relative;
     box-sizing: border-box;
+}
 
-    @media (max-width: 768px) {
-        width: 100%;
-    }
+/* A field that spans the row — the signature, which is a canvas rather than a
+   line of text and does not pair with anything beside it. */
+.formItemFull {
+    grid-column: 1 / -1;
 }
 
 .fi__label {
@@ -230,7 +275,10 @@ select:focus + .fi__span {
 .select {
     background: var(--item-color);
     border-radius: var(--br);
-    border: 1px solid transparent;
+    /* Was transparent: on a white panel the fields had no edge at all and the
+       form read as text floating in space. The width stays 1px so the floating
+       label, which is positioned against this box, does not move. */
+    border: 1px solid var(--border-color, #e1e1e1);
     padding: 1rem;
     outline: none;
     width: 100%;
@@ -252,52 +300,105 @@ select:focus + .fi__span {
     border: 1px solid rgba(var(--highlight-color-rgb), 0.85);
 }
 
+/* --- Password strength ---------------------------------------------------
+   `react-password-strength-bar` styles its own markup inline, so a class can
+   only reach it with `!important` — that is what every rule below is for, and
+   the only reason for it. The bar itself is four flex children; the score word
+   is the `<p>` after them.
+
+   The meter is rendered only once the field has a value (see Profile.jsx), so
+   nothing here has to hide the "too short" state that used to sit under an
+   empty password box telling the user off before they had typed. */
+.strengthMeter {
+    /* Sits directly under the input it measures, close enough to read as part
+       of the field rather than as a rule floating beneath it. */
+    margin-top: 0.35rem;
+
+    /* The segments: 2px hairlines with 4px of white between them read as four
+       unrelated underlines. A little more weight and a rounded cap makes them
+       one meter. */
+    div > div {
+        height: 4px !important;
+        border-radius: var(--radius-pill, 999px) !important;
+    }
+}
+
+.strengthWord {
+    margin: 0.3rem 0 0 !important;
+    font-size: var(--font-size-xs, 0.75rem) !important;
+    line-height: 1.2 !important;
+    color: var(--muted-color, #6b7688) !important;
+    text-align: right !important;
+}
+
+/* --- Signature ------------------------------------------------------------
+   The canvas geometry (2px border, 10px padding) is deliberately unchanged:
+   signature_pad maps pointer coordinates through the element's bounding rect,
+   so border and padding shift where the ink lands. Only the colour and radius
+   are tokenised, and the frame moved off the inline `canvasProps.style` onto
+   this class where a project can reach it. */
 .sigcanvas {
-    border: 2px solid #ccc;
-    border-radius: 10px;
+    border: 2px solid var(--border-color, #ccc);
+    border-radius: var(--radius-md, 10px);
     padding: 10px;
+    box-sizing: border-box;
 }
 
-/* Removed signatureContainer and signatureLabel styles as they're no longer needed */
+.signatureField {
+    display: flex;
+    flex-direction: column;
+    gap: 0.4rem;
+    /* A signature is a fixed-size mark, not a field that benefits from more
+       room. Left to fill the row it became the largest thing on the page. */
+    max-width: 32rem;
+}
+
+/* The label was a floating `.fi__span`, absolutely positioned so it landed on
+   top of the canvas border — the treatment belongs to inputs that have a value
+   to sit above. A canvas has none, so this is an ordinary label. */
+.signatureLabel {
+    font-size: var(--font-size-sm, 0.8125rem);
+    font-weight: var(--font-weight-semibold, 600);
+    color: var(--paragraph-color);
+}
 
 .signaturePadWrapper {
     width: 100%;
-    margin-bottom: 0.5rem;
     position: relative;
     z-index: 1;
 
     canvas {
         max-width: 100%;
         height: auto !important;
+        display: block;
     }
 }
 
+.signatureHint {
+    font-size: var(--font-size-xs, 0.75rem);
+    color: var(--muted-color, #666);
+}
+
+/* The form card's footer. This was `position: fixed` in the bottom-right
+   corner of the viewport: the page's Save button floated over whatever the
+   user had scrolled to, unattached to the form it saved. It is now the bottom
+   edge of the card, on the same hairline-and-padding spec as the header at the
+   top of it. Secondary before primary, so the destructive-ish action is not
+   the one under the thumb. */
 .polActions {
-    position: fixed;
-    bottom: 20px;
-    right: 20px;
-    width: auto;
     display: flex;
-    gap: 0.5em;
-    z-index: 10;
-
-    @media (max-width: 768px) {
-        position: static;
-        margin: 1.5rem 0;
-        justify-content: flex-end;
-        width: 100%;
-    }
+    align-items: center;
+    justify-content: flex-end;
+    gap: var(--spacing-sm, 0.5rem);
+    width: 100%;
+    max-width: var(--profile-form-width, 56rem);
+    box-sizing: border-box;
+    padding: var(--spacing-md, 1rem);
+    border-top: 1px solid var(--border-color, #e1e1e1);
 }
 
 .polActions button {
-    padding: 0.65em 1em;
     margin: 0;
-    transition: all 0.2s ease;
-
-    &:disabled {
-        opacity: 0.6;
-        cursor: not-allowed;
-    }
 }
 
 
@@ -333,7 +434,7 @@ select:focus + .fi__span {
 
 /* Two-Factor Authentication Styles */
 .twoFactorSection {
-    padding: 0.75rem;
+    padding: var(--spacing-md, 1rem);
     width: 100%;
     display: flex;
     flex-direction: column;
@@ -356,79 +457,76 @@ select:focus + .fi__span {
         line-height: 1.4;
         color: var(--paragraph-color);
         font-size: 0.9rem;
+        /* This tab is deliberately full width — it holds a QR code and a grid
+           of recovery codes — but the prose explaining 2FA ran the whole way
+           across a 1600px panel. Only the paragraphs are capped. */
+        max-width: 68ch;
     }
 }
 
 .statusEnabled {
-    color: #2ecc71;
+    color: var(--success-color, #2ecc71);
     font-weight: 500;
     padding: 0.2rem 0.4rem;
-    background-color: rgba(46, 204, 113, 0.1);
-    border-radius: 4px;
+    background-color: var(--success-color-light, rgba(46, 204, 113, 0.1));
+    border-radius: var(--radius-sm, 4px);
     font-size: 0.85rem;
 }
 
 .statusDisabled {
-    color: #e74c3c;
+    color: var(--danger-color, #e74c3c);
     font-weight: 500;
     padding: 0.2rem 0.4rem;
-    background-color: rgba(231, 76, 60, 0.1);
-    border-radius: 4px;
+    background-color: var(--danger-light, rgba(231, 76, 60, 0.1));
+    border-radius: var(--radius-sm, 4px);
     font-size: 0.85rem;
 }
 
 .confirmedMessage {
     margin: 1rem 0;
     padding: 0.75rem;
-    background-color: rgba(46, 204, 113, 0.1);
-    border-left: 3px solid #2ecc71;
+    background-color: var(--success-color-light, rgba(46, 204, 113, 0.1));
+    border-left: 3px solid var(--success-color, #2ecc71);
     border-radius: var(--br);
     color: #333;
     font-size: 0.9rem;
 }
 
+/* The three variants each carried their own font size and padding, so a row of
+   them next to `.btn` — which comes from the shared `_controls` mixins — sat a
+   few pixels short and half a step smaller. They are colour only now; the
+   shape comes from the one place that defines it. */
 .btnPrimary {
-    background-color: var(--primary-color);
-    color: white;
-    font-size: 0.85rem;
-    padding: 0.5rem 0.85rem;
-
-    &:hover:not(:disabled) {
-        background-color: rgba(var(--primary-rgb), 0.9);
-    }
+    @include button-primary;
 }
 
 .btnSecondary {
-    background-color: #f1f1f1;
-    color: #333;
-    border: 1px solid #ddd;
-    font-size: 0.85rem;
-    padding: 0.5rem 0.85rem;
-
-    &:hover:not(:disabled) {
-        background-color: var(--bg-color, #f5f3ef);
-        border-color: var(--border-color, #e1e1e1);
-    }
+    @include button-secondary;
 }
 
 .btnDanger {
-    background-color: #e74c3c;
-    color: white;
-    font-size: 0.85rem;
-    padding: 0.5rem 0.85rem;
+    @include button-base;
+
+    background: var(--danger-color, #e74c3c);
+    border-color: var(--danger-color, #e74c3c);
+    color: #fff;
 
     &:hover:not(:disabled) {
-        background-color: rgba(231, 76, 60, 0.85);
+        /* Deliberately still a solid fill rather than the library's ghost
+           `button-danger`: this is the control that turns a second factor off,
+           and it should not read as quieter than the button that turned it
+           on. */
+        filter: brightness(0.9);
     }
 }
 
 .twoFactorSetup {
     margin-top: 1.5rem;
     padding: 1.5rem;
-    background-color: white;
+    background-color: var(--paper, #fff);
     border-radius: var(--br);
-    border: 1px solid #e0e0e0;
-    box-shadow: 0 3px 8px rgba(0, 0, 0, 0.05);
+    border: 1px solid var(--border-color, #e0e0e0);
+    box-shadow: var(--shadow-sm, 0 3px 8px rgba(0, 0, 0, 0.05));
     width: 100%;
 
     h3 {
@@ -437,7 +535,7 @@ select:focus + .fi__span {
         font-weight: 500;
         color: var(--primary-color);
         padding-bottom: 0.75rem;
-        border-bottom: 1px solid #f0f0f0;
+        border-bottom: 1px solid var(--border-color, #f0f0f0);
     }
 }
 
@@ -531,17 +629,14 @@ select:focus + .fi__span {
     justify-content: center;
     margin: 0.75rem 0;
     padding: 1rem;
-    background-color: white;
+    background-color: var(--paper, #fff);
     border-radius: var(--br);
-    border: 1px solid #e0e0e0;
+    border: 1px solid var(--border-color, #e0e0e0);
     width: 100%;
-    box-shadow: 0 2px 6px rgba(0, 0, 0, 0.08);
-    transition: transform 0.2s ease, box-shadow 0.2s ease;
-
-    &:hover {
-        transform: translateY(-2px);
-        box-shadow: 0 4px 8px rgba(0, 0, 0, 0.12);
-    }
+    /* The panel used to lift and deepen its shadow on hover. Nothing here is
+       clickable — it is a QR code to point a phone at — so the affordance was
+       promising an interaction that does not exist. */
+    box-shadow: var(--shadow-sm, 0 2px 6px rgba(0, 0, 0, 0.08));
 }
 
 .qrCodeSvg {
@@ -588,7 +683,7 @@ select:focus + .fi__span {
     background-color: #f9f9f9;
     padding: 0.75rem;
     border-radius: var(--br);
-    border: 1px solid #e0e0e0;
+    border: 1px solid var(--border-color, #e0e0e0);
     width: 100%;
 }
 
@@ -635,7 +730,7 @@ select:focus + .fi__span {
     background-color: #f9f9f9;
     padding: 1rem;
     border-radius: var(--br);
-    border: 1px solid #e0e0e0;
+    border: 1px solid var(--border-color, #e0e0e0);
     width: 100%;
 }
 
@@ -670,7 +765,7 @@ select:focus + .fi__span {
     padding: 1rem;
     background-color: #f9f9f9;
     border-radius: var(--br);
-    border: 1px solid #e0e0e0;
+    border: 1px solid var(--border-color, #e0e0e0);
     width: 100%;
 }
 
@@ -710,8 +805,8 @@ select:focus + .fi__span {
 .recoveryCodesWarning {
     margin-top: 0.75rem;
     padding: 0.75rem;
-    background-color: rgba(231, 76, 60, 0.1);
-    border-left: 3px solid #e74c3c;
+    background-color: var(--danger-light, rgba(231, 76, 60, 0.1));
+    border-left: 3px solid var(--danger-color, #e74c3c);
     border-radius: var(--br);
     color: #333;
     font-size: 0.85rem;
@@ -722,7 +817,7 @@ select:focus + .fi__span {
     justify-content: flex-end;
     margin-top: 1.5rem;
     padding-top: 1rem;
-    border-top: 1px solid #f0f0f0;
+    border-top: 1px solid var(--border-color, #f0f0f0);
 }
 
 .crmtitle {

package/src/components/styles/global.css

@@ -39,6 +39,16 @@ select {
     font: inherit;
 }
 
+/* Floor under every textarea: the browser default (`rows` omitted) is two
+   lines, which renders a multi-line field as a slit. Three lines of text
+   plus the field's own padding is the least a free-text box can be and
+   still read as one. A bare element selector, so any component that sets
+   its own height with a class still wins. */
+textarea {
+    min-height: 5.5em; /* ~3 lines + padding, for browsers without lh */
+    min-height: calc(3lh + 2rem);
+}
+
 /* Global radio button styling */
 input[type='radio'],
 .radio-input {