Hamid Abdalla (HA)• 250
@hamid-abdallah-mohammed
Submitted
Dev saidAli
@Dev-saeedAli
All comments
- Dev saidAli• 400
@Dev-saeedAli
Posted
Hi hamid-abdallah-mohammed nice work solving the solution. You can provide sementic HTML like
<header> <main> <section>, also putting all<a>inside the<ul><li><a href="#"></li></ul>which exist in the footer also provide aria-label to the buttons and also addaltto all images. instead of addingelement.style.bg and font sizefrom js you can add it to a class and can useelement.classlist.add(classname)Looks nice just a suggestion...Marked as helpful
1 - Monica• 260
@Monica-MR
Submitted
Hi, Here is my solution for this landing page.
I guess 'hover' in mobile and tablet version is useless right? Thank you.
Dev saidAli• 400@Dev-saeedAli
Posted
good work.
1 @rica999
Submitted
Hi everyone. I finished a new challenge. I hope your recommendations. Thanks :D
Dev saidAli• 400@Dev-saeedAli
Posted
good work.
1- Xin Chen• 240
@xchen1778
Submitted
Dev saidAli• 400@Dev-saeedAli
Posted
nice.
0 - Riha Shehzadi• 80
@codingwithriha
Submitted
I would like your lovely feedback.
Dev saidAli• 400@Dev-saeedAli
Posted
pictures are not visible.
Marked as helpful
1 - Siddhant Sharma• 70
@siddhants1
Submitted
Hi all. This is my first time building a landing page. Making the navbar responsive with JavaScript was a tough task for me. I would appreciate feedback from community on how can I style the footer better, how can I make text styles better. Thanks!
Dev saidAli• 400@Dev-saeedAli
Posted
Good work.
Marked as helpful
0
