-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LF-4219: Create component for weight application rate tabs #3204
LF-4219: Create component for weight application rate tabs #3204
Conversation
e101944
to
7812d54
Compare
The design has changed, and this component is no longer necessary. Closing this pull request as it is no longer relevant. |
😢 Again! |
import styles from '../styles.module.scss'; | ||
import clsx from 'clsx'; | ||
|
||
const VARIANTS = { | ||
PILL: 'pill', | ||
PLANE: 'plane', |
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.
"plain" 😄 not plane
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.
🙈🙈🙈
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.
Thank you for reviewing Duncan and Joyce!!! |
Description
StateTab
component tovariant
.Semibold
withbutton
so that tabs become focusable. (I plan to use:focus-within
to style cards when focused, which will require this change. The outline is "none" for now.)RoterTab
to be compatible with the new CSS.Jira link: https://lite-farm.atlassian.net/browse/LF-4219
Type of change
How Has This Been Tested?
Checklist: