Dan P.

Latest comments

• 

  Shihab328•20

  @Shihab328Submitted 9 months ago

  

  Order summary component

  1

  30
  

  Dan P.•270

  @DanielfwwPosted 9 months ago

  Hey Shihab, your project looks amazing. If I had to recommend anything here is to change your background. If you put this stuff into body:

  background-repeat:no-repeat; background-color:hsl(225, 100%, 94%);

  You should not have the double background anymore.

  0
• 

  nikhil-0009•210

  @nikhil-0009Submitted 9 months ago

  

  grid_main_project

  1

  20
  

  Dan P.•270

  @DanielfwwPosted 9 months ago

  Hey Nikhil - great job on finishing this project. If I had to recommend anything here, I would suggest you expand a little more on the grid here - make more space for your columns and lower a little bit on the row space and that way they should expand more to the left/right on boxes should become a little smaller.

  If you use grid-area it would be easier to adjust them.

  0
• 

  Mahim•30

  @M-a-h-i-mSubmitted 9 months ago

  

  social-links-profile-main

  1

  30
  

  Dan P.•270

  @DanielfwwPosted 9 months ago

  Hey Learn - great job on finishing this project, it looks really close to the original design. If I had to recommend anything here, I would suggest you would use the following to center your div to the middle of the page:

  body {

  min-height: 100vh;
  display: flex; 
  justify-content: center;
  align-items: center;
  

  }

  Everything else looks great.

  Marked as helpful

  0
• 

  Johnny Haranaka•10

  @johnnyharanakaSubmitted 9 months ago

  

  Only HTML/CSS Social Links

  1

  30
  

  Dan P.•270

  @DanielfwwPosted 9 months ago

  Hey Johnny - great job on finishing this project! If I would recommend anything - it would be adding a: hover into your style.css file and that way your text would change when clicked on

  For example:

  a: hover { color: black; cursor: pointer; }

  Good luck and great job!

  1
• 

  Erfan•310

  @ERFAN-REVENANTSubmitted 9 months ago

  

  nothing fancy

  1

  20
  

  Dan P.•270

  @DanielfwwPosted 9 months ago

  Hey Erfan - great job on finishing this project. If I had to recommend anything here, I would suggest you would use the following to center your div to the middle of the page:

  body {

  min-height: 100vh;
  display: flex; 
  justify-content: center;
  align-items: center;
  

  }

  Everything else looks great!

  1
• 

  razanabbas•150

  @razanabbasSubmitted 10 months ago

  What are you most proud of, and what would you do differently next time?

  I am proud of the progress I am making in my CSS skills, thanks to frontend mentor. I used a CSS reset and a CSS lint as the guy from Discord advised me last time and they turned out to be really helpful. I will start using them more in my upcoming projects.

  What challenges did you encounter, and how did you overcome them?

  I made some stupid mistakes which got me stuck for a while but I reviewed my code several times and discovered them.

  What specific areas of your project would you like help with?

  Nothing specific this time, just any general feedback would be helpful!

  

  Social links profile using HTML and CSS

  2

  40
  

  Dan P.•270

  @DanielfwwPosted 10 months ago

  Hey razanabbas - great job on finishing this project! If I would recommend anything - it would be adding a href into your divs and doing hovers, that way your text would change when clicked on. Everything can be done in html so you dont have to chance your css file. Good luck and great job!

  Marked as helpful

  0