Fidel Lim• 2,775
@fidellim
Posted
Hi @aelhamrawy ,
Great job finishing the project! It looks great on desktop and mobile devices. Some suggestions I would like to share are of the following:
- you can add some padding to your component so that there is some space during mobile views.
- heading tags are not in order. This is important for assistive technology users to navigate the website. It is stated in your report that the heading should only increase by one. So it is preferred that you use h1.
I hope these help :)
Marked as helpful
0
Ahmed Elhamrawy• 170
@aelhamrawy
Posted
Hi @fidellim ,
Thank you very much for your comment , very appreciated your time reviewing my app. I Will work on your suggestions and re-upload the app again.
Best Regards
0