Skip to content
Submitted over 4 years ago

Scroll animations with scss

Isy•215
@isy-creates
A solution to the Bookmark landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I'd love to hear some feedback, thank you very much.

Code

Please log in to post a comment

Log in

Community feedback

  • ApplePieGiraffe•30,525
    @ApplePieGiraffe

    Hey, Isy! 👋

    Nice job on this challenge! 👏 Your solution looks good and I think the animations that you added to the page are very nice! 👍

    I'd like to suggest,

    • Making sure the header navigation doesn't jump to the middle of the screen in the under 1200px.
    • Darkening the background of the mobile navigation a bit more so that the navigation links are more readable.
    • Adding text-align: center to the text content under the "Share Your Bookmarks" section in the mobile layout (as in the original design).
    • You have a few issues listed in your solution report. I suggest looking into those and trying to clear a few of them up to make your markup more semantic and your solution more accessible! 😉

    Keep coding (and happy coding, too)! 😁

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