What I found difficult a little bit is differentiating between the ellipsis hover and the body hover, but I've figured it out at the end :) and Question about best practices ... is there a better way to solve this kind of problems?
Chitrang
@cwebdevAll comments
- @mohamedmostafakhudariSubmitted over 2 years ago@cwebdevPosted over 2 years ago
Hi,
Good work on the project.
I just wanted to give main feedback regarding responsiveness:
The site layout is not responsive for mobile version. You can check mobile layout in the design files. Also, one suggestion would be to start website development with mobile-first workflow since it would be easier to make desktop version later.
Hope this is helpful.
Marked as helpful1 - @Kivi02Submitted over 2 years ago
I'm new to Frontend development & learning it now. It will be a great help, if you could provide me the feedback on my solution. So I can improve the areas mentioned.
Thanks in advance.
by Kivi02
@cwebdevPosted over 2 years agoHi Kivi02,
Congratulations on finishing your challenge! 🎉
I looked at your solution and here are few observations from my side:
- Add more metadata like description to describe what the page is about
- Add cursor pointer for buttons like "Proceed Payment" and "Cancel Order"
Hope this is helpful. 😁
Marked as helpful1