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.
Jamie Moore
@jamiem89All comments
- @sagun-kSubmitted over 3 years ago@jamiem89Posted over 3 years ago
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.
0 - @venkateshbaddelaSubmitted over 3 years ago@jamiem89Posted over 3 years ago
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.
0 - @Uzor13Submitted over 3 years ago@jamiem89Posted over 3 years ago
Hmm, not sure this is the ideal solution. It looks broken at tablet sizes and the columns are all different widths up at desktop.
0 - @didheeSubmitted over 3 years ago
Is there anything I should have done? Are the codes clean enough?
@jamiem89Posted over 3 years agoAll 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.
0 - @Sayyid-MajorSubmitted over 3 years ago
All comments are deeply appreciated.
- @ahngeSubmitted over 3 years ago
Is it ok to not know grid if you know flex-box?
@jamiem89Posted over 3 years agoFlexbox 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!
0 - @balliets10Submitted over 3 years ago
Any feedback would be lovely
@jamiem89Posted over 3 years agoLooks 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!
0 - @jamiem89Submitted over 3 years ago
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!
@jamiem89Posted over 3 years agoI 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.
0 - @edosauwaliaSubmitted over 3 years ago
I had difficulty flipping the image and text in the mobile responsiveness. Please how do I achieve that?
@jamiem89Posted over 3 years agoHey Edosa.
I achieved it by using flexbox, and setting the order when switching to the desktop view.
Hope this helps!
0