-
Notifications
You must be signed in to change notification settings - Fork 14
[CEM-2548] Chair #395
base: master
Are you sure you want to change the base?
[CEM-2548] Chair #395
Conversation
ralph-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting on in meeting review changes
|
|
||
| type Position = 'top' | 'bottom' | 'left' | 'right'; | ||
|
|
||
| export interface IAdvancedChair { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re use existing interface
ralph-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Story is missing action() and arguments
|
Update the picture to showcase the different states btw |
ralph-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to fix colors
ralph-dev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Props need a description |



Created a chair using HTML & CSS
