Article preview component | HTML CSS & JS | Flexbox & Grid

Community feedback

• 

  Szymon Rojek•4,540

  @SzymonRojekPosted almost 4 years ago

  MasterKrab

  Your solution looks nice.

  @alnahian2003 has got right - the box-shadow is too strong.

  You have used the tag main. Ok, but maybe it will be better to change the name of the selector to “card”, what do you think? Also, the main selector has got border-radius 1em, then profile social—active is set to 0.5em. I would do it equally. Check it out.

  Also, I am just thinking 🤔 maybe I would use the tag section here and inside the article tag but I don’t want to make it more complicated.

  You have got two index.html in your repository and also unnecessary files can go to .gitignore.

  Ps. don't forget to upvote any comments on here that you find helpful.

  Many greetings :D

  3

  ---

  

  MasterKrab•940

  @MasterKrabPosted almost 4 years ago

  Thank you :)

  0
• 

  Al Nahian•775

  @alnahian2003Posted almost 4 years ago

  Hey, Looks good & nice. But would you mind to decrease the opacity of the box-shadow? It looks a little bit heavy.

  Keep going <3

  Happy Coding :)

  2

  ---

  

  MasterKrab•940

  @MasterKrabPosted almost 4 years ago

  Thank you :)

  0

  

  Al Nahian•775

  @alnahian2003Posted almost 4 years ago

  It's my pleasure

  1