-
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
feat(icons): add all svg icons in the @fluentui/react-icons package #12641
feat(icons): add all svg icons in the @fluentui/react-icons package #12641
Conversation
Co-Authored-By: David Zearing <dzearing@microsoft.com>
Co-Authored-By: David Zearing <dzearing@microsoft.com>
Co-Authored-By: Elizabeth Craig <ecraig12345@gmail.com>
Co-Authored-By: Elizabeth Craig <ecraig12345@gmail.com>
# Conflicts: # packages/office-ui-fabric-react/package.json # yarn.lock
…-package-icons # Conflicts: # packages/react-icons/src/components/BorderBlindsIcon.tsx
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: 66e61600f1af22b6d49ca87f3bb80b43158a2174 (build) |
# Conflicts: # packages/office-ui-fabric-react/src/components/Icon/examples/Icon.SvgFactory.Example.tsx # packages/react-icons/src/index.ts
-reverted removed icons
Perf AnalysisNo significant results to display. All results
Perf Analysis (Fluent)Perf comparison
Perf tests with no regressions
|
🎉 Handy links: |
🎉 Handy links: |
…icrosoft#12641) * wip * -added example svg icons * -added props support by diff factory * Update packages/react-icons/src/components/BorderBlindsIcon.tsx Co-Authored-By: David Zearing <dzearing@microsoft.com> * Update packages/react-icons/src/components/BorderBlindsIcon.tsx Co-Authored-By: David Zearing <dzearing@microsoft.com> * Update packages/react-icons/src/utils/createSvgIcon.ts Co-Authored-By: Elizabeth Craig <ecraig12345@gmail.com> * Update packages/react-icons/README.md Co-Authored-By: Elizabeth Craig <ecraig12345@gmail.com> * -cleanups -removed circular dependencies * -removed comments * -improvements * -exported utilities * -added icons as react components * -added storybook example with all svg icons * -updated readme * -addressed comments * -added tests * -added dependencies * -removed merge conflicts changes * -picked changes from master * -reverted changes * -updated package json * -removed example icons * -reverted changes in svg factory -reverted removed icons * -reverted some changes * -removed example icons * -reverted unintended changes * Change files * -updated test * -updated snapshots * Change files Co-authored-by: David Zearing <dzearing@microsoft.com> Co-authored-by: Elizabeth Craig <ecraig12345@gmail.com>
This PR is continuing the work from #12608
Things done in this PR:
@fluentui/react-icons
packageOneDriveIcon
,YammerIcon
andBorderBlindsIcon
components.test.tsx
's paths