Pls Check out my Product Landing Page and give feedbacks

[Ebuka_Ukoha]

Hey guys,
So I’m just done with the Product Landing Page Project and I’m left only with the Personal Portfolio Website.
Can anyone give me feedbacks on this project?
It would be really appreciated
Here’s the code

FCC Product Landing Page Project

[jwilkins.oboe]

Hi @Ebuka_Ukoha !

I think your page looks good.

A few things from me

It looks like your submit button here is not showing

I think the Buy now buttons should have a cursor pointer

You have a few places with empty alt attributes like this

    <img src="https://www.carlogos.org/car-logos/lamborghini-logo-1000x1100.png" alt="" id="header-img" />

That is not good practice. Alt attributes are important for accessibility.
The importance of Alt Tags - Siteimprove Help Center.

Hope that helps!

[Ebuka_Ukoha]

Oh.
Thank you for that.
I’ll try to implement those changes

[osiephriejiro]

.form{
text-align: center;
border-raduis:10px;
font-size:30px;
font-weight:bolder;
font-family: cursive;
color:red;
}
.logo{
font-size:20px;
font-style:italic;
font-weight:normal;
text-align:center;
color:red;
}
.goal{
padding-left:40px;
font-size:20px;
font-style:“Times new roman”;
color:white;
}
.icon{
padding-left:40px;

}
body{
background-image:
url("/storage/emulated/0/Download/isham-fernandez-GZ8mq7RAAbq-unsplash.jpg");
background-repeat:no-repeat;
}
.love{
font-size:20px;
font-family:“Times new roman”;
color:white;
}
.sum{
font-size:20px;
font-family:“Times new roman”;
color:white;
}
.role{
white-space: nowrap;
font-family:“Times new roman”;
font-size:20px;
color:white;
}
.data{
font-family:“Times new roman”;
}
.rage{
font-size:35px;
font-family: cursive;
font-weight:180px;
}
. favorite{
font-size:15px;
font-family: courier;
font-style:italic;
}
bold{
width:250px;
height:100px;
no-resize:none;
}

.button{
border-raduis:10px;
width:120px;
color:white;
}
.login input[type=“Submit”]
{
border:none;
outline:none;
height:30px;
color: orange;
}
.main{
width:380px;
height:1100px;
background: purple;
color: Orange;
top:20%;
left:20%;
position: absolute;
box-sixing:border-box;
padding-left:40px;
padding-right:25px;
border-raduis:10px;
}
.cn{
background:lightblue;
color: orange;
height:20px;
width:150px;
}
.cn a{
text-decoration:none;
outline:none;
}
This is the code for the CSS if you can incorporate it

[Ebuka_Ukoha]

Tell us what you want us to help you with. What you wrote doesn’t really help much.
There are many people in this forum willing to help