All comments

• 

  P

  Fluffy Kas•7,715

  @FluffyKasSubmitted over 3 years ago

  Hey guys, if there's anything I could improve on, please let me know. Have a great day! :)

  Edit: I set an animation delay on my cards so they don't show up in the challenge screenshot. Is there a way to fix this?

  

  Four card feature section with vanilla css

  2

  10
  

  Mikko•15

  @mikkoo99Posted over 3 years ago

  Project is well coded. Almost everything is legible (like that comments which separate sections), and good use of variables. On my 13" laptop i have to scroll down to see last card. I suggest to give your body 'height: 100vh' in '@media (min-width: 900px)'. In that case you will have to reduce size of elements.

  0
• 

  Abdullahi Bashir•50

  @ab-sowSubmitted over 3 years ago

  Hi am a newbie please rate my work this is my first challenge and its open for critics and advice

  

  Responsive web section with HTML and CSS

  1

  10
  

  Mikko•15

  @mikkoo99Posted over 3 years ago

  You've got error in opening your project from github. Make sure that your repository hasn't got index.html in any folder (in my case it worked after doing that)

  0
• 

  Rohit•150

  @git-0rSubmitted over 3 years ago

  suggestions are welcome.

  

  3Column-Preview-Card-Component

  1

  20
  

  Mikko•15

  @mikkoo99Posted over 3 years ago

  Great job! It looks well on my IP11. About code, don’t use #id for css, leave it for JavaScript, use .class for css only. You can be more specific with width of your columns by adding flex-basis: 100% in your class=„card”. Doing that you dont have to give your columns width: 33,3333…% What’s more in your media quaries you have in your .card flex-wrap: nowrap and flex-wrap: wrap

  0