Hania B.• 1,360
@techanthere
Posted
The problem happening because you have said explicit height of 18rem on all the cards which is not required. Some other points:
- The height values of 100% are not required, height will fit the content by default. In fact explicit value of height for div is not required at all.
- Level 1 heading (h1) should be used only once because it is used for the whole content like Join our community is the only one that fits the rule. Replace the h1 with h2 while keeping the same styling
- Instead of setting margin-top and margin-bottom value on the container class, use padding on body which will align the content in the center both vertically and horizontally.
- Also remove the height from the anchor tag and instead set padding for vertical and horizontal alignment.
- The div with class answer-container should be an unordered list (ul) while p tag will be replaced by li tags which is logically correct and follows the semantics approach.
Good luck!
Marked as helpful
0
Abhishek choudhary• 1,570
@dazzlerabhi30800
Posted
@techanthere Thank you for the feedback it helped! .😁
0
Hania B.• 1,360
@techanthere
Posted
@dazzlerabhi30800 if it's been really helpful please mark it as helpful as well :)
Marked as helpful
0