Pénélope Martin• 40
@Pennymrtn
Posted
Hello
First of all, concerning your code, I am rather new too to Flexbox but I think you used it correctly and your code is organized and easy to understand so congrats on that! You have done a great job!
However I have spotted a few redundance and "mistake" in your files :
- You have image and . image img : they can be put together in img{ } you don't need to create a class for that. Be careful with the use of <div> as it makes it difficult for screen readers to navigate your content. If you don't need it don't use it!
- In your html you introduce the class name but you never use it ! Same thing for the class named link. Introducing too many classes can make your code a bit harder to read.
- In your html, there is no need to create different classes for each link because you give them the same aspect in your css.
Otherwise you have a very nice code! And I thank you for introducing to me the fabulous existence of the vh unit. You have solved one of my problems with my code ! <3
PS: That is going a bit far put it would be nice if the links you click on changed color. You can do that by using :focus. It gives a visual cue that yes this element is clickable and you clicked it (when website are a bit slow it is nice)
0
