All comments

• 

  moniquevnn• 20

  @moniquevnn

  Submitted 2 months ago

  

  Blog post challenge - HTML&CSS Solution

  2

  20

  

  NG• 40

  @nguernse

  Posted 2 months ago

  Good job! Visually looks good. Some suggestions:

  • You have multiple uses of the h1 tag. It's generally advised to keep it to one h1 per page. See here. Also, in the context of this project, using the h1 for the .TAG and .post-copy doesn't quite make sense here. Might opt for <span> or <div> for the .TAG and <p> for the .post-copy
  • The .container-img class could be refactored by moving the padding into the .container class. Then you could remove the div which creates an unnecessary nesting.

  0
• 

  Ahmed Raza• 60

  @ahmedraza032

  Submitted 2 months ago

  

  QR component made using HTML and CSS

  1

  30

  

  NG• 40

  @nguernse

  Posted 2 months ago

  Looks good as is. Great job!

  For future consideration, you might refactor the style.css file to utilize css variables. This would benefit in the future, if the design system changes or you add more classes with similar attributes, allowing you to edit a single variable versus everywhere the same value is being used.

  For example, working with the design system colors:

  :root {
      --color-white: hsl(0, 0%, 100%);
      --color-slate-300: hsl(212, 45%, 89%);
      --color-slate-500: hsl(216, 15%, 48%);
      --color-slate-900: hsl(218, 44%, 22%);
  }
  

  So then whenever you use the dark grey color you can do:

  p {
      color: var(--color-slate-900);
  }
  
  .text-content {
      color: var(--color-slate-900);
  }
  

  Marked as helpful

  0

  This comment has received 1 replyShow