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

Fylo Landing Page with Changeable Themes

Agnik Bakshiβ€’ 480

@Agnik7

Desktop design screenshot for the Fylo dark theme landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi everyone, This is my solution for the Fylo Dark Theme Landing Page. I have tried to stick to the design as closely as possible, however, I made some changes here and there.

I have also added a custom them changer to toggle between dark and light mode.

Any feedback or comment is always welcome and appreciated.

Community feedback

Abdul Khaliq πŸš€β€’ 72,580

@0xabdulkhaliq

Posted

Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

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

ANCHOR ELEMENT πŸ”΄:

  • The <a> elements lacks aria-label attribute which is way more important for social links in an <a> tag can help provide more context to users with visual impairments who use assistive technologies such as screen readers to access your website.
  • When a screen reader encounters an anchor tag with a social link, it may announce the link's text content, such as "Facebook" or "Twitter," by including an aria-label attribute that points to a nearby element containing a description of the link's purpose, you can provide more context and clarity to the user.
  • By providing this additional information, you can help users with visual impairments to better understand the purpose and value of social links, and encourage them to engage with your content. This can ultimately improve the user experience on your website, and make it more accessible and inclusive for all users.
  • Example:
<a href="/"  aria-label="Facebook profile of Fylo">
<i class="fa-brands fa-facebook-f"></i>
</a>
  • If you have any questions or need further clarification, you can always check out my submission 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

1

Agnik Bakshiβ€’ 480

@Agnik7

Posted

@0xAbdulKhalid Hi, Thanks for bringing this to my attention, I will surely make the necessary changes so that it becomes even better. Your comment was really helpful!!!

Have a nice dayπŸ˜„ πŸ˜„

Happy coding!!

0
_nehalπŸ’Žβ€’ 6,710

@NehalSahu8055

Posted

Hello Coder πŸ‘‹.

Congratulations on successfully completing the challenge! πŸŽ‰

Your custom toggling feature looks cool.

Just a small issue I found.

  • In my screen size 1236px your footer whole content is not visible, some part is hiding to the right side.
  • Try to fix it.

I hope you find this helpful. Above all, the solution you submitted is great !

Happy codingπŸ˜„

0

Agnik Bakshiβ€’ 480

@Agnik7

Posted

@NehalSahu8055 Thanks for bringing this to my attention. I have fixed the following issue.

Have a nice dayπŸ˜„

Happy Coding!!

1

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