Ham

Latest comments

• 

  Thais•255

  @thaiscodeSubmitted over 2 years ago

  

  Article preview component using flexbox and js vanilla

  1

  50
  

  Ham•340

  @hmac100Posted 7 months ago

  Hi nice job 👍. Really like the breakpoints - very smooth transitions. If anything - Would be good to have the popup positioned in a way that's uniform from the share__button element so it's in the same position relative to it across all non mobile screen sizes.
  Would also be nice to have a simple toggle to hide the popup after a button re click maybe ? function displaySocials() { socials.classList.toggle("hide"); }

  But great job - you're a lot further down the road than I am - well done.

  Marked as helpful

  0
• 

  Abhishek•30

  @Abhishek-pakaSubmitted 8 months ago

  

  QR Component

  1

  00
  

  Ham•340

  @hmac100Posted 8 months ago

  Hi - looks pretty good - but I'd probably try to match the border radius on the qr code image with the card container - as in the design. Also perhaps a touch more bottom padding or bottom margin on the body text to match the space on the design - but well done - still looks cool !

  Marked as helpful

  0
• 

  Abdallah Aboelsaoud•220

  @Saoud2021Submitted 9 months ago

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

  i learned alot during this project and i even add some tweaks in the learning tag and the profile picture of the author

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

  the challenge was in the text as i wasn't quite sure how to categories them but i was able to add the arrow which points to the child directly

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

  i think my projects are wide and it's scale is larger than the original designs and i am not sure how to scale it down a little bit

  

  Blog preview card with some tweaks

  4

  50
  

  Ham•340

  @hmac100Posted 9 months ago

  Hi Abdullah Looks pretty good to me - I reckon the max-width suggestion from Krushna is a really good idea I noticed the avatar and author name are a bit out of alignment . For what it's worth - I wrapped both in a container - that seemed to do the trick.

   <div class="author">
   src="./assets/images/image-avatar.webp" alt="avatar of james hooper"/>
  <div class="author-name">Greg Hooper</div>
  </div> 
  

  Then used display:flex and align-items:center on the .author class

  Well done 👍

  0
• 

  Juan•570

  @JEWebDevSubmitted 9 months ago

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

  I'm very proud that i finished this challenge. I still have lots to learn but it was very fun.

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

  I didn't knew how to use media queries and downloaded fonts. I searched web.dev page for learning responsive design and MDN for how to use downloaded fonts.

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

  Any feedback is appreciated. Have a nice day!

  

  Responsive recipe page using flexbox with vanila CSS and HTML

  2

  50
  

  Ham•340

  @hmac100Posted 9 months ago

  Hi Juan - Well done 👍 With the ingredients list you can do ** list-style: disc **; to get it closer to the design - if that's what you want to do. Still looks ok the way you have it. I found the Instruction list the trickiest to try styling the numbers by adding the colour. I ended up using a thing called my-awesome-counter which is a free 3rd party bit of css that helps you style the numbers. There's bound to be another way to do it but that really helped me out. All the best :)

  0
• 

  P

  Prem Kumar•370

  @prem-kumartSubmitted 9 months ago

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

  • Work-flow improved greatly when using grid and flex box.

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

  • alining the photo and name ; but flex box did the trick.

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

  • how can I improve organization of code and increased minimalism.

  

  responsive-testimonial-grid

  1

  40
  

  Ham•340

  @hmac100Posted 9 months ago

  Hi Prem overall looks good 👍 - I noticed no box shadow on the white card areas - I'm assuming it was a design element you didn't want include ? To me the breakpoint for smallscreen / laptop devices seems to kick in really early but perhaps more importantly the mobile appears to need a lot more width (as the design indicates) which would make it much better for readability - but well done.

  Marked as helpful

  0
• 

  Marcelina•140

  @marce7inaSubmitted 9 months ago

  

  Four card feature section Grid

  1

  50
  

  Ham•340

  @hmac100Posted 9 months ago

  Nice job - Have you considered also using css custom variables in the your font settings and from my limited knowledge I'm not sure of the rationale for using <list> tags.

  1