Shortly URL shortening API Challenge (Html, scss, flexbox, vanilla js)

Community feedback

• 

  Candela Bramucci•680

  @bramuccciPosted about 3 years ago

  Hi Nguyen! This amazes me, it's beauty and clean like the original. Congratulations :D The only thing I would change it's the background in the setion-1, when I try it with different sizes it overlaps with the title. I played and think background-size: 33em; background-position: top left -10em for mobile it's good, and then other media querie with background position: top center. Background are hard, but maybe this help you. Oh and, cursor: pointer; in the social media icons will be great. Again, amazing solution!!!

  Marked as helpful

  0

  ---

  

  Nguyen Duy•245

  @koalalikecodePosted about 3 years ago

  @Liltanie thank you, bro. I have thought of the background and some cases too.

  0
• 

  Nazeer Haidari•895

  @Nazeer2020Posted about 3 years ago

  It is nice

  0

  ---

  

  Nguyen Duy•245

  @koalalikecodePosted about 3 years ago

  @Nazeer2020 thank you!

  0
• 

  Dušan Lukić•1,660

  @dusanlukic404Posted about 3 years ago

  Hey Nguyen, well done on this challenge, I am amazed how you did it!

  0

  ---

  

  Nguyen Duy•245

  @koalalikecodePosted about 3 years ago

  @dusanlukic404 Thank you, bro

  0