eclipse-cdt-cloud / theia-trace-extension

Eclipse Theia trace viewer extension using the Trace Server Protocol (TSP), through the tsp-typescript-client. Also the home for reusable JavaScript libraries: traceviewer-base, traceviewer-react-components
MIT License
49 stars 60 forks source link

XY Chart: Create our own chart library #42

Open tahini opened 4 years ago

tahini commented 4 years ago

XY Component is using ChartJs wrapped in a react-chartjs-2 library. We should create our own instead of using the library. It's not hard to do and will offer way more flexibility to change the axis, draw over the chart and add plugins to chartjs. This should be looked into before subsequent tasks as it may have an impact on how to solve them. Some problems occur when panning. Observe what is happening, that shouldn’t be.

MatthewKhouzam commented 2 years ago

We appear to be moving towards D3. @Rodrigoplp-work does this still make sense?

Rodrigoplp-work commented 2 years ago

Yes, absolutely! I totally agree. This goes right with what is discussed in the "0003-D3-charts" ADR. With the examples that were included in the ADR, plus the work made for the y-axis recently, we are moving really close to being able to make this transition.