Skip to content
Submitted over 3 years ago

PHOTOSNAP multipage- with Vue3 and Sass

Mohamed Donzo•165
@demahom18
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Photosnap multi-page website
  • HTML
  • CSS
  • JS
4advanced

Design comparison

SolutionDesign

Solution retrospective

This is probably the most advanced website I've made so I would reaaly appreciate any advice to improve my code. It is full responsive with some animations (first time trying gsap). I learned a lot doing this challenge!

Community feedback

  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted over 3 years ago

    Hi there, Mohamed Donzo! 👋

    Good work on this challenge! 👍 This quite a large project and you've done a nice job on it! 🙌 I really like the various animations that you added to elements and the gradient-colored social media icons in the footer of the page! 👏

    I just suggest taking a look at your solution and report and trying to clear up some of the errors that are there.

    Also, there's a very slight horizontal scroll bar that appears along the bottom of the screen in the mobile layout. Adding overflow-x: hidden to the body should get rid of it, though. 😉

    Keep coding (and happy coding, too)! 😁

    1
  • Mohamed Donzo•165
    @demahom18Posted over 3 years ago

    Hi ApplePie, Thank you for your feedback. I fixed those errors but there are still 2 remaining. I don't really think they are important. One is because there 's no text inside a 'a' tag (it is the svg logo) and the second one is because the animation does not display the text before you scroll to it.

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord