HTML,CSS

Community feedback

• 

  Zeynal Mardanli•180

  @LshirocPosted almost 3 years ago

  Hi, I took a look your code. I think you should notice the font-weight. Header and button texts are bold on the design. And in the music plan section I think it would be better to use CSS Grid or Flexbox. If you put music icon , music plan and music rate to the same div and the change text to other div, you can use flexbox. justify-content: space-around; You don't need to give 750px width. However I couldn't find why page is go down.

  I wish it'll help :D Keep coding

  0

  ---

  

  Zeynal Mardanli•180

  @LshirocPosted almost 3 years ago

  @Lshiroc I think I found it. You should give the background image to the body tag. And then...

  background: url(images/pattern-background-desktop.svg);
  background-repeat: no-repeat;
  background-color: hsl(225, 100%, 94%);
  

  0
• 

  Attila•290

  @unexunderscorePosted almost 3 years ago

  Hi there! Nice work, just keep going! I’m a beginner, but I work every day to get better, so keep it up!

  My comments: Try to set the background-size: 100% I think this should solve the background size problem.

  Also if you hover over the change link, you should remove the underline. Use this if you hover -> text-decoration: inherit;

  Have a nice day!

  0