-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Set font-family inherit on button and input elements #15172
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 4443aa9:
|
Asset size changes
Baseline commit: 261dea7a150115ce904768724cbeac80ded20382 (build) |
Perf AnalysisNo significant results to display. All results
Perf Analysis (Fluent)Potential regressions comparing to master
Perf comparison
Perf tests with no regressions
|
🎉 Handy links: |
🎉 Handy links: |
🎉 Handy links: |
Pull request checklist
$ yarn change
Description of changes
Currently,
Fabric
component appliesfont-family: inherit
forbutton
,input
andtextarea
here.This causes a specificity issue because they will win over component-level styles. see example here.
This change adds
font-family: inherit
to component styles so they are styled correctly without the styles fromFabric
.This change does not remove styles from
Fabric
, since it can be a breaking change for users. However, in the long term, we will not have these styles by default inThemeProvider
(a replacement ofFabric
).Focus areas to test
(optional)