@Bishalsnghd07
Posted
Hi, @NexusLo 👋
Congrats for completing this challenge🎉 and you did a great job👏
I would have some suggestion for you:
1)Give some padding in the parent container, so that your layout will look more responsive in mobile view also.
2)Headings level should not be break, it should be like this 👉 h1, h2, h3, h4, h5 & h6. h1 will be the most important tag and h6 will be the least important tag. After h1, use h2 and so on, in ascending order, did not break the flow and it will good practice to use as a developer. Heading level is not doing large or small your size of element. It is used to maintain semantic of your web page.
By doing this changes your code quality and semantic ui will be much improved.
Hope, this suggestions will help you ahead in your projects too!
Happy Coding ❤️
Marked as helpful
@NexusLo
Posted
Hi @Bishalsnghd07 👋
I'll definitely take your suggestions into consideration and make sure to implement these changes in my future projects. Thanks for taking the time to provide such helpful suggestions!
Have a nice day!
