Ishmael• 330
@Ishmaelsealey
Posted
Hi There
This is a great solution. Near completely identical to the original design!
The one thing that is deviant from the design is the line height of the heading and paragraph. I suggest you make the line height smaller.
Additionally, for the purpose of accessibility, you should switch the heading from an <h2> tag to an <h1> tag.
I hope you find my suggestion helpful
Marked as helpful
0
Comradeintense• 110
@comradeintense
Posted
@Ishmaelsealey Thank you for the feedback ! I went with a H2 for the reasoning that if this was in a bigger website, the H1 would be somewhere else, but strictly looking at this, h1 would make sense of course.
Yeah totally messed up the line height!
1
