-
Notifications
You must be signed in to change notification settings - Fork 183
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
[LOOM-1295][BpkClearButton] Remove the BpkClearButton integrate directly with the BpkInput #3403
Conversation
// remove any default button styling | ||
padding: 0; | ||
border: 0; | ||
background-color: transparent; | ||
color: tokens.$bpk-text-secondary-day; | ||
cursor: pointer; | ||
appearance: none; // hidden by default | ||
|
||
@include bpk-rtl { | ||
right: inherit; | ||
left: bpk-spacing-md(); | ||
} | ||
|
||
@include utils.bpk-hover { | ||
color: tokens.$bpk-text-primary-day; | ||
} | ||
|
||
&:active { | ||
color: tokens.$bpk-text-primary-day; | ||
} | ||
|
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.
Lifted from BpkClearButton.module.scss
Browser supportIf this is a visual change, make sure you've tested it in multiple browsers. |
Visit https://backpack.github.io/storybook-prs/3403 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3403 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3403 to see this build running in a browser. |
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.
brilliant 👍
…tly with the BpkInput (#3403) * Update the BpkCloseButton for Class2 fixes * add snapshots * remove the ClearButton combine with BpkInput --------- Co-authored-by: metalix2 <matthewr@skyscanner.net>
…tly with the BpkInput (#3403) * Update the BpkCloseButton for Class2 fixes * add snapshots * remove the ClearButton combine with BpkInput --------- Co-authored-by: metalix2 <matthewr@skyscanner.net>
Remember to include the following changes:
[KOA-123][BpkButton] Updating the colour
README.md
(If you have created a new component)README.md
.d.ts
) files updated