All comments

• 

  Azzy dvyastia kesuma•1,345

  @azzykesumaSubmitted over 3 years ago

  i am having difficulty in adjusting the color for the image. i know that using css filter will do the trick, but i don't know for sure which filter is suitable for this project

  

  responsive stat preview using CSS and HTML only

  2

  10
  

  minthing•145

  @minthingPosted over 3 years ago

  Hello, Azzy! Nice Work! ✨

  I'm not good at English too... but here is my suggestion!

  when display size is around 1200px, the section part are not in center so i think it's better to give width(like 200px) .section and give margin: 0 auto and .section > p{text-align:center}

  and give margin : 0 auto to p under title

  Happy Coding! :)

  0

  This comment has received 1 replyShow
• 

  Fabio Augusto•50

  @FabioAugustoDamettoSubmitted over 3 years ago

  Any feedback ll be welcome / Qualquer feedback será bem-vindo :)

  

  React.js - CSS Grid - Responsive

  3

  10
  

  minthing•145

  @minthingPosted over 3 years ago

  Hello! Nice Work Fabio!

  Here is my suggestion, if i do 2*0 = 0 and press any other number, then 0 doesn't dissapear like 02. butI think it's better to be 2, not 02.

  0

  This comment has received 1 replyShow
• 

  Gonza Zabaleta•155

  @gonzabaletaSubmitted over 3 years ago

  For some reason, when I hover the buttons they get a little bit bigger (and I think the text inside of them gets a bit smaller too). How can I fix this?

  Thank you!

  

  3-column preview card component using vanilla CSS Mobile-First

  1

  10
  

  minthing•145

  @minthingPosted over 3 years ago

  The reason is that you only gave the border only when the button was hovering or it's display is inline-block. if you want to fix this with box-sizing, make sure that the button should be display:block; or it can't have height or width

  nice work, Gonza ✨

  0

  This comment has received 1 replyShow
• 

  Haldhar Dwivedi•10

  @HaldhardwivediSubmitted over 3 years ago

  Any kind of feedback is highly appreciated

  

  i used flex and media query to handle the mobile view

  1

  10
  

  minthing•145

  @minthingPosted over 3 years ago

  Nice work! Haldhar ✨

  Here's my opinion!

  How about to use justify-content:center in wrapper class? (It makes set the element center easier)

  And I think it’s better not give height:75% the 3 columns, it makes the box size change different

  cheer! :)

  1

  This comment has received 1 replyShow
• 

  Art•420

  @Art-wdtSubmitted over 3 years ago

  Hey! This is my new work. I would be glad to any advice:) Any suggestions you might have! Any comments!

  Feedback definitely appreciated

  

  3-column card with Flex, BEM, HTML5, CSS3

  2

  10
  

  minthing•145

  @minthingPosted over 3 years ago

  container is not centerd (if the display is bigger than 1552) how about use justify-content: center and remove margin:50px in wrapper class?

  nice work! cheer! ✨

  1

  This comment has received 1 replyShow
• 

  farildsen•55

  @farildsenSubmitted over 3 years ago

  Hi fellow frontenders,

  This is my first challenge on Frontend Mentor Feel free to give any feedback or constructive criticism :)

  Have a wonderful day

  

  HTML, CSS, JS

  2

  50
  

  minthing•145

  @minthingPosted over 3 years ago

  if multiply 2 by 0 and then 0 still remains when i press another number key and i think it's better to add ellipsis and overflow:hidden when the number size gonna bigger than the result box

  love your work, cheer!

  0

  This comment has received 2 repliesShow