All comments

• 

  P

  Noemi Gali•280

  @NomiDomiSubmitted almost 3 years ago

  Hello guys,

  This is my second attempt to recreate this design. This was one of my first designs to implement before I was fully familiar with concepts like:

  • keeping your code DRY
  • using the right units
  • CSS variables

  Appreciate any feedback! :)

  

  Responsive mobile-first nft card component

  3

  100
  

  Favour Christopher•90

  @phaevourkhrisPosted almost 3 years ago

  Hi Noemi, I think you really did a good job. Goodluck

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Mohamed Abdessamed•140

  @Brivan-26Submitted almost 3 years ago

  I really enjoyed solving this challenge where I applied the grid system, I used only 1 grid container, and I displayed sub-cards using the great command: grid-template-areas. I'm wondering if you could use breakpoints under 750px because I used 960px(under this view, the cards will be adjoining Your comments and criticisms are welcome

  

  Responsive Testimonials grid using only 1 grid container

  3

  20
  

  Favour Christopher•90

  @phaevourkhrisPosted almost 3 years ago

  Hi @Brivan-26 you did a great job. And yea, I think you can use breakpoint under 750px, it all depends on what you want to do at those breakpoints.

  Also, see if you can fix the accessibility issues indicated in the report.

  Goodluck

  0
• 

  Favour Christopher•90

  @phaevourkhrisSubmitted almost 3 years ago

  I'd really love a feedback. I challenged myself to do this with ReactJs. Let me know if you think I delivered this properly. Is there something you feel I should have done differently?

  

  Responsive URL shortening website using ReactJS and SCSS

  3

  40
  

  Favour Christopher•90

  @phaevourkhrisPosted almost 3 years ago

  Hi @skyv26 thanks for the feedback, but I don't quite follow.

  There's barely any sections relating to the menu buttons in the nav bar, that is why they weren't linked. Seeing as we only worked on the landing page. I assumed that those menus will link to other pages.

  Also, From the design, I don't quite see the social media icons aligned to the center for the desktop screen. Except you're referring to a smaller screen size. So if you'd please make me understand what you mean better, I'd appreciate that.

  Thank you

  0

  This comment has received 2 repliesShow
• 

  KrzysztofPa•10

  @KrzysztofPaSubmitted about 4 years ago

  It was my first Mobile First - project, is it done correctly

  

  Mobile First, SASS Social Proof Section

  1

  11
  

  Favour Christopher•90

  @phaevourkhrisPosted about 4 years ago

  Hi Krzysztofpa, I think you did pretty well! Happy coding!

  0