Latest solutions
Latest comments
- @bryancarias@MeganPhillips
Your site looks great! I had a few small suggestions: consider adding aria labels to make your sites more accessible to viewers. You could also use the <del> tag for the original price instead of the <span> tag with CSS for the strikethrough to make your CSS slightly more concise.
- @rllz0@MeganPhillips
It looks great, the only suggestion I would have is to maybe try to use rem instead of pxs for font sizes and such to make it responsive. Also, consider separating each section into a <section> tag as you did for the prep time section.
- @bhuiyan33What are you most proud of, and what would you do differently next time?
This is easy for this challenge using HTML and CSS
What challenges did you encounter, and how did you overcome them?I don't have any problem solving this.
@MeganPhillipsIt looks really good!
- P@BaraKalvoWhat are you most proud of, and what would you do differently next time?
I've managed to resolve how to make font size responsive without using media queries, so I've learned a new technique. I wouldn't do anything differently, I guess.
What specific areas of your project would you like help with?Any comments or suggestions are more than welcome :-)
@MeganPhillipsIt looks great! That's great that you learned a new technique as well, the only difference I see on yours is your line-height is slightly shorter than the template and your card is positioned slightly higher than the template. For the line-height you could just use line-height on your text and alter the pixels. I also had trouble with the positioning so I'm not too sure what to fix on that, I am assuming it could be margins or padding interrupting the centering though. Overall, great job!
- @BigBoyBenz@MeganPhillips
Your attention to detail shows up in your work and it turned out great! The only thing I would suggest is just the placement of the card.