Hello,
I just finished my product landing page, I’m pretty happy with the results. But surely there is something to be said about it.
I think my code is not very clean …
Hello,
I just finished my product landing page, I’m pretty happy with the results. But surely there is something to be said about it.
I think my code is not very clean …
Welcome to the forum @r.fantou84. Your page looks good. Some things to revisit;
body
element in HTML. (No need to include the body
tags). For anything you want to add to the <head>
element click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box.
<br>
element to force line breaks or spacing. That’s what CSS is for.Hey @r.fantou84!
I think your page looks good.
When it comes to your code, I noticed a few things.
When I ran your code through the html analyzer I noticed a few errors. So I would address those issues.
Also , I noticed that you would repeat the same color alot.
color: rgb(48 107 79);
Maybe you could place that color in the body and then if you need to use another color like black then you can simply apply that when needed.
body {
color: rgb(48 107 79);
}
Hope that helps!
Happy coding!
Hello @r.fantou84, welcome!
This is a very decent project. I like it. Simple & Clean. But, some suggestions:
html { scroll-behavior: smooth; }
to your CSS code and notice the smooth scrolling effect when clicking on the header links.head
tag and not the body
tag. Everything inside the CodePen’s HTML box goes inside the body
tag. To add resources to the head
tag, click on the settings icon at the top of the HTML box and then paste the code inside the "Stuff for <head>
" box.alt
tag. You can find it by clicking on the down arrow and then “Analyze HTML” in the HTML box itself.a
tag so that when I click on it, it brings me to the top.Anyway, wonderful work! Best of luck with your next project!