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

Avatar/AvatarGroup: VR Implementation Changes #3765

Merged
merged 9 commits into from
Oct 3, 2024

Conversation

se7en-illa
Copy link
Contributor

@se7en-illa se7en-illa commented Sep 19, 2024

Avatar/AvatarGroup: VR Implementation Changes #3765

AVATAR

CLASSIC VISUAL REFRESH
MainClassic MainVR
ColorsClassic ColorsVR
ColorsDarkClassic ColorsDarkVR
VerifiedClassic VerifiedVR
FocusClassic FocusVR

AVATARGROUP

CLASSIC VISUAL REFRESH
AGClassic AGVR
AGClassic AGVR
DarkModeClassic DarkModeVR
FocusClassic FocusVR
HoverPressedClassic HoverPressedVR

@se7en-illa se7en-illa requested a review from a team as a code owner September 19, 2024 15:13
Copy link

netlify bot commented Sep 19, 2024

Deploy Preview for gestalt ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3d9fa2f
🔍 Latest deploy log https://app.netlify.com/sites/gestalt/deploys/66fede3e55dd240008b74223
😎 Deploy Preview https://deploy-preview-3765--gestalt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@se7en-illa se7en-illa marked this pull request as draft September 19, 2024 22:52
@AlbertCarreras AlbertCarreras added the patch release Patch release label Sep 20, 2024
@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

@se7en-illa se7en-illa marked this pull request as ready for review October 2, 2024 00:28
@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

@AlbertCarreras

This comment was marked as resolved.

author Sarah Alli <[email protected]> 1726758738 -0400
committer Sarah Alli <[email protected]> 1727819323 -0400

avatar vr changes

separated css from avatargroup and avatar

avatar group prop drilling

updated snapshots

added internal props

fixed avatar group props

added avatarColorIndex

updated example

changed keys

fixed typing

colors working

text ui working

replaced original foundation layer for classic

fixed

fixed colors

prettier

fixed focus outline

git adfixed container

fixing docs

fixed prop naming

working on docs

building examples

added more VR examples

changed colors on examples

updated images

fixed pressed state

cleanup

examples fixed

removed focus from regular avatar

updated AvatarGroup examples

prettier

small fixes

snapshot

snapshots

snapshots

dark mode support for temp color values
fixed outline

small fixes

adjusted prop naming

examples fix
yarn.lock Outdated Show resolved Hide resolved
ref,
) {
const [hovered, setHovered] = useState(false);
size = isInVRExperiment ? 'md' : 'fit',
Copy link
Contributor

Choose a reason for hiding this comment

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

this is risky but we'll give it a shot.

@AlbertCarreras AlbertCarreras self-requested a review October 3, 2024 18:19
@AlbertCarreras AlbertCarreras added minor release Minor release and removed patch release Patch release labels Oct 3, 2024
@AlbertCarreras AlbertCarreras merged commit f6f8b89 into pinterest:master Oct 3, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor release Minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants