Thank you for your help i think i’ve managed to fix all the bugs through W3C!
Is this error message is ok ? Error : Element head is missing a required instance of child element title .
The problem now is the video takes all the width of the page! I tried to give it a max-width value but it does not work! Should i use a JS solution instead?!
Hey @ahmed512! This project is so good! I like the clean design and it matches Netflix. But, you can improve a few things:
I like smooth scrolling a lot. So, add this line of code to your CSS: html { scroll-behavior: smooth; } and click on the links on the header. Now you can see a really cool scrolling effect.
Try changing the color of the text and icons of the features section to white. It is more easy to read.
You included the FCC test suite in the head tag! You should not do so. Instead, add the script tag linking to the test suite before your header element, that is, at the top of your HTML code.
In mobile devices, making the header fixed would be annoying. Let it be normal for mobile-only.
That you don’t need to worry because they are tags that are outside the body tag and you don’t need to specify those in CodePen. But, if you are working outside CodePen, you need to specify the entire markup: from <!DOCTYPE HTML> to </html>.
Did you see any error in the validator for this?
It is ok to wrap your iframe around a div.
Anyway, the project is wonderful and the design and layout are so neat. Thumbs up!
Yes, it’s okay. In my post I said when copy/paste from codepen it’s okay to ignore the first warning and first two error messages as codepen provides them behind the scenes.
Again, this message is because there is no head element since you copy/paste from codepen. This is the second error I said you could ignore.
Yes, in the body the header is correct and you’ve used it correctly.
Thank you for your feedback i’ve added! The scroll and changed the color to white too! It made more improvement to it!
No it was no error on validator! I was just thinking, if it was ok to code it that way!
I was thinking to add a background like in the official netflix account, but i feel i’m not so confident using backgrounds, i need to relearn it again! If there is a good tutorial don’t hesitate to send me!
Also i’m thinking to try to include some soft animations too, i think it can help make better design in general!