Skip to content
OverviewSolutions
18
Comments
3
HutHut
@j-hutchison

All comments

  • HutHut•310
    @j-hutchisonPosted over 2 years ago

    Hi Akshita,

    Great job on this component!

    I'm still on my journey to improve my front-end development skills, but a couple of things that have helped me with structuring my CSS and HTML better are:

    • Give all html elements a class name, this will help simplify your css instead of nesting element selectors --> To help with this you can look into a naming methodology called 'BEM' (Block Element Modifier) - sometimes it can look a little unsightly but i think it is worth it on larger projects and also if you work with SCSS in the future

    Above all great work and keep it up 👍

    Marked as helpful
    0
  • HutHut•310
    @j-hutchisonPosted over 2 years ago

    Hey Great Job!

    One tip for getting your solution proportions correct is to use the Figma files provided, download and unzip the figma design files and then open on the figma website. You can use this to get more detail around sizing and spacing : )

    Overall great job!

    0
  • Devika Sujith•175
    @DCoder18Submitted over 2 years ago

    This is attempt two of this challenge. I found this project particularly challenging, especially positioning the elements in the box section responsively. With the help of responsive CSS properties like clamp() and min(), I am quite satisfied with my solution now.

    Feel free to leave suggestions or feedback. Happy to keep improving :)

    Responsive landing page using CSS Grid and Flexbox

    1
    HutHut•310
    @j-hutchisonPosted over 2 years ago

    Great job! Had a look through this, and hope i can apply some of what you've done to my own solution! Your solution works great and scales well!

    Couple of small things:

    • hover states not added
    • footer is commented out
    Marked as helpful
    1