@rounakkumarsingh
kad5
@kad5All comments
- @kad5
Great job, Rounak! You were able to complete the challenge successfully. If I might add, try using native html tags like <main> and <section> which would be better for accessibility. Also don't forget about adjusting for smaller screens using media queries when needed.
- @KseniiaMasna@kad5
great work! consider using variables in your CSS for colors and font weights. This way, you could easily change the color at the root and it will automatically affect all the places where this color is used.
- @dave-teg@kad5
Good job, Dave. very organized. It was very informative reading your code. The challenge mentioned to make the font size slightly smaller on smaller screens without using media queries, however your solution is elegant and functional.
- @thurgarion2@kad5
Good challenge. Thanks. didn't understand why a container div inside it a single container flex div. I wish there were comments in the code to explain why things were done in that particular way.
