Design comparison
Solution retrospective
Hi there š, Iām Kamil and this is my solution for this challenge. š
š ļø Built With:
- vanilla HTML/CSS
- npm - prettier
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. šāļø
Community feedback
- @vanzasetiaPosted over 1 year ago
Hello, Kamil! š
Good alternative text! š You don't include the word "image" or "avatar" in it.
Headings must be in a logical order. Users of assistive technology can use heading tags to navigate the website. If headings are not in a logical order, they can get confused.
I don't think the content that is currently being wrapped by
<h2>
should be heading. I think it should be<p>
with bold styling or with<strong>
tag.I recommend wrapping the quote with
<blockquote>
. So, the<blockquote>
would be the parent element of the<p>
that wraps the quote.Never use
px
unit for font sizes. Userem
orem
instead. Relative units such asrem
andem
can adapt when the users change the browser's font size setting. Learn more ā Why you should never use px to set font-size in CSSI hope this helps. Have fun coding! š
Marked as helpful1
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