All comments

• 

  Gorkem Basbug•310

  @gbasbuSubmitted almost 4 years ago

  How to make it better? Your feedbacks are most important to me. Thanks.

  

  Responsive profile card component using Sass

  1

  40
  

  Franzockel•40

  @franzockelPosted almost 4 years ago

  Hey, your solution is looking good.

  Just some tips:

  • you could place the background circles as background images in the body, since they're not content but decorative elements.
  • the positioning of the card itself could be done with display: flex; justify-content: center; and align items: center; I think this would be a better (or easier) solution, than the use of positioning absolute (my opinion 🤷‍♀️)
  • as you can see the report mentioned some html issues. You can use this validator to check your html and css: https://validator.w3.org/ I think it's pretty helpful because you not only see issues but the cause.

  I hope this helps :) keep on coding 🤙

  2

  This comment has received 1 replyShow