Conversation
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com> Agent-Logs-Url: https://github.com/bQuery/ui/sessions/01360682-98d4-484f-a913-88f2d137cdee
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com> Agent-Logs-Url: https://github.com/bQuery/ui/sessions/01360682-98d4-484f-a913-88f2d137cdee
There was a problem hiding this comment.
Pull request overview
Adds a new bq-stat-card dashboard primitive to the component library, wiring it into the library’s per-component entrypoints, root registration map, docs, and Storybook so it can be consumed consistently alongside existing components.
Changes:
- Introduces
bq-stat-cardcomponent implementation (props, theming tokens, loading skeleton, i18n-based loading announcement). - Registers the component in the root bundle (
src/index.ts) and side-effect component bundle (src/components/index.ts), plus adds the component entrypoint (src/components/stat-card/index.ts). - Adds coverage across tests, Storybook stories, and VitePress docs/roadmap updates.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/stat-card.test.ts | Adds unit coverage for registration, rendering, normalization, and loading a11y text. |
| stories/stat-card.stories.ts | Adds Storybook stories for default/compact/loading usage. |
| src/index.ts | Registers stat-card in the global component entry map. |
| src/components/stat-card/index.ts | Adds per-component entrypoint export for build/modular imports. |
| src/components/stat-card/BqStatCard.ts | Implements the new bq-stat-card component UI, styling, and a11y/i18n behavior. |
| src/components/index.ts | Adds side-effect import to include bq-stat-card in the full bundle. |
| docs/guide/architecture-roadmap.md | Updates roadmap language to include dashboard/summary primitives and mentions the new component. |
| docs/components/stat-card.md | Adds documentation page for bq-stat-card API and usage. |
| docs/components/index.md | Adds bq-stat-card to the component catalog. |
| docs/.vitepress/config.ts | Adds Stat Card to the docs sidebar. |
| README.md | Updates component coverage table to include Stat Card. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com> Agent-Logs-Url: https://github.com/bQuery/ui/sessions/e73fbf15-2dc3-48ce-92fe-341ccd3a8032
Addressed in
Screenshot: https://github.com/user-attachments/assets/2e1bbc31-c6bd-48f5-9794-b27cfd47f852 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
Co-authored-by: JosunLP <20913954+JosunLP@users.noreply.github.com> Agent-Logs-Url: https://github.com/bQuery/ui/sessions/12a3900c-2656-4eda-b7cb-028de4fa9f9e
Addressed in The loading status in Screenshot: https://github.com/user-attachments/assets/2e1bbc31-c6bd-48f5-9794-b27cfd47f852 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bq-stat-cardloading announcement to include explicit polite live-region behavior💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.