Built with mostly css grid

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi olasunkanmi, Great job!!

  You have added a nice feature. Some graphic design tips:

  • increases the contrast between the colors for the background and titles especially in the dark mode
  • try using the css transition property to make changing the theme more pleasant

  Overall you did well, keep it up! And Happy coding!

  Marked as helpful

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container

  -> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

  -> But use header tag only once in main heading element.

  -> Always use h1 first and then h2, h3 and so on

  Keep up the good work!

  Marked as helpful

  1
• 

  Aditya Singh•460

  @adityas24Posted almost 3 years ago

  Good job with grids buddy!

  Biggest problem with your web page is accessiblity. Color of your text and headings is very light. Make them bit darker so that they are accessible to everyone. They might be inaccessible to person with weak eyesight. Many other factors come under accessiblity. Google about accessiblity in web dev and you will get more insight into this.

  And here i am sharing a great tool to know that color contrast is good or bad and how readable/accessible is your text: https://coolors.co/contrast-checker/112a46-acc8e5

  All the best 🙂👍

  Marked as helpful

  1

  ---

  

  olasunkanmi•180

  @SiR-PENtPosted almost 3 years ago

  @adityas24 thanks, will do just that. I really appreciate.

  0