Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds BpkCheckboxCard, a new selectable card component for the Backpack design system. The component provides a flexible, accessible checkbox card pattern with rich content support (icons, images, labels, descriptions, prices) and is designed for use across Hotels, Flights, and Car Hire verticals.
Changes:
- Implements BpkCheckboxCard component with compound component architecture (Root, Control, Content, Stack, Inline)
- Adds comprehensive specification, research, and implementation planning documents
- Includes extensive documentation files covering API design, styling guides, bug fixes, and Figma alignment
- Provides multiple example files demonstrating various usage patterns
Reviewed changes
Copilot reviewed 56 out of 57 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| specs/001-checkbox-card/*.md | Specification, research, planning, and implementation documentation files |
| specs/001-checkbox-card/examples/*.tsx | Example usage files for different component patterns |
| specs/001-checkbox-card/checklists/*.md | Requirements validation checklist |
| packages/bpk-component-checkbox-card/src/BpkCheckboxCard/*.tsx | Core component implementation files |
| packages/bpk-component-checkbox-card/src/BpkCheckboxCardSimple/*.tsx | Backward-compatible simple wrapper |
| packages/bpk-component-checkbox-card/*.ts | Package exports and theme configuration |
| examples/bpk-component-checkbox-card/*.tsx | Storybook examples and stories |
| *.md and *.html files | Supporting documentation and demo files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div style={{ display: 'grid', gridTemplateColumns: '1fr 1fr', gap: '16px' }}> | ||
| <BpkCheckboxCard | ||
| checked={selectedWith} | ||
| onChange={(checked) => setSelected With(checked)} |
There was a problem hiding this comment.
Corrected spacing in function name 'setSelected With' to 'setSelectedWith'.
| onChange={(checked) => setSelected With(checked)} | |
| onChange={(checked) => setSelectedWith(checked)} |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
|
Visit https://backpack.github.io/storybook-prs/4173 to see this build running in a browser. |
Remember to include the following changes:
[Clover-123][BpkButton] Updating the colourREADME.md(If you have created a new component)README.md