html,CSS flex and js

Community feedback

• 

  Avinash Dhauni•170

  @avinashdhauniPosted over 2 years ago

  Your main-container box is too wide. You can give your .main-container a width of 30% or something. Or you can give a fixed width of 330px to the .main-container. Also, I noticed you haven't changed your font. The given font in the challenge is Overpass. But congratulations on completing the challenge

  Marked as helpful

  0
• 

  Ahmed Kamel•1,190

  @hmadamkPosted over 2 years ago

  you should read the read me file and add the font specified to your app if you go to the link in the readme it will open google font choose the specified weights, and copy the generated link to the top of your html if you need help you can ask me

  Marked as helpful

  0
• 

  Hyron•5,870

  @hyrongennikePosted over 2 years ago

  HI Kehinde

  Congratulations on finishing the project.

  Just a few things I picked up when looking at the live version when you click the submit button without clicking on the numbers it doesn't show a number out of 5 on the thank you card. You can maybe use a variable to either store a default number of 1 or only show the thank page if a number is selected. Also you should indicate which number has been selected I clicked a couple of times thinking that it wasn't working you can just add a class that makes the circle orange when the number is clicked.

  Marked as helpful

  0