Dušan Lukić• 1,660
@dusanlukic404
Posted
Hi Juveria, you did very nice job! I like your solution 😃
There is a few things you should probably change:
- My suggestion is to use button element instead of input with type of button and then to add cursor: pointer; and some transition to button
- Your accessibility is good but you have one problem. You should increase heading element only by one. You have <h3> so use <h2> instead
- Why Us section should be unordered list with list items for a way better accessibility instead of writing a paragraph with line breaks.
There is a saying that you always look at everything to make it look good without using CSS. Then, you are 100% sure that accessibility is excellent and you can start styling your website. 🤩
Marked as helpful
0
Juveria Dalvi• 1,375
@Juveria-Dalvi
Posted
@dusanlukic404 thanks I've noticed that accessibility issue and I used margin-padding instead of line-break <br> tag
0
Dušan Lukić• 1,660
@dusanlukic404
Posted
@JuveriaD yeah it is also common mistake. I don't suggest you to do in your next projects :)
0
Juveria Dalvi• 1,375
@Juveria-Dalvi
Posted
@dusanlukic404 thanks I'm not going for same mistake again 😃
0
