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

Re-styled products page with actual data #329

Merged
merged 5 commits into from
Jun 21, 2024

Conversation

Ridhima10
Copy link
Contributor

Hello PA @Manaregr8
Issue resolved - #322

Recording.2024-06-19.165922.mp4

Kindly review and merge.
I hope you like it

Copy link

vercel bot commented Jun 19, 2024

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

Name Status Preview Comments Updated (UTC)
community-page ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 21, 2024 8:34am

@Ridhima10
Copy link
Contributor Author

@Manaregr8 kindly review this

Copy link
Collaborator

@Manaregr8 Manaregr8 left a comment

Choose a reason for hiding this comment

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

It's awesome but responsiveness is lagging behind kindly fix responsiveness for product page.
i will mark this as level 3 issue.

@Ridhima10
Copy link
Contributor Author

Ridhima10 commented Jun 20, 2024

@Manaregr8 can you please elaborate? Its working fine in my system

Recording.2024-06-20.171519.mp4

@Manaregr8
Copy link
Collaborator

Manaregr8 commented Jun 20, 2024

@Ridhima10
dsdaxa
@ 991 - 768 range of screen width the above overlapping cards issue is arising,
dasdasdasdxa
@ below 768 range of screen the above non centered card issue is arising.

Thank You.

@Ridhima10
Copy link
Contributor Author

@Manaregr8 Kindly review. I have resolved all the conflicts as well

@Manaregr8
Copy link
Collaborator

@Manaregr8 Kindly review. I have resolved all the conflicts as well

Now its looking really awesome,

Thanks You

@Manaregr8 Manaregr8 merged commit ca6c948 into ChromeGaming:main Jun 21, 2024
2 checks passed
@Ridhima10
Copy link
Contributor Author

@Manaregr8 Thankyou for appreciating my work. Means a lot !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants