Skip to content
OverviewSolutions
3
Comments
5
Chris Conner
@cconner57

All comments

  • Nikhil Verma•80
    @iamnikhilvermaSubmitted over 3 years ago

    This is my first project on frontend mentor, and Should I need to improve this code I would be very grateful if you could provide some feedback on my code I learned some new stuff while doing it. Have a good day !

    profile card

    1
    Chris Conner•115
    @cconner57Posted over 3 years ago

    Hi Nikhil,

    Everything looks good, the project and your code are clean and readable. Great job!

    0
  • Radu Andrei Mihaila•35
    @gloriellSubmitted over 3 years ago

    Accordion with SASS and BEM made...if you have any suggestions please go ahead and tell me what i could have done better, especially in the JS part.

    I used SASS and BEM

    2
    Chris Conner•115
    @cconner57Posted over 3 years ago

    Hi Radu,

    Great job on the project. Everything looks great and your coding looks good! Only criticism I have that the container's height and width are a lot bigger than the example's height and width

    1
  • s4m-s3pi0l•20
    @s4m-s3pi0lSubmitted over 3 years ago

    Hi, this is my first solution to a challenge here. Also my first time using css flexbox. Any feedback would be apreciated!

    Profile Card - HTML/CSS

    1
    Chris Conner•115
    @cconner57Posted over 3 years ago

    Awesome job! 👏

    This was your first time using flexbox? You did an excellent job. From looking at your css, I would have thought you've been coding for awhile

    0
  • Chris Conner•115
    @cconner57Posted over 3 years ago

    Everything looks great. Might want to make the form a little taller, but still looks good 👍🏻

    1
  • Chris Conner•115
    @cconner57Posted over 3 years ago

    Hi Carol,

    Awesome project! Your transitions and animations look great

    1