Responsive result summary page using flexbox

Community feedback

• 

  Furk•600

  @doganfurkanPosted about 1 year ago

  Hey, good job! My suggestions would be:

  • Use aspect-ratio for a perfect round shape
  • Try to avoid fixed widths and heights as much as you can. Especially for mobile, where the screen is already limited.
  • Use em and rem units instead of px
  • Your flex-item-1 classed div has a negative margin-bottom for what? Remove those margins and use "gap" for better control over margins.

  I think if you handle those, you're going to end up with an excellent job.

  Marked as helpful

  1

  ---

  

  Adetona Blessing•150

  @adetona54Posted about 1 year ago

  @doganfurkan Thanks for the advice, I will try to work on it

  1
• 

  Mr.Bs•310

  @MoroccoBSPosted about 1 year ago

  Maybe you can make the circle a little better, giving different random values to the padding is not the right approach, try giving a fixed width and aspect ratio of 1. But overall good work, Well done!

  1

  ---

  

  Adetona Blessing•150

  @adetona54Posted about 1 year ago

  @MoroccoBS Thank you, I will try and work on it

  0