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

  • @JesusAtao96

    Submitted

    Hi all 👋

    I decided to do this challenge with Svelte, CSS Grid, Flexbox, BEM and animations.

    Any feedback is appreciated.

    Happy coding 😊.

    @Pustur

    Posted

    Congrats, you got very close to the design 👏

    I would remove the border-left from the cards that are not "featured", but otherwise it looks great.

    Nice animations too, keep it up 😉

    1
  • @Pustur

    Posted

    You put the wrong url in the solution, it should be https://sakthi0314.github.io/Frontend-metor-challages/loopstudios-landing-page-main/src/

    0
  • @Pustur

    Posted

    Good work so far, a few improvements I would make:

    • No focus style for the text input and submit button, consider adding some or leave the default outline so keyboard users have an easier time knowing where they are focused :)
    • When clicking the submit button with an invalid input, an error appears, but it shouldn't continue creating new error messages when I try to submit multiple times. Consider making a single element for the error and just show / hide it when necessary.
    • Impossible to scroll the page and see all the content when the browser window is small vertically.
    1
  • P
    AndyBeable 120

    @AndyBeable

    Submitted

    Hi all,

    A bit of a love/hate relationship with this one! There are a lot of areas I think I could improve. In particular the responsive mobile nav. I've managed to get it working (though i'm sure there's a much better way)!

    Would be great to get some tips on how to do the following, as these are parts I've been going round in circles on:

    • Hover underlines on links in main nav and footer. I did manage to get this working, but for some reason the line would only show up in one place, not under each link. So i've removed for now.
    • Would like to add some hover states to the creations tiles, but again couldn't figure it out.
    • The 'See all' cta, on desktop, I wanted to move it to the top right of the section as per the design, but could only seem to do that using absolute positioning and then that caused issues when the screen-size changed.

    Lots to improve!

    @Pustur

    Posted

    Hey Andy,

    Hover underlines on links in main nav and footer. 
    I did manage to get this working,
    but for some reason the line would only show up in one place, not under each link. 
    So i've removed for now.
    

    Sounds like you positioned the lines with position: absolute; but the links didn't have position: relative;? So they were all relative to some other common parent element. If that's the case you should add position: relative; to your links.

    Would like to add some hover states to the creations tiles, 
    but again couldn't figure it out.
    

    I did it using ::before and ::after, one was for a black gradient to improve readability of the white text, the other is a white overlay that shows up on hover/focus. I did this with the position: relative; in the parent and position: absolute; in the pseudo element.

    The 'See all' cta, on desktop,
    I wanted to move it to the top right of the section as per the design,
    but could only seem to do that using absolute positioning and 
    then that caused issues when the screen-size changed.
    

    I used grid and the order property for this, I agree using position: absolute; can sometimes get messy.

    Good work overall, I like the animations on scroll. (Too bad they mess up the screenshot 😅)

    Check out my solution if you want to see how I did the things you asked about.

    4