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

  • Alpaar 115

    @AlpaarX

    Posted

    Looks great! But here are some points I would improve:

    • spacing between text elements is too big when viewing the website on fullscreen.

    Hopefully it will help you improve your project!

    0
  • @ChimiRinzin-HWR

    Submitted

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

    From a comment made on from my previous work, I learnt how to properly center HTML elements and also learnt to properly design a web page.

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

    Initially, it was really difficult to arrange the elements according to what I wanted but after patiently experimenting and tweaking with the CSS, I was able to easily arrange all the items according to my desires.

    Alpaar 115

    @AlpaarX

    Posted

    Looks really great! There are some minor fixes that you might apply in your future projects:

    • ol li::marker, ul li::marker{ color: var(--color-brown-800) } Here you don't need to specify the type of list unless you are applying different styles, just li::marker is enough.

    • for sizing and padding/margin try to use relative values as they give you more consistency and potentially responsiveness.

    I hope these advices will be helpful for you!

    Marked as helpful

    1
  • Alpaar 115

    @AlpaarX

    Posted

    Here are some points that might help you improve your project

    • Use shorter duration for transition
    • Use relative measurements for sizing text and elements (em, rem, %)
    • Use CSS variables to assign colors, it might help you change colors by changing code only in one place, instead of changing it on every element

    Marked as helpful

    0
  • Alpaar 115

    @AlpaarX

    Posted

    Too many wrapping containers just for 1 child element. An element can be used as a container itself.

    Failed to find styles file, even though it's mentioned in html code. Please make sure that all folders in your GitHub repo are visible.

    0
  • Alpaar 115

    @AlpaarX

    Posted

    It looks good overall, but CSS for the card is a little overcomplicated in my opinion, could've made it simpler.

    0
  • Alpaar 115

    @AlpaarX

    Posted

    you need to use border-radius: top left bottom right; in order to get rounder corners wherever you want.

    Marked as helpful

    0