Skip to content

feat: complete buyers favorite page structure, dummy json logic, and layout styling - #53

Open
Nathan-coder691 wants to merge 2 commits into
mainfrom
feature/buyers-favorites
Open

feat: complete buyers favorite page structure, dummy json logic, and layout styling#53
Nathan-coder691 wants to merge 2 commits into
mainfrom
feature/buyers-favorites

Conversation

@Nathan-coder691

Copy link
Copy Markdown
Collaborator

The Buyer's Favorites page allows users to view and manage their saved items. It dynamically loads products from a data file and displays them in a responsive grid layout.

Each product card shows the item's image, category, title, pricing (including sale items), and availability status. Users can interact with the page by adding in-stock items directly to their cart or removing items from their favorites list instantly. The page automatically counts total items and switches to a dedicated "Empty State" message if there are no saved items left.

@smartee-17

Copy link
Copy Markdown
Owner

Hey! Great work on the buyer's Favorites page. It looks really clean and well put together. 👏

I just have a few suggestions before we do the main backend merge.

I think we're already in a good place on the backend, and most of the functionality is covered. Right now, since the current flow works more like a catalogue than a shopping cart, I don't think an "Add to Cart" button is the best fit for now.

Instead, I think we should replace it with "View Product." Then, inside the product details page, we can have a "Contact Seller" or "Chat on WhatsApp" button. Even if it's temporary, I think it better reflects how the application currently works.

Also, regarding the HTML, I noticed we're trying to keep things consistent across the project. I think we should use a more uniform relative path style for our imports and references instead of mixing formats. It'll help keep the codebase cleaner and more consistent.

Lastly, please add our current logo to the logo/icon area.

Apart from these few things, everything looks really good. Nice work! 🚀

@smartee-17
smartee-17 self-requested a review August 14, 2026 17:16

@smartee-17 smartee-17 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I’ve gone through the code and everything looks good from my end.

You can go ahead and add the necessary API integration, update the website icon area with the app logo, and add the proper header and footer. For now, though, I think this can be merged as a placeholder while you work on the full updates.

Also, I have to say, the animations look really lovely. Great job with those! 🙌

Approving the PR.

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