Refactor UI code
Opened this issue · 2 comments
For the last couple of months, we managed to develop all features planned for the MVP. I think it's a good moment to stop for a while and think about possible improvements in terms of code structure and best practices. This tracking issue will help us organize our ideas for refactoring and optimization.
@k-jano @filwie Please, give your ideas. Do you see any code parts that should be redesigned or refactored? Take into account best practices, code smells, DRY/KISS principles. Also, feel free to explore other open-source projects (Jaeger UI, Skydive, etc.) to inspire changes in our repository.
Below is a list of refactoring items:
- Reorganize src directory (#55).
After some investigation I updated #55 issue. I also think that the main problem is that Graph.js file became a god object with too many functionalities. It would be great to move majority of functions to a helper file (e.g. graphUtils.js) and leave in a main file only the most necessary ones. Other files are OK for me, generally they realize only one functionality and are written with React style.