Let me know if I can improve my code, all suggestions are welcome.
Kiara
@kxtaraAll comments
- @BagataaSubmitted over 1 year ago@kxtaraPosted over 1 year ago
Hi!
You've done really well on this challenge, but I have two suggestions.
- You should add text to your
alt
attributes for accessibility reasons. - The
section
with the class ofsection-featured
should have a heading tag likeh2
-h6
for HTML validation.
HAPPY CODING
0 - You should add text to your
- @MackenanDsouzaSubmitted over 1 year ago
How do i get the overlay to hover properly
- @wajeeNPCSubmitted over 1 year ago
This website was one of the most challenging website that I came through and the mobile view of this was quite hard and I still expect there should be more things to improve on, Please let me know if there's any fault, Thank you
@kxtaraPosted over 1 year agoHi!!
For accessibility reasons, I suggest you change the
<div>
with the class of main to<main>
or you can put your code inside of amain
tag instead of changing thediv
.HAPPY CODING!!!
0 - @AlabasterRigSubmitted over 1 year ago
Can you point out the errors I have made in this project? Solutions and feedback are welcome.
@kxtaraPosted over 1 year agoLooks good! Have you tried changing the first
h3
tag to anh1
for accessibility?Happy Coding
0 - @sandile05Submitted over 1 year ago
i wasn't sure if i should make it responsive. it seems to respond to smaller screen size by default.
@kxtaraPosted over 1 year agoLooks good! Have you tried using h1 instead of h2 to make the text look a little bit more similar...Happy coding!!
1