Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

Fix pricing card height on safari #2682

Merged
merged 1 commit into from
Jun 18, 2024
Merged

Conversation

MananTank
Copy link
Member

@MananTank MananTank commented Jun 18, 2024

PR-Codex overview

This PR updates the styling of the PricingCard component to improve readability and visual hierarchy.

Detailed summary

  • Changed flexDir to "column" for better layout
  • Adjusted padding values for different screen sizes
  • Set height to "full"
  • Increased zIndex to 999 for layering
  • Updated background color based on highlighted prop
  • Changed border color based on current prop

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Jun 18, 2024

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

Name Status Preview Comments Updated (UTC)
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 18, 2024 4:03pm

Copy link

graphite-app bot commented Jun 18, 2024

Merge activity

<!-- start pr-codex -->

## PR-Codex overview
This PR updates the `PricingCard` component in the homepage section with improved styles for better visibility and user interaction.

### Detailed summary
- Updated flex direction to column layout
- Adjusted padding for different screen sizes
- Set height to full
- Increased z-index for layering
- Changed background color based on `highlighted` prop
- Modified border color based on `current` prop

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
@jnsdls jnsdls force-pushed the mnn/fix-homepage-safari branch from 6fcd112 to b57f927 Compare June 18, 2024 15:51
@graphite-app graphite-app bot merged commit b57f927 into main Jun 18, 2024
10 checks passed
@graphite-app graphite-app bot deleted the mnn/fix-homepage-safari branch June 18, 2024 16:04
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.

2 participants