Fix/event browser html

[danhemerlein]

This updates some classNames to use a string (no need for {} if no dynamic logic)
If we're not using dynamic logic or variables, just pass the string directly

This also moves the EventsBrowser to a directory in DataView. I did this so I could move some types into their own file. I would like do another push of cleaning up this file (breaking it into smaller components)

Thank you!

[vklimontovich]

Although this PR totally make sense, we generally hesitant of making cosmetic changes and moving files around without strong reason to do so in scope of adding new features.

Speaking of classNames, that would make sense if we could add a prettier/linter rule to enforce that.

Looks like react/jsx-curly-brace-presence does exactly this, I'll try to add via separate commit

[danhemerlein]

Although this PR totally make sense, we generally hesitant of making cosmetic changes and moving files around without strong reason to do so in scope of adding new features.

Speaking of classNames, that would make sense if we could add a prettier/linter rule to enforce that.

Looks like react/jsx-curly-brace-presence does exactly this, I'll try to add via separate commit

All good thank you very much for the review @vklimontovich please feel free to close this out if it doesn't make sense. Happy to help out with new feature development - especially if it's console app related. thanks!

[vklimontovich]

Here's the change: ccbc8bf

We will swap it from warn to error once all PRs are merged to avoid merge conflicts