Fabricio1919• 20
@Fabricio1919
Submitted
Wasu44
@Wasu44
All comments
- Wasu44• 50
@Wasu44
Posted
Hello I'm not good at frontend development but I will try to help.
Next time try to do project more similar. Your font is not the same as they wanted. You should stick more to the project. About html it looks pretty good but next time try to use sementic elements to improve the accessibility and ceo. Your css looks great You did good job.
0 - khanwelcomes• 60
@khanwelcomes
Submitted
Wasu44• 50@Wasu44
Posted
Hello there! I'm a newbie too but let me give you some advice.
Having your styles in the same html file doesn't look good, and also, when it gets larger it will be kinda confusing. Also your site is not responsive for mobile devices there is a video that should help You link. You should also use rem units because they are responsive instead of px.
I hope You find it helpful
Marked as helpful
0 - PinkPig16• 10
@PinkPig16
Submitted
Wasu44• 50@Wasu44
Posted
Hello You did very good job with your project but I can see a few issues.
- You are using px units instead of rem because rem units are resposive and here is video explaining how it works link
- You could be a bit more patient, as your solution isn't exactly similar to the design
- I hope You find it helpful
0
