Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Brand Icon for Microsoft Teams? #937

Open
coreyroth opened this issue Mar 21, 2017 · 6 comments
Open

Brand Icon for Microsoft Teams? #937

coreyroth opened this issue Mar 21, 2017 · 6 comments

Comments

@coreyroth
Copy link

Now that Microsoft Teams has hit GA, it would be nice to get a brand icon for it. I can scrape an image from somewhere but the experience would be better if it came from the icon set provided by Fabric Core.

@mikewheaton
Copy link
Contributor

A new icon, TeamsLogo, has been added in #946 and will be included when Fabric Core 7.0.0 is released.

image

@Danny-Engelman
Copy link

Danny-Engelman commented Apr 5, 2017

FYI, Teamslogo and TeamslogoFill do not follow the naming convention;

should be: TeamsLogo and TeamsLogoFill

Spotted because I have CSS Selectors on Logo

https://365csi.nl/toolkit/iconator

@mikewheaton
Copy link
Contributor

Good catch, @Danny-Engelman, I'll submit a new PR to fix this and will raise the issue with our icons team to make sure it doesn't sneak into a future release.

@mikewheaton
Copy link
Contributor

Opened #951 to fix the capitalization issue.

@htryggva
Copy link

htryggva commented Jun 4, 2018

@mikewheaton I think a proper Teams Brand Icon is still needed to keep the white areas of the brand icon.

Notice how the background color bleeds through the TeamsLogo:

image

@mikewheaton
Copy link
Contributor

Good point, @htryggva. I'll re-open this to see if we can get SVG and PNG resources added to the Office brand icons.

There is a TeamsLogoFill icon that seems to be meant to solve this issue. You can layer it under TeamsLogo like so: https://codepen.io/mikewheaton/pen/qKbzzE

But it doesn't look right:
image

@mikewheaton mikewheaton reopened this Jun 4, 2018
@mikewheaton mikewheaton removed their assignment Jul 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants