-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[icons] Add social icons #17274
[icons] Add social icons #17274
Conversation
mbrookes
commented
Sep 1, 2019
•
edited
Loading
edited
- I have followed (at least) the PR section of the contributing guide.
These changes open the door to a broad topic: should we author our own icons? I think that it's the type of decisions we need to be careful with (type 1, hard to revert). Cost. Fontawesome has 1648 icons for a given variant. material.io is 60% behind. I would trust Fontawesome for having prioritized the most important ones. That might be all that it's needed. https://materialdesignicons.com/ has more icons, I haven't count, maybe 2k? We document this alternative. Maybe we don't have to author anything. @leMaik has an interesting project leveraging these components. Value. The situation seems significantly better since the last synonyms iteration, but I think that we should wait it has more data and see if we still can improve the list. For instance, we might realize that the brand icons aren't the most important ones to add: https://github.com/google/material-design-icons/issues?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc A few stats:
I would propose that we keep a close eye on the no results of the search icon page, before doing anything. cc @mui-org/core-contributors |
A second thought, almost all of the popular icon sets available have brand icons, so even if they are not the most important icons we could add, they are probably still important. There is one important aspect that I have overlooked, brand icons name and shapes are supposed to be invariant, it should be safe to add them, we should be able to keep them for a decade. |
Disclamer, adding the brand icons wasn't the idea of Matt, this thread is the results of my self doubts and reflection, blame me, and only me. |
The data came from the Material Design Icons top 5000 search result that @Templarian kindly shared on Twitter:
Here's the breakdown:
I've aggregated stubs that are strongly associated with the full name, but not those that could be associated with other icons, or are full words in their own right (tele, pin, link etc.). For comparison are google, android and iphone that there are already icons for in the Material Icons set. One thing in favour of adding social icons is that for other UI elements, there is usually a reasonable substitute to be found. But brands are brands, so for a user hoping to use a single source for icons, and needing social icons, these will help. Of course they could always use the paths taken from MDI along with the SvgIcon component. I don't think that this should automatically open the door to adding a multitude of additional icons. |
@mbrookes Thanks for the stats, very valuable! Ok, I think that we should add them for convenience and because it's pretty safe :).
Agree |
You may find this useful also. We came to a decision roughly a year ago to do a large brand deprecation as part of v5. The roadmap doc below outlines that process. |
@mbrookes |
No bundle size changes comparing 833738d...bc7dbfb |
I had assumed that this was fixed in #17259, since that retains the icons removed by Google in the latest release. If not, then we need to accommodate those too. |
b8daef0
to
07053f6
Compare
07053f6
to
bc7dbfb
Compare