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

  • @BANKOLEDO

    Submitted

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

    I learnt a lot, it was a tough challenge but i did it

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

    when i tried to give a margin between list markers and its text, i overcomed it by utilizing pseudo selectors ::before and flex properties

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

    I am open to corrections

    DuduNube 80

    @DuduNube

    Posted

    it's very well structured. But the fonts seems too flat. You could use @font-faces on your css and and put the fonts that's in assets. For the ol,ul and article, they're perfect. It would be clever to clean more of the divs and use more the tag <hr> as well.

    Marked as helpful

    1
  • DuduNube 80

    @DuduNube

    Posted

    The execution is very good, simple and clear. Keep going ;)

    1
  • @ShoaibShuja

    Submitted

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

    The Layout was easily designed using CSS flexbox.

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

    I was able to import the figma file which was available, and I even run it, but I had absolutely no idea how to inspect the code or properties used in the design.

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

    How can I inspect the properties of the different parts of the layout in the figma file?

    DuduNube 80

    @DuduNube

    Posted

    The executation is perfect. But if you want just more details, you can put the line on the site footer inside the card and add some info about you.

    But that's just a suggestion. :)

    0
  • DuduNube 80

    @DuduNube

    Posted

    I would put the margin-top = 5% to adapt to other types of screens, and put a align item = center. For the border i would put a thin border-radius = 5% and border = solid 1px black

    Marked as helpful

    0