bikeinman• 1,080
@BikeInMan
Posted
Nice work!
0
Ozan Zorlu• 240
@zorluozan
Posted
@BikeInMan thanks
0
Submitted
FAQ Accordion Card using HTML,CSS and media queries
#accessibility
Ozan Zorlu• 240
@zorluozan
Design comparison
Solution retrospective
Any feedback about how to position desktop background image would be appreciated. :)
Community feedback
Fraser Watt• 1,790
@fraserwat
Posted
Hey Ozan, this is looking good!!
I would remove the height attribute from card__body and add some padding to the bottom. At thinner screen widths this is causing issues with the text spilling out of the container. Most of the time you don't need to explicitly state the height and width for components (although max-width is often useful), and doing so often raises responsiveness issues.
Keep up the good work!!
0
Ozan Zorlu• 240
@zorluozan
Posted
@fraserwat Hey Fraser, thank you for your feedback. I actually add this height attribute because when I open a question, the height of the card is expanding but I don't want that so I gave height property.
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord