All comments

• 

  Osama Saeed• 125

  @OsDroidi

  Submitted over 3 years ago

  This is the first challenge I have done using pure CSS, I'm still struggling with flexbox and CSS grid, I would appreciate your feedback, Thanks.

  

  Single price grid component (Html, Css, box shadow, Flexbox)

  3

  4

  0

  

  Jan• 500

  @eljasiu

  Posted over 3 years ago

  Hello, you did quite a good job but I would personally change a few things. I would use 'h2' or 'h3' headings instead of 'p' elements in the top section. In the 'container' you do not need 'justify-content: center', because the child elements take the whole width. And in the 'center' you dont need need it as well, because you wrote 'flex-direction: column'. Have a nice day, keep coding!

  1

  This comment has received 1 replyShow
• 

  nganbarova• 190

  @nganbarova

  Submitted over 3 years ago

  

  Huddle landing page with alternating feature blocks

  1

  1

  0

  

  Jan• 500

  @eljasiu

  Posted over 3 years ago

  Hello, personally I would put #on-top div inside the footer element, and apply "position: relative;" on the footer. It would make positioning the #on-top div easier

  0

  This comment has received 1 replyShow
• 

  Dina• 460

  @dinakonto

  Submitted over 3 years ago

  1. Was CSS grid the right choice here, or do you think flexbox could have served the design better? 1.2) Could I have made the cards flow into their places more easily, ie without having to manually input their grid positions?
  2. Is it possible to animate the movement of cards between media queries? I couldn't find a transition property that seemed to work.

  

  Mobile first with HTML, CSS grid and jQuery

  1

  2

  0

  

  Jan• 500

  @eljasiu

  Posted over 3 years ago

  Hello, I think that both Grid and Flexbox is a good option in this project. Just depends on your personal preference:-)

  0