Skip to content

feat(storybook): add UX Error and Validation layout pattern - #1089

Merged
dbacomputer merged 2 commits into
mainfrom
codex/ux-error-validation-pattern
Aug 27, 2026
Merged

feat(storybook): add UX Error and Validation layout pattern#1089
dbacomputer merged 2 commits into
mainfrom
codex/ux-error-validation-pattern

Conversation

@dbacomputer

@dbacomputer dbacomputer commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a new "UX Error and Validation" reference page under Apollo Wind > Patterns > Layout Patterns, showing how validation/error state should be communicated consistently across every surface of a canvas workbench: left sidebar, canvas node, properties panel, and bottom panel.
  • Section messages, navigation validation (tab error counts), and inline field validation are demonstrated together in one realistic DAP-style "Send email" node configuration, matching the existing state-pattern language already established in alert.stories.tsx and NodePropertyPanel.stories.tsx.
Screenshot 2026-08-26 at 5 09 29 PM

Test plan

  • biome check clean on the new file
  • Verified in Storybook (dark + light themes): node error border/badge, section message, tab error-count badges, inline field errors, sidebar status dot, and bottom panel error badge all render correctly
  • Design/eng review

🤖 Generated with Claude Code

Copilot AI lite review requested due to automatic review settings August 27, 2026 00:10
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Apollo Coded App preview deployments are ready.

Project Status Preview Updated (PT)
apollo-design Ready Preview · Logs Aug 26, 2026, 05:32:32 PM
apollo-docs Ready Preview · Logs Aug 26, 2026, 05:32:32 PM
apollo-landing Ready Preview · Logs Aug 26, 2026, 05:32:32 PM
apollo-vertex Ready Preview · Logs Aug 26, 2026, 05:32:32 PM

@github-actions

Copy link
Copy Markdown
Contributor

Dependency License Review

  • 1951 package(s) scanned
  • ✅ No license issues found
  • ⚠️ 2 package(s) excluded (see details below)
License distribution
License Packages
MIT 1722
ISC 88
Apache-2.0 55
BSD-3-Clause 27
BSD-2-Clause 23
BlueOak-1.0.0 8
MPL-2.0 4
MIT-0 3
CC0-1.0 3
MIT OR Apache-2.0 2
(MIT OR Apache-2.0) 2
Unlicense 2
LGPL-3.0-or-later 1
Python-2.0 1
CC-BY-4.0 1
(MPL-2.0 OR Apache-2.0) 1
Unknown 1
Artistic-2.0 1
(WTFPL OR MIT) 1
(BSD-2-Clause OR MIT OR Apache-2.0) 1
CC-BY-3.0 1
0BSD 1
(MIT OR CC0-1.0) 1
MIT AND ISC 1
Excluded packages
Package Version License Reason
@img/sharp-libvips-linux-x64 1.3.2 LGPL-3.0-or-later LGPL pre-built binary, not linked
khroma 2.1.0 Unknown MIT per GitHub repo, missing license field in package.json

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Storybook “Layout Patterns” reference story focused on consistent UX for validation and error states across the canvas workbench surfaces (sidebar, canvas node, properties panel, bottom panel). This fits into the Storybook app as a documented UX/layout pattern under the Apollo Wind patterns area.

Changes:

  • Adds a new fullscreen Storybook story: “UX Error and Validation” workbench pattern with coordinated validation states across multiple UI surfaces.
  • Updates Storybook’s stories configuration to include the new src/patterns directory.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
apps/storybook/src/patterns/LayoutPatterns.stories.tsx New “UX Error and Validation” layout pattern story demonstrating consistent validation/error UI across the canvas workbench.
apps/storybook/.storybook/main.ts Registers apps/storybook/src/patterns so the new pattern story is picked up by Storybook.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread apps/storybook/src/patterns/LayoutPatterns.stories.tsx Outdated
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

📊 Coverage + size by package

Per-package bundle size on this PR (no JS/TS source changes detected under packages/* or web-packages/*).

Package Coverage New-line coverage Packed (gzip) Unpacked vs main
@uipath/apollo-core 41.59 MB 49.70 MB ±0
@uipath/apollo-react 7.56 MB 29.11 MB ±0
@uipath/apollo-ui-icons 2.85 MB 6.91 MB ±0
@uipath/apollo-wind 424.0 KB 2.73 MB −21 B
@uipath/ap-chat 43.46 MB 56.09 MB ±0

"Coverage" is each package's own coverage.include scope (e.g. apollo-core instruments only scripts/). "Packed"/"Unpacked" come from npm pack --dry-run and only cover built packages — "—" means not measured this run (package not affected / not built). "vs main" is the packed (gzipped) delta against the last successful main build (the package-sizes artifact from the Release workflow); "—" there means no main baseline was available this run. The baseline is main's latest build, not this PR's exact merge-base, so it includes any drift since the branch diverged. Packages with no vitest config are omitted.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Storybook visual diff

⚠️ Visual changes detected: 1 added (of 1 compared, 0 unchanged). View report

Baseline is the deployed main Storybook, so changes merged to main after this branch was last updated can also appear here. Logs

Updated (PT): Aug 26, 2026, 05:33:57 PM

Copilot AI review requested due to automatic review settings August 27, 2026 00:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

@bryan-uipath bryan-uipath left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dbacomputer for the additional pattern guidance

@dbacomputer
dbacomputer merged commit 8059e20 into main Aug 27, 2026
47 checks passed
@dbacomputer
dbacomputer deleted the codex/ux-error-validation-pattern branch August 27, 2026 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app:storybook size:XL 500-999 changed lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants