Huddle Landing page with a single introductory section challenge hub

Community feedback

• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi Mishalee!

  Great job finishing the project! It looks perfectly well for desktop view. However, it is not responsive for mobile views. In order to achieve this, try to implement media queries. This will help you change the style of your component depending on the size of the screen. You can have a look at this link for reference.

  I hope it helps! :)

  Marked as helpful

  1
• 

  Muhammad Shajjar•1,100

  @muhammadshajjarPosted about 3 years ago

  Hi, you already get the valuable feedback, I want to add some more points

  • Don't use width on .huddle-introduction-content instead of use max-width, becasue it causes horizontal overflowing

  .huddle-introduction-content {
      display: flex;
      /* width: 40rem;  (max-width: 40rem)
  

  • Add max-width to a body or make a container class that wraps your all content, by adding it you can control your content from growing too much on larger viewports
  • The logo should be in anchor a, logos are used to navigate on the home page in most the cases
  • Links must have discernible text, [check out this link] (https://help.blackboard.com/Ally/Ally_for_Websites/Instructor_Editor/Improve_Content_Accessibility/Indiscernible_Link#:~:text=What%20does%20discernible%20text%20mean%3F%20Links%20generally%20have,looks%20like%20this%3A%20%3Ca%20href%3D%22%23best-practices%22%3EBest%20practices%20for%20links%3C%2Fa%3E)
  • use media queries to prevent the layout from breaking

  Aside from this Great effort, keep it up

  0