Three Column Preview Card

Community feedback

• 

  William Boehringer•130

  @williamboehringerPosted over 1 year ago

  Hi Lucas, Greetings from another Brazilian!

  Looking briefly at your code, I could notice some points that may help you improve considerably in this challenge and in many others.

  The first thing I noticed is that you've used the flexbox. It's a personal choice rather to use flex or grid, but I will give my sincerely opinion. It's way easier in this case to use grid, you would certainly write less code and get the same result, i suggest you take a look at it, search some videos on youtube about grid and how to use it. In this case you would divide your div "conteudo" in 3 parts, and use the property "place-items: center", instead of position absolute to center the whole thing.

  Another advice would be to use height and width instead of padding to give the size you want to your columns. Sometimes use padding is good, but in cases like this is definetily not a good option, specially using fixed values. The fixed values will be annoying to deal with when you start making things responsive.

  For now, I think that this advices should give you some things to work and some tools to. Hope I've helped, keep on going and good coding :)

  0