responsive notification panel

Community feedback

• 

  Aman Singh Bhogal•1,010

  @asbhogalPosted over 1 year ago

  Hi Kartik,

  The reason why most of your pictures are showing is because of the / at the beginning of the src path. Remove these where they're present and your images will show.

  Just a few other things:

  • Locally host your Google Fonts for privacy and performance reasons. Here's a good link explaining why and how to do this Link
  • Use clearer variable and class names, ie. "notification" instead of "noti", as this can make it harder for other developers to read (always code with a mindset of others reading it to work on or debug it)
  • Make sure to add an alt attribute for screen readers and other assistive technologies (this helps them to understand the purpose and context of images for accessibility)

  Hope this helps!

  Marked as helpful

  0