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

Submitted

Bookmark landing page

@Yejin-Han

Desktop design screenshot for the Bookmark landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

It took too much time because there was something unexpectedly blocked in the small areas. I'm proud to have implemented it similarly.

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

I've been struggling with the background behind the illustration, so I'm barely close to it, but from now on, I'll have to look at how to mark up and start from the beginning.

What specific areas of your project would you like help with?

Anything you have found. How long will it take to finish normally?

Community feedback

BT738844 600

@BT453567

Posted

Hello

Excellent solution. There are many parts I like about this solution.

I have identified a small issue where the logo is not visible in the mobile menu.

If you uncheck the text indent the logo is visible in the menu.

Rather than re-creating the facebook and twitter logo for the mobile menu, you could manipulate the DOM from Javascript, by using the following commands (This is taken from my solution):

footerSocialsContainer.remove(); mobileMenuFooterContainer.appendChild(footerSocialsContainer);

I like how you calculated the distance of the graphic to edge of the screen and also how you implemented the tab buttons.

I also found it took longer than expected to code.

Hope that helps

Marked as helpful

0

@Yejin-Han

Posted

@BT453567 Thank you for your feedback! I miss that logo visibility issue. I fix it. Thanks :) I didn't manipulate the DOM because I thought it would be bad in terms of performance or maintenance. But I'll try it next time.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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