Skip to content
OverviewSolutions
4
Comments
4
Aurore GOURRAUD
@AuroreTurtle

All comments

  • Aurore GOURRAUD•60
    @AuroreTurtlePosted 2 months ago

    The computer version looks like the model. However, for the lists it's just the brown bullets and not the text. However, I haven't seen the mobile version.

    0
  • Aurore GOURRAUD•60
    @AuroreTurtlePosted 3 months ago

    The code is easy to understand and the design is respected. I like the hover transition. Well done ! On the other hand, the focus is just missing.

    Marked as helpful
    1
  • Napgolden•60
    @NapgoldenSubmitted 3 months ago
    What are you most proud of, and what would you do differently next time?

    I am proud of the fact that I was able to get the code to look as similar to the one given to be replicated. I don't think I would do anything differently next time.

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

    The biggest challenge was importing the font and styling the hover function, I remembered I could search on google for a solution.

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

    The text next to the profile image I was not able to put it directly by the side of the image, Any suggestion would be greatly appreciated.

    Sizing of HTML Elements played a huge role in this challenge.

    1
    Aurore GOURRAUD•60
    @AuroreTurtlePosted 3 months ago

    For the text next to the profile image, you can use flexbox. To avoid repetition in CSS (such as fonts), you can apply the CSS rule to the entire page using *.

    0
  • Aurore GOURRAUD•60
    @AuroreTurtlePosted 4 months ago

    The code is well structured and easy to understand. It's a good idea to use variables in the CSS. However, the font-family is missing for the text. You've imported it into the HTML but it's not used.

    0