This first project took me a week to complete. Please give me feedback on my website. I am really down to your Honest opinions. Also, let me know if my way of doing things is optimal or not, and if there’s a better way of doing it, I will really appreciate it.
I think the whole thing in the middle would look better
1 Like
Hey! I like this project. Two things I noticed are:
- You don’t need a
<main>
followed by<div id="main">
. The<main>
tag is adiv
as well so here’s some unnecessary repetition.
<main>
<div id="main">
- I see you are skipping headers: after
<h1>
comes<h3>
. This hurts your page’s SEO and also accessibility. Headers should come in numerical order and if you’re worried about styling, you can style them with css.
Good job, otherwise!
2 Likes
Thanks, I will look into it, to style the page better. Really appreciate you for giving some feedback!
1 Like
Okay, I will correct these mistakes immediately. Thank you so much for this important feedback!
1 Like
Your page looks good @manavsaspal. A couple of things to revisit;
- Don’t use
<br>
to force spacing or line breaks. Use margin and/or padding in CSS -
- Review the lesson about giving meaningful text to links.
- Web Accessibility in Mind has a more thorough explanation
- ‘wikipedia link’ is not accessible
1 Like
Thanks, Will surely look into these.