Muhammad Abdullah• 2,650
@0xabdul
Posted
Hey there ! 👋 Congratulations you finished the Order summary component component...🎉
- Some suggestions for you improve your code🤔
Html 🏷️:
- for Accessibility reports using the non-semantic elements (or) semantic elements..
- semantic elements :
<aside> , <artical> , <main>, <header> ,<section><footer>, <form> ect.. - non- semantic elements :
<div> , <span> ect ... - for easy way to clear the Accessibility reports using non semantic elements Ex :
<div class="container" role="main">
//Whole html code wraping the div tag named "container"..📍
</div>
- the html code should be contained one main landmark ❗..
- Any way using the img tag should be put the alt Attribute✅ and mention the alt discription Ex :
<img src="logo.jpg" alt="logo"></img> - I Hope it's useful for you and wating for your next project ❤️
- Happy Coding 😃
0
P
visualdennis• 8,295
@visualdenniss
Posted
@0xAbdul Hi,
Thanks, but please avoid copy-pasting same generic text over and over on every new submission.
Try to make your suggestion related to the posted submission and to the issues that are actually there or relevant.
I suggest you to please read and follow the community guidelines in order to not risk your account red flagged: See section called Copy/paste the same minimal feedback on multiple solutions in https://medium.com/frontend-mentor/frontend-mentor-community-guidelines-44303cc38aa6
Thanks.
0
Muhammad Abdullah• 2,650
@0xabdul
Posted
@visualdenniss ok I keep it mind 🙂
0
