responsive 3 column preview using flexbox

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 elements 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 button element

  button {
  cursor: pointer;
  }
  

  • Now your component's button elements have 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!

  0
• 

  Ed-dyb Abdelilah•1,560

  @eddybproPosted over 1 year ago

  Hi, nina1234567896

  Congratulation on finishing this challenge, I hope it was fun.

  Here are some suggestions that might help improve the code:

  • CSS code:

  main{
  */remove min-width property*/
  width:95%;
  border-radius:0.625rem;
  overflow: hidden;
  }
  
  @media only screen and (min-width: 769px)
  main{
  /*remove that line 'width: 21.875rem;'*/
  }
  p {
  /* remove that line 'width: 15rem;' */
  }
  

  Remember there are also devices with width < 350px, so you should update the media query in CSS .

  /*From this*/
  @media only screen and (min-width: 350px) and (max-width: 768px){}
  /*To this*/
  @media only screen and (max-width: 768px){}
  

  For the units, it's better to use relative units rem em instead of absolute units px.

  That's it, HAPPY CODING

  0