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

Wishlist new component #238

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nandakumarbm26
Copy link

Fixes Issue

Changes proposed

Implemented Wishlist component and functionality.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

Copy link

netlify bot commented Jan 24, 2024

Deploy Preview for ar-webstore ready!

Name Link
🔨 Latest commit 79c1db0
🔍 Latest deploy log https://app.netlify.com/sites/ar-webstore/deploys/65d0cb4883bfb500084e7706
😎 Deploy Preview https://deploy-preview-238--ar-webstore.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.

@ShwetKhatri2001
Copy link
Owner

Good work ! It can be merged after these UI improvements.

image

  • On the product card, Remove the plus icon by commenting it and place your heart icon instead in the first row.
  • In the Wishlist Drawer instead of Close X add a cross icon on the right top.
    Also, see if you can close it on click to somewhere outside the drawer.

@ShwetKhatri2001
Copy link
Owner

@nandakumarbm26 Will you be able to complete this in the next few days ?

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.

None yet

2 participants