microsoftgraph / microsoft-graph-toolkit

Authentication Providers and UI components for Microsoft Graph šŸ¦’
https://docs.microsoft.com/graph/toolkit/overview
Other
946 stars 305 forks source link

mgt-person-card css custom properties do not match the documentation [BUG] #333

Closed vogtn closed 4 years ago

vogtn commented 4 years ago

Describe the bug The css custom properties for the mgt-person-card component:

https://docs.microsoft.com/en-us/graph/toolkit/components/person-card#css-custom-properties

do not match the available properties in the component: https://github.com/microsoftgraph/microsoft-graph-toolkit/blob/master/src/components/mgt-person-card/mgt-person-card.scss#L11

To Reproduce Steps to reproduce the behavior:

  1. Install and run repository to index.html
  2. Give the mgt-person-card component css custom properties as shown in the documentation:
    --font-size: 14px;
    --font-weight: 600;
    --height: '100%';
    --background-color: transparent;
  3. See none of the properties have changed

Expected behavior documentation matches implementation behavior.

Environment (please complete the following information):

Additional context

Stackoverflow Ref: https://stackoverflow.com/questions/60765595/font-size-not-working-as-a-parameter-inside-mgt-person-card

ghost commented 4 years ago

Hello vogtn, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback šŸ™Œ