All comments

• 

  Guilherme Cardoso Martins•35

  @GuiCardosoMartinsSubmitted over 3 years ago

  I ask you not to skimp on constructive criticism

  

  HTML and CSS

  1

  40
  

  Pedro Miguel•175

  @pedromigueldevPosted about 3 years ago

  Hi I noticed some things in your site that you could change and improuve.

  • Your container have height in the desktop view , it couses problems like the content of it get not aligned the way you wanted.
  • You didn't put a width to your card so it gets (fuzzy? shaken? idk sorry my english) cuz the card tries to fit the content every time the content rearrange.
  • There is a lot of unecessary @media, witch in this design can be removed cuz it has one form only.
  • You didn't style the ancor links.
  • Try to use for images rem, em or just px so it dont fade away.
  • Your container dont need a width or even a height as I said. (changing it resolve most of the visual problems, but the card needs a fixed width to work).

  i can't help with the bg cuz it is hard for me also >o< well, hope i helped you in anyway. :D

  1

  This comment has received 1 replyShow
• 

  Pedro Miguel•175

  @pedromigueldevSubmitted over 3 years ago

  Nothing in particular but I'd really apreciate any tips :]

  

  I've used Grid(finally lernt) and flex-box and scss

  1

  10
  

  Pedro Miguel•175

  @pedromigueldevPosted over 3 years ago

  i think i didn put the right link, what do i do...? https://stats-preview-card-component-main-gfmrp3opk-pedromiguel-dev.vercel.app

  0