Looking to hire developers?
@jamiem89
@sagun-k
Submitted
I am just a begginer with this.Though my design doesn't totally match however I have tried cloning it as closest as I could. Hope you guys understand me and kindly suggest ideas how could I improve. Any reviews and suggestions are welcomed. Any suggestions for me.
@jamiem89
Posted
There seems to be a lot of visual differences between your solution and the original design. I'd go back and double check to make things look correct before any further feedback is required.
@venkateshbaddela
Submitted
@jamiem89
Posted
This solution doesn't fit on my laptop screen. You should try using %-based width, rather than absolute values if you want the solution to look good on all screen sizes.
@Uzor13
Submitted
@jamiem89
Posted
Hmm, not sure this is the ideal solution. It looks broken at tablet sizes and the columns are all different widths up at desktop.
@didhee
Submitted
Is there anything I should have done? Are the codes clean enough?
@jamiem89
Posted
All looks good to me. The only feedback i'd give is this: You're switching to the desktop view far too early. Between 375px and 900(ish)px, everything looks very squished in and it gets uncomfortable for a viewer to read.
@Sayyid-Major
Submitted
All comments are deeply appreciated.
@jamiem89
Posted
Looks really good. Nice attention to detail!
@ahnge
Submitted
Is it ok to not know grid if you know flex-box?
@jamiem89
Posted
Flexbox and grid have very different use cases. I would recommend learning both as grid is way better for multi-dimensional layouts. Flexbox works great for a layout like this though!
@balliets10
Submitted
Any feedback would be lovely
@jamiem89
Posted
Looks pretty close to me! I think the text size used for the name might be a bit big, and the spacing under the name and location might be too big too. The padding around the stats at the bottom could also use a bit more padding, but other than that, I think you did a great job!
@jamiem89
Submitted
The only thing I struggled with was getting the two background cirlces to play nicely. I could position them ok with a percent value, but as soon as I change my screen size, they fly around all over the place!
@jamiem89
Posted
I just looked at other solutions and I had no idea I could you could use vh/vw for background positioning! I feel like that will be an easy fix now.
@edosauwalia
Submitted
I had difficulty flipping the image and text in the mobile responsiveness. Please how do I achieve that?
@jamiem89
Posted
Hey Edosa.
I achieved it by using flexbox, and setting the order when switching to the desktop view.
Hope this helps!
