Skip to content
This repository has been archived by the owner on Feb 5, 2025. It is now read-only.

[Graphite MQ] Draft PR GROUP:02fe3c (PRs 486) #489

Conversation

graphite-app[bot]
Copy link

@graphite-app graphite-app bot commented Jan 3, 2025

This draft PR was created by the Graphite merge queue.
Trunk will be fast forwarded to the HEAD of this PR when CI passes, and the original PRs will be closed.

The following PRs are included in this draft PR:

**Fixes or implements VF-4364**

### Brief description. What is this change?

Fixed the carousel button hitbox issue by making the 'surrounding' button component a little bit bigger so when the image is enlarged it will still be inside of the 'hitbox'.

Added the `NewChat` component wrapper to the Carousel story, so that it looks good in storybook.
Added a `2px` left margin to the first carousel card to align it with the other messages displayed above or below it.

Also updated `@vanilla-extract/css` and `@vanilla-extract/vite` packages.
Initially did this cause I thought there was a vanilla-extract related bug - there wasn't, but the update is still helpful to stay updated.
Copy link

vercel bot commented Jan 3, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
react-chat ⬜️ Ignored (Inspect) Visit Preview Jan 3, 2025 6:53pm

Copy link

sonarqubecloud bot commented Jan 3, 2025

@graphite-app graphite-app bot closed this Jan 3, 2025
@graphite-app graphite-app bot deleted the gtmq_02fe3c_486_1735930425871-6caacb58-2ad5-4085-8e9f-cd0d0e3829d7 branch January 3, 2025 19:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant