Dev saidAli
@Dev-saeedAliAll comments
- @hamid-abdallah-mohammedSubmitted over 2 years ago@Dev-saeedAliPosted over 2 years ago
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 addalt
to all images. instead of addingelement.style.bg and font size
from js you can add it to a class and can useelement.classlist.add(classname)
Looks nice just a suggestion...Marked as helpful1 - @Monica-MRSubmitted over 2 years ago
Hi, Here is my solution for this landing page.
I guess 'hover' in mobile and tablet version is useless right? Thank you.
- @rica999Submitted over 2 years ago
Hi everyone. I finished a new challenge. I hope your recommendations. Thanks :D
- @xchen1778Submitted over 2 years ago
- @codingwithrihaSubmitted over 2 years ago
I would like your lovely feedback.
- @siddhants1Submitted over 2 years ago
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!