Design comparison
Community feedback
- @Manisha3196Posted 18 days ago
Hi.For the name i think headings tag will suit better and testimonial text i think p tag suits better than div. I think you used the style for name and graduate in HTML but you can style it in directly in css style so that you will not be repeating the code again and again in HTML. I think you used divs more than you want.you can try checking out other HTML tags like h1,h2,h3 or section or p or article tags. It is not responsive page as you used fixed div tags.rather that you can wrap each one of the testiomonial in div and use flex for mobile and then use grid to arrange the testimonial cards.It will be more responsive and efficient.
1@MariekafPosted 18 days ago@Manisha3196 Hello and thank you for your feedback. I am fully aware of your comments and will try to take them into account in the next challenge. but my problem is with the framing of the elements. I use so many divs because I can easily apply a border to see the frame of the div and be able to frame its content as I wish. I don't know if it can be done any other way
0
Please log in to post a comment
Log in with GitHubJoin 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