html,css

Community feedback

• 

  P

  beowulf1958•1,450

  @beowulf1958Posted about 2 months ago

  Congratulations on completing this challenge. It looks great so far. The only problem is with the buttons. The problems are easily fixed.

  First, the buttons are too short; they look cramped and the "Frontend Mentor" text bleeds out. All you need to do is make ul li { width: 200px; } and that will fix the text overflow.

  Second, the buttons need a hover state. The style guide lists a bright green color for this purpose. Add the following:

    ul li:hover {
      background-color: hsl(75, 94%, 57%);
      color: black;
    }
  

  Now your site looks awesome.

  Marked as helpful

  1

  ---

  

  IBNA SINA SAMI•50

  @sinasami000Posted about 2 months ago

  @beowulf1958 Thank you very much for your advice. I will try not to make the same mistake again.

  0