Social Media Dashboard with Theme Switcher (HTML, CSS, JavaScript)

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 2 years ago

  Hi, Niña Kaye Galon! 👋

  Good effort on this challenge! 👍

  As far as I know, you can't use the border-radius property in combination with the border-image property. To create the border with the rounded corners and gradient background, you may have to create a separate element (such as a pseudo-element) and position and style it like it's a border. You can inspect my solution to this challenge if you'd like any ideas for how exactly how to do this.

  Besides that, a couple of things I'd like to suggest are,

  • Using a button element for the theme toggle to make it accessible and keyboard-focusable. Right now, it is a simple div which is not very meaningful and can't be accessed via keyboard.
  • Adding a favicon to the site. There should be one in the starter files for this challenge that you can use.
  • Taking a look at your solution report and trying to clear up some of the errors that are there. Looks like you may need to add some alt text to the icon images in the site (although I would set them to an empty string so that they are ignored by screen readers). 😉

  Hope you find this helpful. 😊

  Keep coding (and happy coding, too)! 😁

  0