Product Landing Page - Duff Beer

Product Landing Page - Duff Beer
0.0 0

#1

I just finished up my Product Landing page project. Looking for some feedback. How is the structure of my code, use of id/classes, color scheme(I tried to match it from The Simpsons), etc. Any feedback is appreciated!

The link is here:

Thanks,

Charles


#2

Hi @cdurflinger,

HTML

Line 22:

The frameborder attribute on the iframe element is obsolete. Use CSS instead.

<iframe id="video" width="560" height="315" src="https://www.youtube.com/embed/fSfOEBriE3I?rel=0&amp;controls=0&amp;showinfo=0&amp;start=16" frameborder="0" allow="autoplay; encrypted-media" allowfullscreen></iframe>

MDN documentation:

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe

Deprecated attributes:

frameborder: Obsolete since HTML5

The value 1 (the default) tells the browser to draw a border between this frame and every other frame.
The value 0 tells the browser not to draw a border between this frame and other frames.
You should instead use the CSS property border and its longhand properties to create borders around <iframe>s.

Design:

  • Red background make black letters hard to read (see screenshots):

Screenshots

  • 360x640


  • 800x600


  • 1204x768

Cheers and happy coding :slight_smile:

Note:
Tools used:
validation:
https://validator.w3.org/#validate_by_input
screenshots and testing:
https://github.com/GoogleChrome/puppeteer