https://master--venerable-madeleine-9b6753.netlify.app/

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 2 years ago

  Hi, Troneci Andrei! 👋

  Nice job on this challenge! 👏 Your solution looks good! 👍

  A couple of things I suggest are,

  • Adding some space between the content of the header and the edges of the screen in the desktop/tablet view of the site to ensure that the content isn't right up against the edge of the screen when the width of the screen decreases.
  • Adding some more descriptive alt text to the main image (as "Main image" doesn't really describe the purpose/content of the image).
  • The "Register" and "Log more" elements may looks like buttons, but they are likely to take the user to another page when clicked, so they should be links instead. It may be helpful to keep in mind that it is not uncommon to style links like buttons, so you should always think about the function of an element rather than it's styling when deciding which HTML element to use. 😉
  • Wrapping the items in the dropdown menus in the header of the page in a tags to turn them into links.

  Hope you find these suggestions helpful. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  1

  ---

  

  Troneci Andrei•260

  @TroneciAndreiPosted almost 2 years ago

  @ApplePieGiraffe Thank you for your feedback I will make all the changes that you suggest

  Happy coding!

  1