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

princing page #12

Merged
merged 16 commits into from
Dec 29, 2024
Merged

princing page #12

merged 16 commits into from
Dec 29, 2024

Conversation

Binit08
Copy link
Collaborator

@Binit08 Binit08 commented Dec 17, 2024

pricing page done

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for currenci-ecell ready!

Name Link
🔨 Latest commit 2d5ec4b
🔍 Latest deploy log https://app.netlify.com/sites/currenci-ecell/deploys/67715930c567130007ef472d
😎 Deploy Preview https://deploy-preview-12--currenci-ecell.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 60
Accessibility: 96
Best Practices: 75
SEO: 91
PWA: -
View the detailed breakdown and full score reports

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

@Binit08 Binit08 changed the title Binit frontend princing page Dec 17, 2024
src/app/components/Pricecard.jsx Outdated Show resolved Hide resolved
src/app/components/Pricecard.jsx Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 20, 2024

image

The text in the middle is a bit small in smaller devices (visibility issues)

@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 20, 2024

image

Provide some padding in the left and right side of the page.

Check overall responsiveness for one last time.

</h2>
<div className="h-[2px] bg-gold"></div>
<div className="relative flex justify-center -top-[25px]">
<button className="bg-gold text-2xl h-[50px] w-[130px] rounded-[57px] hover:bg-yellow-500 font-semibold">
Copy link
Collaborator

Choose a reason for hiding this comment

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

This button will use WhatsApp Api/URL to send message to seller from customer's whatsapp. Use the simplest thing you can get.

@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 23, 2024

image

most elements are overflowing out of the html root node in desktop view

visibility issue for 2nd line still exist in smaller device

@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 24, 2024

image

Something is missing here check design.

@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 29, 2024

image

Minor alignment issue. Also add the metadata for the page. Change the global metadata to "Currenci" instead of "Next App Template: Sahin"

@Ruler45
Copy link
Collaborator

Ruler45 commented Dec 29, 2024

Most containers are still in overflow condition. Keep that in mind and fix it afterwards.

@Ruler45 Ruler45 merged commit e9b33b8 into dev Dec 29, 2024
5 checks passed
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.

2 participants