huddle-lending-page-with-alternating-feature-blocks-master

Community feedback

• 

  Ritik Kumar•120

  @itswebcoderPosted almost 2 years ago

  Your soulution look like nice but in this solution there will be some issue. You will solve this solution in your solution there will be silly mistakes. You will be use box shadow in your solution as border. Box-shadow: 0 0 20px #cdcdcd; just like that.

  Marked as helpful

  0
• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  HI! Your solution looks nice though there are a couple of things you can improve which I hope will be helpful! 😊

  For containers with classes "container-two", "container-three" and "container-four" you don't need the border. Instead, you can remove the border and set a box shadow:

  box-shadow: -1px 0px 5px -2px hsl(208, 11%, 55%);
  

  Also, make sure to wrap the entire code in the main tag. It will help to remove report issues and improve accessibility as well. If you are using, for example, header or footer tags, you can place them outside the main tag.

  Make sure to use rem (relative length value) or em units instead of pixels for the purpose of “respecting the user preferences”, to say so. It makes the “sizes” of the website fluid according to the zoom/sizes set by the user. I see you do use rem sometimes but sometimes you use px for font-size, so make sure to stick with one thing.

  You can read more about it here: https://www.freecodecamp.org/news/what-is-rem-in-css/

  Marked as helpful

  0