@MojtabaMosavi
Posted
There are some minor things:
1- I see that you're using bem, it really beneficial to be consistant with it, it would be a great practice for the next challange that you do, try to implement everthing with bem becuase it's reap the many great reward of it.
2- The button in the header and text beside it need some need some margin-right (going for mobile view to desktop).
3- Always when you write markup, firstly see if there is a hmlt tag specific for that task, for instance the div with the class main could be substituted with a main tag.
Keep coding :=)
Marked as helpful
@imonaar
Posted
@MojtabaMosavi Thank you. I noticed my deficits with bem, i just started using it recently. I wanted to get a hang of it and then ingrain the fundamentals. Improving on that in the next project.
