All comments

• 

  Madeleine De Guzman•70

  @deguzmanmadeleine22Submitted almost 2 years ago

  I encountered the same problem with the nft preview card with overlay not covering the whole image when resizing to other devices. Will try using the suggestion on the other project to fix the issue.

  

  stats-preview-card

  3

  30
  

  Alisher Toshmurodov•330

  @alishertoshmurodovPosted almost 2 years ago

  Hi, Madeleine.

  Great job working on this project. To solve the issue with your image overlay, you can try changing the width and height of your .main-img-overlay block to %100 instead of a fixed size.

  I hope it will help. Best of luck!

  0
• 

  Hosanna Bivan•140

  @HossannaSubmitted almost 2 years ago

  

  Simple Calculator

  1

  20
  

  Alisher Toshmurodov•330

  @alishertoshmurodovPosted almost 2 years ago

  Hi Hosanna! You did a great job on this project. However, you can do a few things to improve this project:

  1. Theme switch does not look as good as in the design files.
  2. Not precise subtraction when subtracting numbers with floating points.
  3. It would be best if you use CSS Grid for your keyboard.
  4. Validate your HTML code (You can use W3C Validator).
  5. Use correct fonts according to the style guide.

  I hope this will be helpful. Best of luck, Hosanna!

  0