Blogr landing page solution built with React and CSS Modules

Community feedback

• 

  Karol Binkowski•1,620

  @GrzywN

  Posted almost 3 years ago

  Great job 👌

  I have some suggestions regarding your solution:

  • The first one is really simple. Your headings should be bold 😅
  • I would suggest adding some simple transitions for hover states. Make sure to use them inside the hoverable element, not on :hover selector / pseudoclass.
  • If you wanna use a <section> tag, make sure it contains headings. Generally <section> tag isn't really that semantic overall and sometimes it's not worth using it. You can watch a video on it here
  • Make sure to wrap your content into meaningful semantic elements e.g. header, main, nav etc. It's easier for screen readers and crawlers to navigate on your webpage/app. Not using them causes validation issues, which can be easily solved by using these tags instead of divs. (it's kinda weird, because section is different than others, but yeah it's like that)

  Have a nice day and happy coding 😎

  00

  Marked as helpful