npm - @brightspot/ui - Versions diffs - 6.3.2 → 6.3.3-pr.337.1 - Mend

@brightspot/ui 6.3.2 → 6.3.3-pr.337.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (24) hide show

package/README.md CHANGED Viewed

@@ -47,6 +47,8 @@ Now edit anything in `src/legacy/tool-ui/src/` and the browser updates on its ow
 _Seeing `502` errors? The CMS backend isn't reachable — check the docker step._
+_Seeing `The 'btu-…' class does not exist` after a pull? Run `yarn install` here. New utilities can arrive with new dependency versions — icon names come from `lucide-static` — and a pull updates the code without updating what's installed. Running it at the repo root installs the `src/legacy/tool-ui/` workspace too, so one command covers both._
 ### Full way — build the real package (for a final check only)
 Slower, but produces the actual file the CMS ships. Needs two temporary edits in `brightspot` — **don't commit them**.

package/dist/storybook/project.json CHANGED Viewed

	@@ -1 +1 @@
1	- {"generatedAt":~~1788974870315~~,"hasCustomBabel":false,"hasCustomWebpack":false,"hasStaticDirs":true,"hasStorybookEslint":true,"refCount":0,"knownPackages":{"testPackages":{"@storybook/addon-vitest":"10.2.12","@vitest/browser-playwright":"4.0.18","playwright":"1.58.2","vitest":"4.0.18"},"stylingPackages":{"tailwindcss":"^3.4.13","autoprefixer":"10.4.21"}},"hasRouterPackage":false,"packageManager":{"type":"yarn","agent":"yarn","nodeLinker":"undefined"},"preview":{"usesGlobals":true},"framework":{"name":"@storybook/web-components-vite","options":{}},"builder":"@storybook/builder-vite","renderer":"@storybook/web-components","portableStoriesFileCount":0,"applicationFileCount":6,"storybookVersion":"10.2.12","storybookVersionSpecifier":"10.2.12","language":"typescript","storybookPackages":{"@storybook/addon-themes":{"version":"10.2.12"},"@storybook/web-components-vite":{"version":"10.2.12"},"@wc-toolkit/storybook-helpers":{"version":"10.0.0"},"eslint-plugin-storybook":{"version":"10.0.7"},"storybook":{}},"addons":{"@storybook/addon-docs":{"version":"10.2.12"},"@storybook/addon-vitest":{"version":"10.2.12"}},"packageJsonType":"module"}
1	+ {"generatedAt":1789570944348,"hasCustomBabel":false,"hasCustomWebpack":false,"hasStaticDirs":true,"hasStorybookEslint":true,"refCount":0,"knownPackages":{"testPackages":{"@storybook/addon-vitest":"10.2.12","@vitest/browser-playwright":"4.0.18","playwright":"1.58.2","vitest":"4.0.18"},"stylingPackages":{"tailwindcss":"^3.4.13","autoprefixer":"10.4.21"}},"hasRouterPackage":false,"packageManager":{"type":"yarn","agent":"yarn","nodeLinker":"undefined"},"preview":{"usesGlobals":true},"framework":{"name":"@storybook/web-components-vite","options":{}},"builder":"@storybook/builder-vite","renderer":"@storybook/web-components","portableStoriesFileCount":0,"applicationFileCount":6,"storybookVersion":"10.2.12","storybookVersionSpecifier":"10.2.12","language":"typescript","storybookPackages":{"@storybook/addon-themes":{"version":"10.2.12"},"@storybook/web-components-vite":{"version":"10.2.12"},"@wc-toolkit/storybook-helpers":{"version":"10.0.0"},"eslint-plugin-storybook":{"version":"10.0.7"},"storybook":{}},"addons":{"@storybook/addon-docs":{"version":"10.2.12"},"@storybook/addon-vitest":{"version":"10.2.12"}},"packageJsonType":"module"}

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@brightspot/ui",
-  "version": "6.3.2",
+  "version": "6.3.3-pr.337.1",
   "type": "module",
   "license": "UNLICENSED",
   "description": "A UI library for building Brightspot CMS components.",

