All comments

• 

  Mohamed Mostafa Khudari•660

  @mohamedmostafakhudariSubmitted over 2 years ago

  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?

  

  Responsive Time Track Dashboard (CSS Grids, JS EventListeners)

  1

  40
  

  Chitrang•150

  @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 helpful

  1
• 

  Kivi02•30

  @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

  

  Order Summary HTML, CSS

  2

  30
  

  Chitrang•150

  @cwebdevPosted over 2 years ago

  Hi 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 helpful

  1

  This comment has received 1 replyShow