Responsive Summary Component

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  ---

  POINTING CURSOR ↗️:

  • Looks like the component's button element has not a pointer, this property plays a major-role in terms of both UI & UX

  ---

  • The cursor: pointer CSS property is important for button-like elements because it changes the cursor from the default arrow to a pointer when hovering over the element. This provides a visual cue to the user that the element is clickable and encourages interaction.

  ---

  • In terms of UI/UX, using cursor: pointer helps to improve the usability of the interface by making it easier for users to identify interactive elements. It also helps to provide feedback to the user by indicating which elements are clickable and which are not.

  ---

  • So we want to add this property to the following div class=button element

  .button {
  cursor: pointer;
  }
  

  • Now your component's button has got the pointer & you learned about this property as well

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  0
• 

  Kamlesh Rajesh Yadav•5,060

  @Kamlesh0007Posted over 1 year ago

  Congratulations on completing the challenge! 🎉.That's a great achievement, and I'm sure you put a lot of effort into it. I really liked the way you approached the challenge and the code you wrote. You demonstrated a good understanding of the concepts and applied them effectively to solve the problem.I have a few suggestions to improve your code further. You need to use flexbox to make the container center because it is preferred over positioning By applying below styles, the container will be centered both horizontally and vertically on the page. This approach is preferred over using position: absolute because it allows for more flexibility and responsiveness, adapting to different screen sizes and content.

  body
  {
  display: flex;
  flex-direction: column;
  justify-content: center;
  align-items: center;
  min-height: 100vh;
  margin:0;
  }
  

  now you can remove the below commented styles

  @media screen and (max-width: 750px)
  {
  .results-summary-component {
  color: white;
  background-color: white;
  border-radius: 0px;
  box-shadow: 0px 30px 60px rgba(255,255,255,0);
  /* margin: 0; */
  /* position: relative; */
  width: 100%;
  /* top: 0%; */
  /* left: 0%; */
  /* transform: translate(0%, 0%); */
  display: flex;
  flex-direction: column;
  align-items: center;
  }
  }
  

  Marked as helpful

  0