@marwinfb
Latest solutions
Meet Landing Page
Submitted 12 months agoI think that my code is a big mess, especially CSS one. I hope I used rem values correctly.
Testimonials grid section using CSS GRID
Submitted about 1 year agoGeneral feedback is welcome but if I'd specify something is: Am I using too much classes in CSS?
Social Links Profile
Submitted about 1 year agoIs the HTML structure okay? Is the RWD acceptable/okay? Should I rename classes in a different way? I'm looking forward for every tip I can get!
Latest comments
- @Asv000
Hey! Your solution looks good! I'd suggest using bigger font sizes so it matches the design. Also your footer has some white space which is unnecessary. Otherwise good job mate, you nailed it! Also I think your code looks good but I can give you a little tip: In your next solutions try to use this CSS order: group1[position, top/left/bottom/right display] group2[margin, padding, width, height], group3 [color, background-color, border] group4 [font-size, line-height]. group5[transition, cursor, z-index] so your code can be easy to read and change
Marked as helpful - @WB52What are you most proud of, and what would you do differently next time?
Mobile first design...
Next time I would probably experiment and try grid areas.
Maybe I’d include more breakpoints in the media queries as it goes straight from mobile to desktop and back. I never thought much about changing the design for tablet size, although the mobile will show and work ok.
What challenges did you encounter, and how did you overcome them?I enjoyed getting to grips with grid and found it tricky on the as desktop layout at times but enjoyed the learning process.
What specific areas of your project would you like help with?Any feedback and suggested improvements always welcome, thanks...
@Asv000Hello there! You design is almost pin-point! You did a veeeeeeery good job there, keep it up! By looking at your HTML code I think you are using too much sections (I'm a newbie but that seems a little bit too many for me). Layout looks great on mobile devices (maybe a little bit of padding for 320px) but overall RWD did great! Your code looks clean and is defiently reusable. I wonder you spent a lot of time calculating those rems tho :P That's a great work! Really great! Keep pushing mate 8)
Marked as helpful - @anniekrdlWhat are you most proud of, and what would you do differently next time?
I did it real quick.
What challenges did you encounter, and how did you overcome them?I didn't get the grid-area right. I googled it.
What specific areas of your project would you like help with?None
@Asv000Even if you don't need help with any area of the project I need to say that this is a good job done! If I could point something out is only that you should check those widths on cards and breakpoint after 375px because it looks sketchy. Overall project is nice, it's good, glad you made it fast and you are proud of that, keep it up 8)
- @MohakGogiaWhat are you most proud of, and what would you do differently next time?
Will try to make it more responsive
What challenges did you encounter, and how did you overcome them?Applying styling according to different breakpoints as in which changes to be done
What specific areas of your project would you like help with?Playing more with relative units to scale automatically with the screen size
@Asv000Hey that looks nearly perfect! You need to check those line heights for text and button size, otherwise I believe that's a good job! Keep going :)
- @Vishumate@Asv000
From my POV HTML semantic looks fine. Your solution differ from design quite significantly so when you comeback to this, please try to fix it. Layout on other devices looks good althought image on 320px and 375 doesn't look quite right. I believe that code is well structured and readable.
Marked as helpful - @6633256621@Asv000
It looks like you need to change the font weight for name and location. Also you should make a little bit more space for buttons. I don't think that you should use strong tag for buttons. If you want to use it only for decoration, b tag is fine, but if you want to add semantic for it then strong is the case there. You alt text should be more descriptive like: picture of Jessica or something like that instead of just "jessica". You should also check media queries for mobiles smaller than 375px. I think rest looks fine, keep it up :)
