Skip to content
OverviewSolutions
10
Comments
8
Dalliger
@jpbascon

All comments

  • Dalliger•170
    @jpbascon

    Looks identical! But small details like fonts, font size, padding, and border radius sometimes can be overlooked. These small details plays a crucial role in front-end development and can definitely transform your front-end skills. I highly suggest you to tweak it. Other than that, everything works fine! Good job

  • Dalliger•170
    @jpbascon

    You did a great job copying the design and nailing the functionalities of each time frame!

    However, in terms of the design, there's a lot of room for improvement. I highly recommend you to explore CSS more so that you have a solid understanding on web-designing.

    Overall, fantastic job! Keep the reps going!

  • Dalliger•170
    @jpbascon

    You absolutely nailed the design! Well done!

  • Mel Ignacio•220
    @decsa17
    What are you most proud of, and what would you do differently next time?

    use of responsive units(rem) and flexbox

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

    N/a

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

    N/a

    HTML, CSS, Flexbox

    1
    Dalliger•170
    @jpbascon

    You did a good job copying the design! Visuals are on point

    One thing I'd like to add is that your buttons, except GitHub and frontend mentor, has no cursor: pointer.

    Assigning cursor:pointer to the element button will prevent this from happening. This also promotes clean code writing as it applies cursor:pointer on all buttons!

    Nonetheless, fantastic work! Keep it going!

    Marked as helpful
  • Dalliger•170
    @jpbascon

    Hi! You did a great job copying the design!

    But I noticed a couple of things: margins are off and the texts are too big. Practice copying the smallest of bits of the design. Doing so will be a day and night difference!

    I know you got it in yourself. I believe you!

  • P
    Michael Andreas•50
    @Michael-Andreas
    What are you most proud of, and what would you do differently next time?

    As im a little experienced and the project is pretty simple i didnt run into any difficulties. I would create a seperate container for the paragraph and heading so that i can change the padding in contrast to the whole card.

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

    The width of the heading and paragraph are a bit narrower than the padding of the card.

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

    None

    QR Code Component

    1
    Dalliger•170
    @jpbascon

    Good job!

  • Dalliger•170
    @jpbascon

    Love the design. Very identical with the reference.

    One thing I'd recommend applying on your design is flex: 1 and min-width for your section containers (left section is the description, right section is the image section) to promote responsiveness when the window is resized.

    You could also try messing with @media to really hone your mobile designing skills. Overall, great job!

  • Dalliger•170
    @jpbascon

    You did a great job copying the design! It looks very identical to the reference.

    One thing you missed is the border-radius and box-shadow. Noticing these kinds of details especially the small ones could make the biggest difference in your design. Take your time observing these little things, and you'll reach the top at no time!

    Overall, fantastic job! Keep up the good work!