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

feat(Avatar): handle chipText #306

Merged
merged 20 commits into from
Jun 22, 2023
Merged

Conversation

jaybharadia
Copy link
Contributor

@jaybharadia jaybharadia commented Jun 18, 2023

Resolves #286

Please NOTE

Latest updates from dev branch

  • chipVariant has been removed and replaced with background property in app.config.ts

Here are the screenshots for updated Avatar.vue chipText support.

Screenshot 2023-06-17 at 10 21 43 PM Screenshot 2023-06-17 at 10 22 00 PM Screenshot 2023-06-17 at 10 22 22 PM

Updated Documentation

Screenshot 2023-06-17 at 10 21 16 PM

@nuxt-studio
Copy link

nuxt-studio bot commented Jun 18, 2023

Live Preview ready!

Name Edit Preview Latest Commit
ui Edit on Studio ↗︎ View Live Preview c3552b2

@nuxt-studio-dev
Copy link

Live Preview ready!

Name Edit Preview Latest Commit
ui Edit on Studio ↗︎ View Live Preview 69363bb

@vercel
Copy link

vercel bot commented Jun 18, 2023

@jaybharadia is attempting to deploy a commit to the Nuxt Labs Team on Vercel.

A member of the Team first needs to authorize it.

@jaybharadia
Copy link
Contributor Author

I think Live Preview is not showing the actual update i did. Component not properly loaded in live preview.

@vercel
Copy link

vercel bot commented Jun 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview Jun 19, 2023 0:59am

@atinux
Copy link
Member

atinux commented Jun 19, 2023

Approved the Vercel deployment so we can properly see the changes applied from Vue code.

@benjamincanac benjamincanac changed the title Feat/chiptext avatar feat(Avatar): handle chipText Jun 19, 2023
Copy link
Member

@benjamincanac benjamincanac left a comment

Choose a reason for hiding this comment

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

No need for a yarn.lock.

@jaybharadia
Copy link
Contributor Author

No need for a yarn.lock.

yarn.lock removed 👍

@jaybharadia
Copy link
Contributor Author

is this PR ready to merge ? or any additional updates needed

@benjamincanac
Copy link
Member

benjamincanac commented Jun 22, 2023

I've made some changes to your PR to match the overall design of the library. Let me know what you think!

@benjamincanac benjamincanac merged commit 759af05 into nuxt:dev Jun 22, 2023
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.

How about text in the avatar chip?
5 participants