Closed shreyaraj13 closed 3 years ago
lgtm, @cseas what do you think?
There's space for improvements:
h1
in global CSS will make for inconsistent styling across headers since that's the only header without a default margin now and we don't want it to feel alone like that. If you want to override the default CSS, just add a local/inline CSS for h1 in your component. The default style will get overridden, that's how CSS works, more specific styles take preference.It's a beginner-friendly project though and we're on a short deadline so none of my above comments really matter too much. They're just suggestions. We can merge anything that seems to do what's intended in the deploy preview and doesn't break/worsen the site.
@cseas Thank you so much for helping me improve my code. I'll work on it. :)
lgtm, @cseas what do you think?