-
-
Notifications
You must be signed in to change notification settings - Fork 179
feat(tab): add Tab and TabsHeader components #193
Conversation
Add Tab and TabsHeader components, move bootstrap import to index.tsx, fix test in button.test.tsx implement HospitalRun#24
src/components/Tab/Tab.tsx
Outdated
|
||
return ( | ||
<li className="nav-item" role="tab"> | ||
<button |
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.
Could this be replaced with our own Button implementation?
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.
Yeah good idea. I tried and it seems like it will work.
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.
Awesome!
Use our Button component, differentiate between ColorVariant and ButtonVariant HospitalRun#24
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.
@jackcmeyer or @MatthewDorner can you resolve conflicts?
3b9cc72
🎉 This PR is included in version 0.30.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Fixes #24.
Changes proposed in this pull request: