All comments

• 

  Ho Van Quoc•20

  @hvQuoc133Submitted 5 months ago

  What are you most proud of, and what would you do differently next time?

  Doing this project helps me learn more about Html and Css.

  What challenges did you encounter, and how did you overcome them?

  There are CSS properties that I don't know about.

  What specific areas of your project would you like help with?

  Please help me consider the appropriate way to use tags in Html?

  

  Social links profile

  2

  30
  

  P

  Fadi Aljohari•230

  @fadialjawhary0Posted 5 months ago

  This is great. I got few suggestions for you.

  1- Make the button's cursor as pointer by using this CSS property "cursor: 'pointer' "

  2- Make the <a> </a> tags to have this " target='_blank' " like this <a href="Link here" target="_blank"> </a>. This will make the website be opened in a new tab.

  3- On small screen sizes (around 450px width and less), the profile picture is not centered in the middle, try to make it in the middle by either using Flexbox for all the items (containers), with flexDirection: 'column' and alignItems: 'center', or by using margin: 0 auto; Try to learn and use Flexbox.

  Great job :).

  Marked as helpful

  1

  This comment has received 4 repliesShow
• 

  LenkaKo•40

  @LenkaKoSubmitted 12 months ago

  

  HTML, CSS, JS

  1

  10
  

  P

  Fadi Aljohari•230

  @fadialjawhary0Posted 12 months ago

  Looks great, and the animations nice.

  Just a quick note, make the button "PLAY AGAIN" cursor pointer. And make it on hover to change the color of the button's text from dark to red. And put some space between the result and the button :).

  Great job.

  0