Pradeep Saini• 990
@pradeeps4ini
Posted
Hey. Good work. You should use whatever you're comfortable with flex or grid. But learn both. Grid can be used to design the macro layout (how your page looks), where as both flex and grid can be used to design micro layout (each distinct element).
You nav toggle shows and hide suddenly. You should give some transition to it. Learn about transform and transition. It will look much better.
You have used JS to toggle the nav in mobile layout. Learn to do it with css only. It will be a great learning exercise.
Marked as helpful
0
Jack Tubbenhauer• 170
@jtubbenhauer
Posted
@pradeeps4ini Thank you very much for your feedback. Much appreciated!
0
Pradeep Saini• 990
@pradeeps4ini
Posted
@jtubbenhauer You're welcome dude. We are all in this together. Learning to be better devs.
0
