@ali-reza-2531Submitted 12 days ago
zzdevs
@zzdevsAll comments
- @tris6Submitted 27 days agoWhat are you most proud of, and what would you do differently next time?
I'm really happy with how I pre-empted cascading layout properties to all cards, but using a separate class for each color setting. The separation between layout and skin means I could minimize code. Taking that minute to look at the design overall, and think about what to cascade and how to apply classes, really pays off.
I didn't pre-empt needing a div for person image and name, so had to go back and wrap each markup individually.
I learned that the gap property can be used for flex children, not just grid cells.
I had to Google background image properties to get the quotes like the design.
What challenges did you encounter, and how did you overcome them?I spent a lot time fine-tuning vertical margin between text elements. Until, I finally realized each section is just evenly spaced. A quick space-around property for the card class, and problem solved.
What specific areas of your project would you like help with?Nothing seemed ambiguous for this project.
- @slnwrldSubmitted 17 days ago
- @TheWraithDevSubmitted 22 days agoWhat are you most proud of, and what would you do differently next time?
I did an entire mobile first approach trying to ensure everything was consistent and focused on accessibility aswell as a responsive layout.
I also focused on the CSS to make sure it was structured and appropriate
What challenges did you encounter, and how did you overcome them?None - my biggest area I ran into was a bug with the scaling of my layout but it was a typo that caused it
What specific areas of your project would you like help with?Responsiveness & Accessibility Best practice guidance where possible as I am always wanting to improve
@zzdevsPosted 22 days agoyou can use the clamp function in css to scale up the font size in higher widths for a better user experience.
0 - @oMarcelosantosSubmitted 23 days ago@zzdevsPosted 23 days ago
Excellent result in my opinion. You could have used the nutrition section to use flex or grid but only to make it more similar to the original, I don't think it's so necessary.
Marked as helpful0 - @RieneeSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I'm absolutely thrilled about this project. It took me less than two days in building this project and I am specifically proud of it's responsiveness.
What challenges did you encounter, and how did you overcome them?To be honest, this has got to be one of the easiest project I have worked on, thus far.
What specific areas of your project would you like help with?All feedback is welcome. Thank you in advance!
@zzdevsPosted 25 days agoIt's really good, the only thing I would do would be to apply padding to the card so that the texts do not exceed the width of the links.
0 - @osuulolaSubmitted 25 days ago@zzdevsPosted 25 days ago
It seems that you didn't use the color from the style guide, but that seems fine to me and I like how it turned out. The only thing that bothers me is the image and the name of the blog author. I think the name should be more aligned with the image or positioned in some way that makes it noticeable that they go together. Just to give an example, maybe using flexbox would solve it. Then you could try to make it more similar to the original in order to practice and hone your CSS skills.
Marked as helpful0 - @osuulolaSubmitted 26 days agoWhat are you most proud of, and what would you do differently next time?
Frontend Mentor | QR code component
Improve your front-end skills by building projects Scan the QR code to visit Frontend Mentor and take your coding skills to the next level Challenge by Frontend Mentor. Coded by Oludare.@zzdevsPosted 25 days agoIt's fine but it differs from the original design. Maybe this is on purpose and it's perfect but if not, keep trying to match it, it's a great way to practice and learn.
0
