Product Landing Page - Face Masks

Kindly give me feedbacks on my 'Product Landing Page ’ Project . Please find below the link.



Page looks good @PVN. The only thing to mention is small. In HTML5, The frameborder attribute on the iframe element is obsolete. Use CSS instead.

1 Like

Hi @Roma,
Thank you for the feedback.As per the suggestion I have added CSS as well but didn’t remove the frameborder attribute as it is there in YouTube embed link.Hope this works.

your project is looking fine, good job.
However, I would recommend some development -

  • expect top-padding to footer
  • change bg color at contacts section
  • on small screen, header is taking mostly half of the screen space, do necessary

Thank you.

1 Like

Thank you for the suggestions.I will definitely work on that.

1 Like

It’s an attribute. It won’t hurt to remove it from the HTML…and you’ve added the correct property: value; pair in CSS

1 Like

Thank you for the response.I have removed frameborder attribute from html.