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

Button: new variants #3931

Merged

Conversation

gabrielpereirapinheiro
Copy link
Contributor

@gabrielpereirapinheiro gabrielpereirapinheiro commented Jan 9, 2025

Adding two new variants to button

Classic

  • Light
Screenshot 2025-01-09 at 08 30 24
  • Dark
Screenshot 2025-01-09 at 08 29 23

VR

  • Light
Screenshot 2025-01-09 at 08 32 13
  • Dark
Screenshot 2025-01-09 at 08 31 40

Adding two new variants to buttonLink

Classic

Screenshot 2025-01-16 at 10 48 50 Screenshot 2025-01-16 at 10 49 11

VR

Screenshot 2025-01-16 at 10 50 46 Screenshot 2025-01-16 at 10 50 59

@gabrielpereirapinheiro gabrielpereirapinheiro requested a review from a team as a code owner January 9, 2025 16:32
@AlbertCarreras AlbertCarreras added the minor release Minor release label Jan 13, 2025
@AlbertCarreras
Copy link
Contributor

@gabrielpereirapinheiro Can we do this also for ButtonLink

Copy link

netlify bot commented Jan 16, 2025

Deploy Preview for gestalt ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit bc11e3c
🔍 Latest deploy log https://app.netlify.com/sites/gestalt/deploys/678fd45f758ef70008184c28
😎 Deploy Preview https://deploy-preview-3931--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.

@gabrielpereirapinheiro gabrielpereirapinheiro changed the title Button - always light/dark variants (Classic, VR) Button - always light and dark variants Jan 16, 2025
@gabrielpereirapinheiro gabrielpereirapinheiro changed the title Button - always light and dark variants Button: always light and dark variants Jan 16, 2025
@gabrielpereirapinheiro gabrielpereirapinheiro changed the title Button: always light and dark variants Button: new variants Jan 16, 2025
@AlbertCarreras
Copy link
Contributor

AlbertCarreras commented Jan 16, 2025

can we document them in the docs? otherwise we cant test
Screenshot by Dropbox Capture

we wanna see in darkmode
Screenshot by Dropbox Capture

@AlbertCarreras
Copy link
Contributor

AlbertCarreras commented Jan 16, 2025

Seems broken in classic
Brave Browser - Button - Gestalt 2025-01-16 at 11 51 29 PM
Brave Browser - Button - Gestalt 2025-01-16 at 11 52 28 PM

ok in VR

Brave Browser - Button - Gestalt 2025-01-16 at 11 54 20 PM

@AlbertCarreras
Copy link
Contributor

Check all the permutation visual checks I do! Ideally, all these has been first reviewed by you, and Im supposed to catch them as a last resource!

@AlbertCarreras
Copy link
Contributor

You need to update the snapshots,
Screenshot by Dropbox Capture

Run 1. yarn jest --updateSnapshot

AND 2. kill the docs, run yarn build, and then run cd packages/gestalt-design-tokens && yarn test --updateSnapshot

@AlbertCarreras AlbertCarreras merged commit 9b73731 into pinterest:master Jan 22, 2025
13 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