theia-trace-extension icon indicating copy to clipboard operation
theia-trace-extension copied to clipboard

Resources Status filler row height too big

Open bhufmann opened this issue 3 years ago • 2 comments
trafficstars

The Resources Status view of the Trace Compass server has a filler row between the CPUs to visually separate the CPUs. However, the filler row is drawn with the same height as a regular row which wastes vertical space and doesn't have a great UX.

To support this independently from the trace server and view, such filler rows need to be styled through the data provider TSP API.

Theia Trace Extension with Trace Compass server: image

In Eclipse Trace Compass it looks like this: image

bhufmann avatar Aug 03 '22 12:08 bhufmann

There is no explicit identifier for the "filler rows", but they all come in with an empty string "" as their name. Filler rows can be identified height set through the Row Style Provider easily. But we can move forward by identifying filler rows either by:

  1. Identify "filler rows" by their empty string name "".
  • Simple fix, can be contained to this repository.
  1. Add an explicit tag for "filter rows" on the back end.
  • More complicated, but cleaner and less 'hacky'.

williamsyang-work avatar Jan 17 '23 18:01 williamsyang-work

what about using height style in the model?

https://github.com/eclipse-cdt-cloud/trace-server-protocol/blob/569aac4cc29f899012b226b37c7b349bba6f78f3/API.yaml#L1721

bhufmann avatar Jan 17 '23 21:01 bhufmann