@visualdenniss
Posted
Hello Sandra :)
congrats on completing the challenge successfully! Nice result.
However there seems to be couple issues to work on. To name a few: Giving a height: 60vh; is not a good idea, because it will stretch and squeeze depending on the screen size and cause unwanted paddings etc. It is best to avoid giving any fixed height or a height using percentages like this, instead just let the content of container determine the height. Use paddings and margins instead to tweak the look.
The color of the body text seems to be too bright, so using a darker value would improve the contrast and reability.
Hope you find this feedback helpful!
Marked as helpful
@sancoza
Posted
@visualdenniss Thanks a lot for your help. It means a lot to me. I will fix the code. All the best in your future work.