fix:CTORNDSD-522: UI issues - #10
Merged
Merged
Conversation
merge develop into main
* chore: update Dockerfile.storybook to include .npmrc and eslint.config.mjs in build context * chore: add typesVersions for renderer subpath in libs/ui/package.json * chore: update workflows to use `actions/setup-node@v7` instead of `v6`
* chore: update Dockerfile.storybook to include .npmrc and eslint.config (#3) * chore: update Dockerfile.storybook to include .npmrc and eslint.config.mjs in build context * chore: add typesVersions for renderer subpath in libs/ui/package.json * chore: update dependencies and regenerate metadata dates - Bump `vite` to `8.1.4`, `rolldown` to `1.1.5`, `postcss` to `8.5.19`, and related bindings to `1.1.5`. - Sync auto-generated component metadata dates in `README.md` and `llms.txt`. - Minor formatting adjustment in Storybook's `main.ts`.
…mo integration (#7) * feat(ui): Optimize A2UI rendering with caching and React.memo integration Added memoization to `A2UINode` using `React.memo` to prevent unnecessary re-renders in unchanged subtrees. Introduced caching for `dispatchAction`, `customRenderers`, `renderChildren`, `mergedStyles`, and `sanitizedChartAttributes` for improved performance and stability. Updated type definitions and utility methods to support these enhancements. * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * test(utils): add memoization tests for `renderA2UISpec` renderer - Ensure custom renderers are not re-invoked when input references remain unchanged. - Verify re-invocation occurs when action references change. - Update auto-generated documentation components count and generation date. - Change license type for `gd-form-configurator` packages to MIT. --------- Co-authored-by: Olek <opovolotskyi@griddynamics.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…and remove deprecated size variants - Introduced "xxl" size to Avatar component and updated associated enums in schema files. - Adjusted size scaling for "sm," "md," "lg," and "xl" tokens to improve consistency. - Removed deprecated "UserCardVariant" and "UserProfileVariant" stories from Avatar examples. - Updated documentation and generated metadata to reflect Avatar size updates.
…. A2UI JSON fields, adjust docs and examples - Clarified the distinction between React component props and A2UI JSON-specific fields in schema descriptions for `Avatar` and `AvatarUser`. - Updated examples, quick starts, and troubleshooting sections in component documentation to use `fallbackComponent` directly for TSX. - Aligned enum and prop descriptions with updated usage patterns, specifying renderer behavior for `fallbackComponent` mapping. - Adjusted README component metadata to include real props and explain renderer mappings for A2UI-specific aliases.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Avatar sizing model and synchronizes related Storybook and A2UI/AI documentation so the new xxl tier is supported and described consistently across tokens, schemas, and generated docs.
Changes:
- Adjust Avatar token sizes and introduce explicit
xl(64px) andxxl(80px) tiers. - Update Avatar Storybook sizing demo to include XXL and remove the
Avatar.Usercard/profile variant stories. - Extend A2UI schemas to allow
xxlforsize/sizeVariant, and expand AI component schema documentation for Avatar/AvatarUser.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| libs/ui/src/tokens/avatar.ts | Updates Avatar dimension tokens and adds xl/xxl size tiers. |
| libs/ui/src/components/atoms/Avatar/Avatar.stories.tsx | Updates size-variation story to include xxl and removes Avatar.User variant stories. |
| libs/ui/src/components/atoms/Avatar/snapshots/Avatar.test.tsx.snap | Updates snapshot output after style/token changes. |
| libs/ui/src/ai/schemas/components/AvatarUser.ts | Expands AI schema documentation; adds xxl to size enums and clarifies A2UI-vs-React props. |
| libs/ui/src/ai/schemas/components/Avatar.ts | Expands AI schema documentation; adds xxl to size enums and clarifies A2UI-vs-React props. |
| libs/ui/src/ai/README.md | Regenerates AI docs output to reflect updated component schemas. |
| libs/ui/src/ai/a2ui/ui-specification-schema.json | Extends sizeVariant enum to include xxl. |
| libs/ui/src/ai/a2ui/spec-schema.ts | Extends size/sizeVariant enums to include xxl and updates schema text (needs a small correction). |
| libs/ui/llms.txt | Regenerates LLM-facing component docs to reflect updated schemas. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tjeleascov
approved these changes
Jul 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://griddynamics.atlassian.net/browse/CTORNDSD-650
https://griddynamics.atlassian.net/browse/CTORNDSD-651
https://griddynamics.atlassian.net/browse/CTORNDSD-652