Skip to content
OverviewSolutions
16
Comments
8
P
Fadya
@MaxCoder-mc

All comments

  • Fadya•220
    @MaxCoder-mcPosted about 2 months ago

    Hello there...

    I have few notes that might enhance your solution...

    • on big screen sizes you may have noticed that the h1 is getting narrow from the very beginning, probably because of (width: 50%) you set to it. maybe by putting gaps between the (.hero) elements would solve the problem.
    • The background image of (.footer) would look nicer if you set the position to top center like this--> background-position: top center;
    • Usually we use anchors instead of buttons if the action of it needs to take us to a different web page or have a certain URL...

    I hope these notes were helpful to you... Enjoy your coding :)

    0
  • Fadya•220
    @MaxCoder-mcPosted 3 months ago

    nice

    0
  • Fadya•220
    @MaxCoder-mcPosted 3 months ago

    Hello there... You did good job in your design but, you might want to:

    • Reduce the line-height of the header title.
    • Reduce the height of the articles.
    • Add some more margin to the top of your main.
    0
  • Fadya•220
    @MaxCoder-mcPosted 4 months ago

    Nicely done, I have only one note though to reduce code lines and make it more convenient...

    • you could remove border-radius from your images and put "overflow: hidden;" into your card class.
    Marked as helpful
    0
  • Fadya•220
    @MaxCoder-mcPosted 5 months ago

    hello there... it's a good solution you made, but i think you forgot to change the (headlines, numbering, and second column of the table) color. I couldn't access your code as it says the page it not found

    0
  • wey song•130
    @teuthixSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    Getting the hover animation to work! I would like to try other animations in the future.

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

    The video I found to reference/learn how to do the animation didn't work with my code exactly, I had to figure out how to get it to work with the z-index property.

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

    If there is any way you would restructure the code or write differently I would love to know!

    Social Links Profile

    1
    Fadya•220
    @MaxCoder-mcPosted 5 months ago

    Your design looks nice, i liked the animation you made... if only you could make it little bit smaller to be more mobile responsive

    0
  • sharangb•60
    @sharangbSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I feel proud about being able to complete the challenge in 1 hour.

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

    I did not encounter any major challenges.

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

    I don't need any help for this challenge.

    Blog Preview Card using HTML, CSS

    1
    Fadya•220
    @MaxCoder-mcPosted 5 months ago

    Nice work

    0
  • André•30
    @AndreCode2093Submitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    Using some preprocessor or maybe tailwindcss

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

    The use of media queries, im just starting with these concepts

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

    Maybe how to setup sass

    Qr Code Component using Flex and Grid

    2
    Fadya•220
    @MaxCoder-mcPosted 5 months ago

    Hello there!

    Congrats on completing the challenge! Your solution is really good.

    You could shorten this line of your code --> margin: 0 2rem 0 2rem; into this one --> margin: 0 2rem
    It gives the same result and save you some time

    As for the help on setup sass, you try this tutorial https://www.youtube.com/watch?v=jfMHA8SqUL4

    Hope that was helpful

    Good luck for your next challenge

    0