@Danil-ShevchenkoSubmitted 7 months ago
Latest solutions
Four card feature section | HTML & CSS
Submitted 7 months agoIf possible, check if my css is well structured and understandable.
Product preview card
Submitted 7 months agoHelp in recognizing where I could've done better or how in terms of the code done. Exe: what could I do to make everything more responsive in less lines of code.
Recipe page - HTML | CSS
Submitted 7 months agoHow could I make my CSS better and what could I do to make it more readable.
Social links profile - HTML | CSS
Submitted 7 months agoCheck my semantics and syntax please, specially whether I should continue using classes for h1's, h2's,... I want to become a consistent Frontend developer.
Blog preview card
Submitted 7 months agoHelp and feedback with best practices and what can I improve with.
Latest comments
- @vknirSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
Learning grids was challenging, I was able to get the basics of it.
What specific areas of your project would you like help with?I was not able to control the behavior of quoted texts in this example. Any feedback on that would be much appreciated.
@rcsilva211Posted 7 months agoOverall is well done! I'd only suggest adding a max-width on the grid container in order to not have it as a huge grid for bigger screens. That's just a personal opinion though :)
Marked as helpful1 - @zbuli-tSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
responsive web design make the card different positions
@rcsilva211Posted 7 months agoThe code is bit messy, but the result is there. Personally, I'd use grid or flex to align them instead of static values. Keep it up :)
0 - @Ashwini202213Submitted 7 months ago@rcsilva211Posted 7 months ago
In desktop, the width of the card is a bit too big. Other than that, amazing work!
0 - @LucieLuBSubmitted 7 months ago@rcsilva211Posted 7 months ago
There's an extra border on the bottom table. You can fix it by applying the
:last-of-typepseudo-class. It should look something like this:tr:last-of-type{ border: 0; }Other than that, great job!
Marked as helpful0 - @codewinchesterSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
Mastering responsive layouts has been a pain. But, this project is a true testimony that with consistency, something's gotta give. I wish i took kevin powells "Conquering responsive layouts" course earlier. His insights on how to think responsively is what got me here. Seriously, if you want a break, check it out. I am really glad that this design is responsive across all devices and i wasn't forced to use media queries.
What challenges did you encounter, and how did you overcome them?The major challenge was trying to make it responsive using media queries not knowing there was another approach. I was tuck on this loop for a very long time. Trying different device-width ranges and ending up messing.
What specific areas of your project would you like help with?Probably for my fellow community members to review my code and advice on where i should improve.
@rcsilva211Posted 7 months agoWell done on using only flexbox to make it adjustable to any screen! I think the only thing missing here is the hover state on the social media links (tip: they should be <a href=""> tags). Other than that, fantastic job mate!
0
