-
Notifications
You must be signed in to change notification settings - Fork 602
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
Remove lodash get from class names in the tooling components #2171
Comments
janechu
added
feature
A new feature
community:good-first-issue
Good issues for first time contributors
labels
Aug 22, 2019
|
This has been done on the |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
EisenbergEffect
added
area:react
and removed
reason : stale
community:good-first-issue
Good issues for first time contributors
feature
A new feature
labels
Jul 20, 2020
closed by #3517 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Convert the managed classes in the tooling components to use the
classNames
utilities that the@microsoft/fast-components-base-react
uses so that there is not so much over-use of the lodashget
utility.classNames
can be imported as a secondary import from the@microsoft/fast-web-utilities
package.This should improve perf of the tooling.
The text was updated successfully, but these errors were encountered: