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, Vite + React + Tailwind

Raza Abbasโ€ข 790

@RazaAbbas62

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


Any Feedback or suggestions are welcome :)

Community feedback

Theunisโ€ข 210

@theYuun

Posted

So I mentioned your reduction of whitespace on a page in another project being for the better, but here is an example of where it can be of use. The top margins on your sections are a bit too tight, making it look like the headings are a part of the previous section. Otherwise, great work!

Marked as helpful

1
Petrit Nurediniโ€ข 2,860

@petritnuredini

Posted

Congratulations on completing your React project! It's great to see your dedication to learning and building with React. Here are a few best practices and suggestions to consider for your next steps:

  1. Component Structure and Props:

    • Your components are well-organized, which is excellent. To enhance this, consider passing data to components via props, especially when the data might change or come from an external source. This makes your components more reusable and easier to test. For more on this, check out React Props.
  2. State Management in Functional Components:

    • You're using React Hooks effectively for state management, which is a good practice. Ensure to keep state logic encapsulated within components where it's most relevant. Sometimes, larger apps benefit from centralized state management using Context API or Redux. Hereโ€™s a guide for managing state in React: State and Lifecycle.
  3. Accessibility and Semantic HTML:

    • Pay attention to accessibility. Ensure that interactive elements are accessible, use semantic HTML, and make sure that your website is navigable via keyboard. Accessibility is crucial for a wide range of users. Learn more about accessibility in React here: Accessibility in React.
  4. Code Consistency and Cleanliness:

    • Keep your code clean and consistent. This includes consistent naming conventions, proper indentation, and avoiding unnecessary code or comments. Tools like Prettier or ESLint can help maintain consistency. Check out Prettier for code formatting.

Keep up the excellent work. Every project is a step forward in your development journey. Happy coding and best of luck in your future projects! ๐Ÿ‘จโ€๐Ÿ’ป๐Ÿš€

Marked as helpful

1
Abdul Khaliq ๐Ÿš€โ€ข 72,620

@0xabdulkhaliq

Posted

Assalamu Alaikum Sister ๐Ÿ‘‹

Congratulations on successfully completing the challenge! ๐ŸŽ‰

  • I have a recommendation regarding your code that I believe will be of great interest to you.

ACCORDION ๐Ÿ”ด :

  • You have done a best job in replicating the solution as close to the design, congrats for that.
  • One thing that i noticed is the way you implemented the Accordions for FAQ Section. Which is an in-accessible one.
  • The best way to go with creating the accordion elements in this challenge would be with the details and summary elements (or perhaps a combination of buttons and other elements).
  • They are already fairly accessible and provided a clean, semantic way to create accordion elements. I see you have used the div & p elements for the accordions, but those are not interactive or accessible by keyboard, so not all users will be able to open the accordions to see the content inside.
  • MDN's reference is a great place to start learning about the details and summary elements if you are interested.
  • If you have any questions or need further clarification, you can always check out my submission for a newbie challenge (which is archived now) and/or feel free to reach out to me.

.

I hope you find this helpful ๐Ÿ˜„ Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

0

Raza Abbasโ€ข 790

@RazaAbbas62

Posted

Thanks, bro @0xabdulkhalid for taking the time to check out my code and provide me with those valuable suggestions. I will surely keep them in mind while doing other projects.

Thanks

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