HTML CSS flexbox

Community feedback

• 

  Joseph•770

  @Jos02378Posted about 3 years ago

  Hey @obaryo, great job on this solution!

  Some Suggestions for you:

  1. For the name and age, you can try to use a span with a heading like this <h1 class="name">Victor Crest <span>26</span></h1> and then style the span.
  2. You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
  3. You can try to use relative units like em in the future for padding. Here is a link to an article that explains when to use which unit see the article.
  4. Try deleting all of the code that you don't need like the ones that you comment in the CSS file just to keep your code looking clean and professional.

  I hope this helps, good luck!

  Marked as helpful

  0