Skip to content
OverviewSolutions
4
Comments
4

saintnic84

@saintnic84120 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • saintnic84•120
    @saintnic84Posted 5 months ago

    Your code looks pretty code. There are only a few real suggestions I think that would help. First would be using line height to improve the space between your list elements, so that they're not so close together. Secondly, you used the wrong color for your hr borders. You used hsl(30, 54%, 90%), instead of hsl(30, 18%, 87%), which I think was a simple mistake. Keep up the great work!

    Marked as helpful
    0
  • P
    Harta Shuwanto•140
    @hartashuSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I am introduced with BEM naming convention and semantic HTML, and gradually implement them and try to get used to them.

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

    There is always a little gap with tag and tag when the display property not set to block. Set the display to block, solve the problem.

    For flex items, if we want to let the item to stretch to its max width, don't forget to set the item's flex property to 1.

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

    Need to learn more about:

    • Semantics HTML in depth.
    • Make an efficient, modular, maintainable CSS code to work with a team.

    Responsive social links profile page with mobile-first workflow

    #semantic-ui#bem
    1
    saintnic84•120
    @saintnic84Posted 5 months ago

    This site wants me to peer review, but that's a challenge when there's visually no difference between your design and the solution. The only thing I could say would be to replace the readme.md file with your own, so that any employers or recruiters will see your work, rather than the front end mentor challenge. Excellent job and keep up the good work.

    Marked as helpful
    0
  • saintnic84•120
    @saintnic84Posted 5 months ago

    Overall, you did this pretty much perfectly! The only thing I could suggest would be including a mobile display component within your styles.css code. Keep up the great work!!!

    Marked as helpful
    0
  • Dimitri Mabom•60
    @dimitrimabomSubmitted 10 months ago
    What are you most proud of, and what would you do differently next time?

    I remembered some forgotten CSS attributes, such as flex-grow which has an element in a flex box take up all the remaining space.

    .content {
  flex-grow: 1;
}
    What challenges did you encounter, and how did you overcome them?

    I haven't really overcome any difficulties. I've just gone back to the source to consolidate my foundations so that I can get back on track with my personal projects.

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

    About Style, can I suggest a personal style?

    QR Code Component

    1
    saintnic84•120
    @saintnic84Posted 10 months ago

    Honestly, this looks great, and you even included the github icon inside your design. No real feedback because even though it's not to specification, it looks perfect

    Marked as helpful
    0