@edburtnieks-private
Posted
Hey @Adarsh.
The design is spot on! The crazy animation on page load made me laugh:D
You did not use any SASS or CSS Modules or any other css compilers, right? I see that you css is 1000+ lines of code, so I would probably think about splitting it into several smaller files. I like that you at left some comments on different sections inside css file :)
About your HTML issue, you are missing an element with id #subscribe-btn
. In the repo the input element has aria-labelledby="news-heading"
though.. But the aria-labelledby
looks for the element with id
of the same name.
@adarshcodes
Posted
@edburtnieks I got the HTML issue and I like the suggestions of code splitting in different files, I will definitely implement this from the next challenge. Thank you for your suggestion