Responsive Webpage | Interacts with API

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Awesome work with this project. Looking at your solution, I would like to suggest the following for you…

  • Import the attribution style in CSS file and remove the style tag
  • Instead of div, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics. Like this line <div class="attribution"> can be wrap with footer tag or placed it inside the main tag
  • Include description with the alt in image tags. Like this line <img src="images/pattern-divider-mobile.svg" alt="" srcset="" class="divider-svg">
  • Clean the whitespaces in the code
  • Try increasing the width size into max-width: 30rem; in .card-section rule set

  ADVICE #38

  "The most delicious cocktails often have the highest alcohol content. Always pace yourself to preserve your dignity."

  Above all, the design is well implemented. Keep up the good work!

  Marked as helpful

  1

  ---

  

  Vaibhav Sachdeva•70

  @sachdevavaibhavPosted almost 3 years ago

  @RioCantre Thanks for taking out the time and provide valuable feedback. Will surely work on your suggestions. Thank you!!

  0