Skip to content
OverviewSolutions
3
Comments
3
rjzhao1
@rjzhao1

All comments

  • Anthony Chaiditya•70
    @AnthonyChaidityaSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    I'm most proud of the way I structured the HTML for this page - I feel like it really helped the styling to be much easier. I am also happy with the way that I approached the problem. What I would do differently is I would use media queries for responsiveness based on screen size - as of right now if the screen gets too small, the buttons stay the same size and outsize the card container.

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

    The most challenging part about this challenge for me was the spacing between elements on the card. To overcome this, I made sure that I was keeping track of where I was implementing margins and padding to the content.

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

    I would like help with responsive design! I would like to know how to use media queries the correct way, and I would also like help with making sure that the card container and buttons shrink with the change in layout.

    Social Media Links using CSS and HTML

    1
    rjzhao1•50
    @rjzhao1Posted 6 months ago

    Styling and main file is very well organized.

    Profile pic link is broken. Profile Title should be bolded and color is slightly different. Otherwise looks good.

    0
  • Kiman•80
    @kimankashiSubmitted 6 months ago

    kim

    1
    rjzhao1•50
    @rjzhao1Posted 6 months ago

    Styles file could be better organized. Instead of first-wrapper, second-wrapper, etc name it based on the components it is used it.

    Layout looks good on different screen size. Code is well structured

    Card is missing outline on the border. Font-style is different from the design. Import the 'Figtree' font as described in the style guide. Avatar image is not scaled correctly and there should be spacing between the image and the name. Also the padding of the card should be less.

    0
  • rjzhao1•50
    @rjzhao1Posted 6 months ago

    Code is well-structured and readable with clear styling files.

    Layout looks good on multiple sizes. Tested on both desktop and mobile sizing.

    Sizing of the card can be bigger to match the design. Other wise looks proportional to the design.

    Marked as helpful
    0