package/src/legacy/tool-ui/src/Conversation.ts CHANGED Viewed

@@ -424,6 +424,9 @@ onFind('.frame[name^=conversation-edit-post].loaded', (widget: HTMLElement) => {
 onFind('.Conversation-add-comment', (button) => {
   button.addEventListener('click', (event) => {
+    // This has no href, so keep the frame plugin from treating it as a link.
+    event.stopPropagation()
     const workflowBody = button.closest('.ContentEdit-workflowBody')
     if (workflowBody) {
       workflowBody

package/src/legacy/tool-ui/src/Entry.ts CHANGED Viewed

@@ -834,34 +834,6 @@ onVisible('.Deferred', (deferred) => {
   }
 })
-/* Frame JS removes the Widget class from frames so we need to re-add it; only target the popup's own
-   frame so nested frames (e.g. the content edit preview) keep their regular styling */
-onFind(
-  [
-    '.Popup[name="edit-guide"] .frame',
-    '.Popup[name="PostPublishActionsPage"] .frame',
-    '.Popup[name^="create-new-object-"] > .Popup-content > .frame',
-    '.Popup[name^="objectId-create-"] > .Popup-content > .frame',
-    '.Popup[name="AddEmbeddedContent"] .frame',
-    '.Popup[name^="conversation-"] .frame',
-    '.Popup[name^="_copy"] > .Popup-content > .frame',
-    '.Popup[name="DateStringCalendarFrame"] .frame',
-  ],
-  (frame: HTMLElement) => {
-    const config = { attributeFilter: ['class'] }
-    const observer = new MutationObserver((mutations) => {
-      mutations.forEach((mutation) => {
-        const target = mutation.target as HTMLElement
-        if (!target.classList.contains('Widget')) {
-          target.classList.add('Widget')
-        }
-      })
-    })
-    observer.observe(frame, config)
-  },
-)
 // Handle Content Edit left pane scrolling to hide and collapse the top bar.
 // Note: Scroll event is used instead of IntersectionObserver because of dynamic content in scroll area.
 onFind('.ContentEdit-left', (leftPane: HTMLElement) => {
@@ -935,7 +907,10 @@ onFind('.Widget-controls', (controls: HTMLElement) => {
   const widget = controls.closest<HTMLElement>('.Widget')
   if (!widget) return
   const update = () =>
-    widget.classList.toggle('has-widgetControls', controls.childNodes.length > 0)
+    widget.classList.toggle(
+      'has-widgetControls',
+      controls.childNodes.length > 0,
+    )
   update()
   new MutationObserver(update).observe(controls, { childList: true })
 })

package/src/legacy/tool-ui/src/ProseMirror.css CHANGED Viewed

@@ -8,6 +8,11 @@
   @apply h-1 overflow-hidden;
 }
+/* Tailwind's preflight makes img a block, which gives ProseMirror's cursor hack a line of its own. */
+.ProseMirror img.ProseMirror-separator {
+  display: inline;
+}
 /* styling enhanced text as it appears in the editor */
 .ProseMirror {
   .rte2-style-bold {

package/src/legacy/tool-ui/src/RepeatableContentSelector.css CHANGED Viewed

@@ -44,8 +44,8 @@
     @apply before:ms-2 before:flex before:size-5 before:cursor-move before:justify-center before:text-sm before:text-gray-500 before:content-[counter(RCS-item)];
   }
-  /* Hide item until the multisort button gets added from Sortable.js */
-  &.Sortable-item:not(:has(.MultiSortable-buttonToggle), .Sortable-ghost) {
+  /* Hide item until Sortable.js has finished initializing it */
+  &.Sortable-item:not(.is-sortableReady, .Sortable-ghost) {
     @apply opacity-0;
   }
 }

package/src/legacy/tool-ui/src/SearchResult.css CHANGED Viewed

@@ -434,6 +434,11 @@
   .SearchResult-fullscreen {
     @apply btu-button btu-button-text-hidden btu-button-gray me-1 ms-3 overflow-visible bg-white opacity-[var(--controls-opacity,_1)] transition-opacity [grid-area:fullscreen] before:btu-icon before:btu-icon-maximize after:absolute after:right-[43px] after:h-full after:w-px after:bg-gray-200 hover:bg-gray-100;
   }
+  .SearchResult-actions-toggle.is-loading,
+  .SearchResult-fullscreen.is-loading {
+    @apply before:btu-icon-loader before:animate-spin;
+  }
 }
 /* Shown when user toggles Summary fiels in "Select Fields" menu */

package/src/legacy/tool-ui/src/Toggleable/index.ts CHANGED Viewed

@@ -37,6 +37,16 @@ export default class Toggleable extends EventEmitterMixin(
     this._handleChange()
   }
+  /*
+    Re-applies the current selection's show/hide + enable/disable state.
+    Called by an ancestor Toggleable after its enable pass, which would
+    otherwise leave this toggleable's toggled-off branch inputs enabled
+    and submitting stale values (BSP-19758).
+  */
+  refresh() {
+    this._handleChange()
+  }
   constructor() {
     super()
     this._forms = []
@@ -177,8 +187,8 @@ export default class Toggleable extends EventEmitterMixin(
                 body: data,
               })
-              form.element.innerHTML = await response.text()
               if (response.ok) {
+                form.element.innerHTML = await response.text()
                 delete form.element.dataset.formFieldsUrl
                 delete form.element.dataset.formFieldsData
@@ -187,6 +197,11 @@ export default class Toggleable extends EventEmitterMixin(
                 )
                 form.element.dispatchEvent(new Event('load'))
                 form.element.dispatchEvent(new UIEvent('resize'))
+                // The selection may have changed while the request was in
+                // flight; re-apply it so content that arrived for a branch
+                // that is no longer selected doesn't come back enabled.
+                this._handleChange()
               } else {
                 form.element.dispatchEvent(
                   new CustomEvent('error', {
@@ -229,6 +244,27 @@ export default class Toggleable extends EventEmitterMixin(
       inputs.forEach((input) => {
         input.toggleAttribute('disabled', disable)
       })
+      if (!disable) {
+        // The enable pass above walks the whole subtree, so it also enabled
+        // inputs in nested toggleables' toggled-off branches (BSP-19758:
+        // duplicate typeId params flip a polymorphic record to the wrong
+        // type on save). Re-apply each nested toggleable's own state, after
+        // the enable pass, to disable those branches again. Document order
+        // guarantees an outer nested toggleable is refreshed before any
+        // toggleable inside its branches.
+        divsToToggle.forEach((el) => {
+          el.querySelectorAll<Toggleable>('btu-toggleable').forEach(
+            (nested) => {
+              // A not-yet-upgraded element applies its own state when it
+              // connects, so it can safely be skipped here.
+              if (nested !== this && nested instanceof Toggleable) {
+                nested.refresh()
+              }
+            },
+          )
+        })
+      }
     }
   }

package/src/legacy/tool-ui/src/main/webapp/dist/2111.99de62ae687524cfc313.css ADDED Viewed

@@ -0,0 +1 @@

+ .ProseMirror{--ProseMirror-numLines:10;min-height:calc(var(--ProseMirror-numLines) * 1.25rem);white-space:pre-wrap;line-height:normal}[dir] .ProseMirror{padding-top:var(--lineHeight-m);padding-bottom:var(--lineHeight-m)}[dir=ltr] .ProseMirror,[dir=rtl] .ProseMirror{border-bottom-right-radius:.5rem;border-bottom-left-radius:.5rem;padding-left:1rem;padding-right:1rem}[dir] .ProseMirror:focus{box-shadow:none}:where(.ProseMirror-fullscreen-container){height:.25rem;overflow:hidden}.ProseMirror img.ProseMirror-separator{display:inline}.ProseMirror .rte2-style-bold{font-weight:700}.ProseMirror .rte2-style-italic{font-style:italic}.ProseMirror .rte2-style-underline{text-decoration-line:underline}.ProseMirror .rte2-style-strikethrough{text-decoration-line:line-through}[dir=ltr] .ProseMirror .cms-textAlign-left{text-align:left}[dir=rtl] .ProseMirror .cms-textAlign-left{text-align:right}[dir] .ProseMirror :where(.cms-textAlign-center,[dir] .rte2-style-align-center){text-align:center}[dir=ltr] .ProseMirror :where(.cms-textAlign-right,[dir=ltr] .rte2-style-align-right){text-align:right}[dir=rtl] .ProseMirror :where(.cms-textAlign-right,[dir=rtl] .rte2-style-align-right){text-align:left}.ProseMirror .rte2-style-subscript{bottom:-.25rem}.ProseMirror .rte2-style-subscript,.ProseMirror .rte2-style-superscript{position:relative;vertical-align:baseline;font-size:75%;line-height:0}.ProseMirror .rte2-style-superscript{top:-.5rem}.ProseMirror .rte2-style-brightspot-core-rte-heading-HeadingThree{font-size:1.125rem;line-height:1.75rem;font-weight:600}.ProseMirror .rte2-style-brightspot-core-rte-heading-HeadingTwo{font-size:1.5rem;line-height:2rem;font-weight:600}.ProseMirror :is(bsp-line,bsp-break){display:block}.ProseMirror :is(bsp-line,bsp-break):has(>.ProseMirror-trailingBreak){line-height:var(--lineHeight-m)}[dir] .ProseMirror :where(ul,[dir] ol):not(.Dropdown,[dir] .ProseMirror-tableSubmenuList){margin-top:1.25rem;margin-bottom:1.25rem}[dir=ltr] .ProseMirror :where(ul,[dir=ltr] ol):not(.Dropdown,[dir=ltr] .ProseMirror-tableSubmenuList){margin-left:2rem}[dir=rtl] .ProseMirror :where(ul,[dir=rtl] ol):not(.Dropdown,[dir=rtl] .ProseMirror-tableSubmenuList){margin-right:2rem}.ProseMirror ul{list-style-type:disc}.ProseMirror ul ul{list-style-type:circle}.ProseMirror ul ul ul{list-style-type:square}.ProseMirror ol{list-style-type:decimal}.ProseMirror ol ol{list-style-type:lower-alpha}.ProseMirror li{position:relative}.ProseMirror:has(+.ProseMirror-moveEnhancementOverlay){caret-color:transparent;--tw-ring-offset-shadow:var(--tw-ring-inset) 0 0 0 var(--tw-ring-offset-width) var(--tw-ring-offset-color);--tw-ring-shadow:var(--tw-ring-inset) 0 0 0 calc(0px + var(--tw-ring-offset-width)) var(--tw-ring-color)}[dir] .ProseMirror:has(+.ProseMirror-moveEnhancementOverlay){border-color:transparent;box-shadow:var(--tw-ring-offset-shadow),var(--tw-ring-shadow),var(--tw-shadow,0 0 transparent)}.ProseMirror:has(+.ProseMirror-moveEnhancementOverlay)>:not(.Enhancement.readOnly.ProseMirror-selectednode){opacity:.6}.ProseMirror:has(+.ProseMirror-moveEnhancementOverlay) .Enhancement.readOnly.ProseMirror-selectednode{--tw-border-opacity:1}[dir] .ProseMirror:has(+.ProseMirror-moveEnhancementOverlay) .Enhancement.readOnly.ProseMirror-selectednode{border-color:oklch(var(--btu-theme-primary-700)/var(--tw-border-opacity))}.ProseMirror:has(+.ProseMirror-moveEnhancementOverlay) .Enhancement.readOnly.ProseMirror-selectednode>*{opacity:.6}.ProseMirror .rte2-style-comment{gap:.5rem;--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-primary-800)/var(--tw-text-opacity))}[dir] .ProseMirror .rte2-style-comment{border-radius:.25rem;background-color:oklch(var(--btu-theme-primary-50)/var(--tw-bg-opacity))}[dir=ltr] .ProseMirror .rte2-style-comment,[dir=rtl] .ProseMirror .rte2-style-comment{margin-left:.25rem;margin-right:.25rem;padding-left:.25rem;padding-right:.25rem}.ProseMirror .rte2-style-comment:before{font-family:lucide;display:inline-flex;font-feature-settings:"liga" 0;-moz-osx-font-smoothing:grayscale;-webkit-font-smoothing:antialiased;font-size:var(--icon-size);font-style:normal;font-weight:400;letter-spacing:normal;line-height:var(--icon-size);text-transform:none;white-space:nowrap;word-wrap:normal;vertical-align:top;--tw-content:"\e116";padding-inline-end:.25rem;content:var(--tw-content);--icon-size:1.25rem}[dir=ltr] .ProseMirror .rte2-style-comment:before{direction:ltr}[dir=rtl] .ProseMirror .rte2-style-comment:before{direction:rtl}.ProseMirror .rte2-style-comment[data-collapse=true]:before{content:var(--tw-content);padding-inline-end:0}.ProseMirror .rte2-style-comment .collapsed{display:none}.is-locked .ProseMirror .rte2-style-comment{opacity:.6}.ProseMirror-comments{display:none}.rte2-style-ins-insert{--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-success-900)/var(--tw-text-opacity));text-decoration-line:none}[dir] .rte2-style-ins-insert{background-color:oklch(var(--btu-theme-success-200)/var(--tw-bg-opacity))}.rte2-style-ins-insert .Enhancement{--tw-bg-opacity:1}[dir] .rte2-style-ins-insert .Enhancement{background-color:oklch(var(--btu-theme-success-50)/var(--tw-bg-opacity))}.rte2-style-ins-delete{--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-error-900)/var(--tw-text-opacity))}[dir] .rte2-style-ins-delete{background-color:oklch(var(--btu-theme-error-200)/var(--tw-bg-opacity))}.rte2-style-ins-delete .Enhancement{--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-error-600)/var(--tw-text-opacity))}[dir] .rte2-style-ins-delete .Enhancement{background-color:oklch(var(--btu-theme-error-50)/var(--tw-bg-opacity))}.rte2-style-ins-delete .ProsemirrorEnhancementMenu{display:none}[dir] .ProseMirror-previewMode .rte2-style-ins-insert,[dir] .ProseMirror-previewMode .rte2-style-ins-insert .Enhancement{background-color:transparent}.ProseMirror-previewMode .rte2-style-ins-delete{display:none}.ProseMirror-aiPreviewMode .rte2-style-ins-ai{--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-purple-900)/var(--tw-text-opacity))}[dir] .ProseMirror-aiPreviewMode .rte2-style-ins-ai{background-color:oklch(var(--btu-theme-purple-100)/var(--tw-bg-opacity))}.ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-highlight{--tw-bg-opacity:1}[dir] .ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-highlight{background-color:oklch(var(--btu-theme-primary-50)/var(--tw-bg-opacity))}.ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-ins{--tw-bg-opacity:1;--tw-text-opacity:1;color:oklch(var(--btu-theme-primary-900)/var(--tw-text-opacity));text-decoration-line:none}[dir] .ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-ins{background-color:oklch(var(--btu-theme-primary-50)/var(--tw-bg-opacity))}.ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-del{--tw-bg-opacity:1;text-decoration-line:line-through}[dir] .ProseMirror-aiInlinePreviewMode .rte2-style-ai-inline-del{background-color:oklch(var(--btu-theme-gray-50)/var(--tw-bg-opacity))}.CIG-row:only-child.is-unlabeled[data-field=richText]{width:100%}

package/src/legacy/tool-ui/src/main/webapp/dist/7718.99de62ae687524cfc313.css ADDED Viewed

@@ -0,0 +1 @@