Design comparison
Solution retrospective
Positive feedback is very important for me.
Community feedback
- @denieldenPosted almost 2 years ago
Hello Muhammad, great work! π
Some little tips to improve your code:
- centering a element with
absolute
positioning is now deprecated, it uses modern css likeflexbox or grid
- use flexbox or grid to the body to center the card. Read here -> best flex guide
- after, add
min-height: 100vh
to body because Flexbox aligns child items to the size of the parent container
Keep learning how to code with your amazing solutions to challenges.
Hope this help! Happy coding!
Marked as helpful1 - centering a element with
- @AdrianoEscarabotePosted almost 2 years ago
Hi Muhammad Adam, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
Consider using rem for font size .If your web content font sizes are set in absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. Relative units βstretchβ according to the screen size and/or userβs preferred font size, and work on a large range of devices.
if you want to continue coding with px, you can download a very useful extension in vscode, it converts px to rem! link -> px to rem
To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5
<h1>Improve your front-end skills by building projects</h1>
The rest is great!
I hope it helps... π
Marked as helpful1@Muhammad-adam778Posted almost 2 years ago@AdrianoEscarabote Thank you Adriano, i`m very glad for receiving your feedback. About accessibility: Thank you, the problem was solved. About Font Size: l want to clarify one point : There is a potential problem of using only relative units for font sizes is that user-resized fonts may break assumptions your layout makes. For example, this might lead to text getting clipped or running too long. If you use absolute units, you don't have to worry about unexpected font sizes from breaking your layout.
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