microsoftgraph / microsoft-graph-toolkit

Authentication Providers and UI components for Microsoft Graph 🦒
https://docs.microsoft.com/graph/toolkit/overview
Other
943 stars 300 forks source link

[BUG] Hover states review for `mgt-person` and usages in other components #2530

Closed sebastienlevert closed 8 months ago

sebastienlevert commented 1 year ago

The current state of the hover is missing key elements. We need a review and suggestions on how to improve.

ghost commented 1 year ago

Hello sebastienlevert, 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 🙌

danial999 commented 1 year ago

Describe the bug pnp v4.9 mgt-person -> person-card on hover with rtl is displayed incorrectly ( left positioning )

To Reproduce added pnp search resault webpart v4.9 use mgt-person with dir="rtl" and person-card="hover" in custom template when hover on person to see card , the card is displayed in far right of the screen

Expected behavior on hover person card should be displayed on relative position to the person image

Screenshots image

sebastienlevert commented 1 year ago

@danial999 thanks for the bug. Can you create a specific issue for this one? I'll reply over there as I don't want to mix two issues. Thanks!