@NomiDomi
Posted
Hey @Gunawardhane,
Great job with not going overboard with classes in your html and with the setting of the color schema and re-setting the margin and making the box-sizing: border-box.
If I were you I would try to work a bit more on making my CSS DRY (Don't Repeat Yourself). This is a concept that I recently started learning about and it comes handy in big projects as it will take less time for your designs to load.
I would also spend some more time on getting away from using pixels. This way your implementation becomes more responsive and will have a smoother transition from desktop to mobile.
Not sure if you forget to implement the active part of the design, but in case you didn't and need tips, you can change the look of your buttons with the :hover selector (https://www.w3schools.com/cssref/sel_hover.asp).
Useful videos:
https://www.youtube.com/watch?v=0px6YH-cauQ&ab_channel=KevinPowell https://www.youtube.com/watch?v=N5wpD9Ov_To&ab_channel=KevinPowell
Hope this helps! :) Keep coding!!
Marked as helpful
