@yasminegamal19
Latest solutions
Four Card Solution - Help Wanted
Submitted 6 months agoI would really appreciate if anyone can help with why my @media command isn't changing the layout the way I've intended it to. I've spent a lot of time looking at it and trying different tweaks to no avail.
Product Description Card with Mobile and Desktop Layout
Submitted 8 months agoInefficiencies in the code Best practices that are missing or done incorrectly
Recipe Page Challenge in HTML and CSS
Submitted 8 months agoBest practices that I'm doing incorrectly and/or missing - Inefficiencies
Social Media Links in HTML and CSS
Submitted 8 months ago- Best practices that I'm doing wrong or missing
- Inefficiencies in the code
Blog Card with Hover Effect
Submitted 8 months agoI feel okay with the code on this one, but if there are some mistakes or inefficiencies, I'd be happy to hear about them. Mostly, I think I need more practice to get faster and grapple with more complicated issues.
QR Code using HTML and CSS
Submitted 8 months agoFor this project, I'm not looking for any specific help. If there's something I've done that would be better done another way, I'm happy to hear about it and would appreciate general advice.
Latest comments
- @ElkuchWaltz
Excellent work! The design looks nearly identical and the code looks clearly and effectively written to my newbie eyes!
The only points I notice where it could be improved are small details:
- PERFUME should be in all caps
- The font weight on 149.99 is too low
- The button doesn't change colors during hover
I think the resolving those would make this solution perfect! Keep up the great work!
- @Born2Flex@ElkuchWaltz
Excellent work! The layout looks nearly identical, nice job!
The only thing that I notice is that the font didn't change to Young Serif for the h1 and h2 as you've written it to. I see that you pulled the font with the @ at the top of the CSS file, but I've found that I have more luck pulling Google fonts in the header of the HTML file. You may want to try that and see if the font works then. I'd also suggest adding ", serif" after "Young Serif" in the stylesheet so that it defaults to another serif font when Young Serif is not available.
Overall, looking great! Keep it up!
Marked as helpful - @Marioomario@ElkuchWaltz
Looks great!
- @Adnan7389What specific areas of your project would you like help with?
- Layout
- font weight and size
- line-height and letter-spacing properties
- color
@ElkuchWaltzExcellent work! I just completed this project too, and it looks like you were able to match the original much more closely than I was, well done!
One piece of advice I received on mine was to use rem instead of px for font size. I know the Frontend mentor style sheet gives us the <p> font-size in px, but rem can scale better for users who want to change the font size in their screens.
- @Sushant203What are you most proud of, and what would you do differently next time?
I am proud of myself that I actually attempted this challenge and genuinely completed it without much help and assistance. I will definitely try to improve my code using more specific HTML5 semantics.
What challenges did you encounter, and how did you overcome them?I was getting problem in centering the div properly and MDN DOCS CSS flexbox property article helped me to overcome that problem and complete my challenge .
What specific areas of your project would you like help with?I'd appreciate if i got helped in writing more clear css codes and usage of flexbox property more specifically.
@ElkuchWaltzLooking good! Keep up the good work!
