Latest solutions
- Submitted 3 days ago
Four Card Feature
#sass/scss- HTML
- CSS
I'd like feedback on the organization and cleanliness of my code. Does my responsive design work? Is everything readable and make sense? Could I have structured the grid for the cards better?
- Submitted 6 days ago
Product Preview Card Component
#sass/scss- HTML
- CSS
The main areas I'd like input on is the mobile-first design, image swapping and the cleanliness of my sass code.
- Submitted 14 days ago
Social Links Profile Component
- HTML
- CSS
Is everything correct? Is it efficient and clean?
- Submitted 18 days ago
Blog Preview Card Challenge
- HTML
- CSS
I want feedback on the responsiveness of the website. Do you think I did a decent job with starting with the mobile design first?
- Submitted 18 days ago
QR Code Component
- HTML
- CSS
My main concern is efficiency. Is my solution efficient?
Latest comments
- P@nitinrs95Submitted 7 days ago@TheRoboRobinPosted 3 days ago
Looks great! I like the way you are thinking when it comes using the mixins for your breakpoints.
But I think that your responsive design is a little lacking. The site starts to break between 1440px and 900px. It's off center and none of the elements adjust to the space available. While the challenge does just pertain to the 1440px and 375px, It's good to think about the breakpoints between that.
Otherwise, I think you could fix the curve in your top border by just lowering its size by 1px.
Marked as helpful1 - @Henzo238Submitted 7 days ago@TheRoboRobinPosted 6 days ago
Looks great! Code is well organized. You might add a little more spacing to your media query for your larger design. It feels really tight towards the top.
Marked as helpful1 - P@LiJunXiang1234Submitted 10 days ago@TheRoboRobinPosted 10 days ago
Woah! I love the main page that you've created. Very clean and fits within the design feel for each of the projects. If I had anything to say about the challenge itself, it would be that you should maybe adjust the line height a bit. Bodies of text are too squished together. Otherwise, great job!
1 - @mohammed1215Submitted 14 days agoWhat are you most proud of, and what would you do differently next time?
responsive design
@TheRoboRobinPosted 14 days agoI like the way that you handled the responsiveness with the width. But at some point the content feels maybe a bit too big for the space. Maybe add some queries addressing font-size and spacing would help? Great job though!
Marked as helpful1 - @Rajsinghh2907Submitted 19 days ago@TheRoboRobinPosted 18 days ago
It looks great! I would maybe make the text a bit bigger on the original. I would say that it might be a bit small.
I also think that the way you handled making your image responsive is interesting!
Otherwise, I think your code is well-structured. Keep up the good work!
0 - @SimonHicklingSubmitted 18 days ago@TheRoboRobinPosted 18 days ago
The heading is a bit too crowded and maybe needs a little less padding or a smaller font size. There is also a bunch of extra margin on the bottom that you might want to remove. I kind of like the switched up colors but I'd say the text color should be adjusted to this. Lighter on the darker background and placing one of the darker colors on the body instead of the pure black. Makes the text seem out of place.
0