Skip to content
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

**Feature:** Add Contact component to for user metadata #850

Merged
merged 7 commits into from
Dec 5, 2018

Conversation

TejasQ
Copy link
Contributor

@TejasQ TejasQ commented Dec 3, 2018

Summary

This PR creates a nice little component that renders a user's name and metadata for placement in various parts of our UI.
image

It is not tightly coupled to Avatar and just shows contacts.

To be tested

Me

  • No error or warning in the console on localhost:6060

Tester 1

  • Things look good on the demo.

Tester 2

  • Things look good on the demo.

@TejasQ TejasQ self-assigned this Dec 3, 2018
@TejasQ
Copy link
Contributor Author

TejasQ commented Dec 3, 2018

@fabien0102 your export test is 💯 🔥

@@ -48,12 +48,19 @@ const separatorColors = {
* A specialized color palette for typography.
*/
const textColors = {
/** `#333` */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍 ❤️

@fabien0102
Copy link
Contributor

It was looking so good, so I tried to challenge a bit more the component 👼

image

fabien0102
fabien0102 previously approved these changes Dec 3, 2018
Tejas Kumar added 7 commits December 5, 2018 13:54
- Contact does not manipulate margins and correctly wraps
- ContextMenu supports React Elements as label and icon props
- Hint\'s API now allows moving its toolbar around
- Topbar needed an update to be compatible with ContextMenu
- make icon Icon
- remove DiscriminatedTooltip
- Fix example
@TejasQ TejasQ merged commit 2fee773 into master Dec 5, 2018
@TejasQ TejasQ deleted the feature/contact branch December 5, 2018 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants