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

Responsive Landing page using Flex and Sass, and interactions using JS

Francelβ€’ 370

@Haemoffy

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 feedbacks will be acknowledge, especially any feedbacks about my usage of ARIA attributes

Community feedback

Roc Tanweerβ€’ 2,500

@RocTanweer

Posted

And about aria...

You have used aria-describedby quite often but you may have got confused and did everything opposite of how it was suppose to be...

Lemmi explain...

Describedby simply mean something is described by something else

When you put that aria on an element, then it will take the id of another element that is describing that element and you have been doing the opposite...

A paragraph will describe a heading and not the opposite...So

You should put aria-describedby="id of paragraph" on heading...

Good luck

You can visit MDN for more info

1

Francelβ€’ 370

@Haemoffy

Posted

@RocTanweer Oh, probaby got confused of the word "by", I keep this wonderful explanation in mind, thank you very much for the response

1
Roc Tanweerβ€’ 2,500

@RocTanweer

Posted

Hello πŸ‘‹

Great work πŸ‘Œ and quite extensive use of arias...

I just reviewed you index.html file and it was really impressive..! I also made the same project but your work is too good...

But I wish to point out some things...

  • In feature tab section, the tab changers are suppose to be buttons and not h tags... Because users are clicking on them and something is happening

  • When you put an icon in a link, you should put a span in the link with text describing the icon and adding sr-only class to that span. All these for accessibility purpose.

<a><span class="sr-only">Facebook icon</span>icon here...</a>

You can download sr-only class from Google...

Hope it helps and happy coding πŸ˜‰

1

Francelβ€’ 370

@Haemoffy

Posted

@RocTanweer thank you very much for the feedback , i didnt know sr-only exists, this will definitely do a big help for my future projects

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