Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Mennatallaā€¢ 90

    @Mennatalla-Khougha

    Submitted

    What are you most proud of, and what would you do differently next time?

    Figuring out the right division for me to use

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

    Getting the typography as close to the design as possible

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

    none

    P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    Not much to write here since you didn't have any questions.

    Very clean and accessible code. Keep up the great work!

    0
  • P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    Just a few suggestions:

    • For accessibility, your index.html should have a <main></main> tag wrapped around the content, right in between the <body></body> tag.
    • If you put the image in it's own container, you have a little bit more control on the width x height.
    • Try using the font-family suggested to get it closer to what the design looks.

    Marked as helpful

    0
  • Anthony Becarneā€¢ 160

    @Abecarne

    Submitted

    What are you most proud of, and what would you do differently next time?

    At first, I really thought that this challenge was too hard for me to achieve. But I learned new HTML and CSS stuff and I think that the final result is good enough.

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

    I hardly managed to make it responsive and asked chat gpt for some help :)

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

    Any feedback would be appreciated :)

    P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    Great job! My only suggestion would be to work more with the spacing if you want it to look as similar to the design.

    1
  • P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    My advice and feedback would be:

    1. Be a little more descriptive with your class names.
    2. Try using an <a> tag for the links seeing as they can be clicked and would direct you to a different page.
    3. The color change to yellow should be on the ":hover" state of the links instead of active. (I might be wrong about this, gotta go double check the design).
    4. Take a look into using Flexbox or CSS Grid. Once you understand it, it's way easier to create this layout without having to use fixed heights and widths and pay attention to the padding and spacing between elements.

    Great job and keep up the good work!

    Marked as helpful

    1
  • P
    Magsā€¢ 270

    @Dshiza

    Submitted

    What are you most proud of, and what would you do differently next time?

    a

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

    a

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

    a

    P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    Very great job! I have no feedback besides checking your HTML to make sure it's semantic. I noticed you have the <main> element inside the .wrapper <div> and not sure if that is correct.

    0
  • Amirrezaā€¢ 270

    @AmirezaMahmoudi

    Submitted

    What are you most proud of, and what would you do differently next time?

    This challenge was really easy i tried my best to make my solution as close as possible to the original design.

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

    this challenge wasn't much of challenge

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

    i want to know if i overlooked something

    P
    Franklin Gomezā€¢ 160

    @frankgomezdev

    Posted

    I would review the HTML that you used for this. I noticed <main> is sitting inside of a <div> and that doesn't seem right. It's affecting the padding of the content inside, specifically the left & right padding. Fixing this would make your solution look even closer to what the original design is. Great job regardless and I hope you find this helpful!

    Marked as helpful

    2