Skip to content
Success

Changes

Summary

  1. tmf: Fix font height in TimeGraphRender.setFontForHeight() (details)
  2. tmf: Remove workaround for font height in TimeGraphControl.drawMarker() (details)
  3. tmf: Adjust font height for time graph name and labels (details)
Commit 32cb6dc8e06b851e78c9fae3359f4c209313da82 by Patrick Tasse
tmf: Fix font height in TimeGraphRender.setFontForHeight()

Implement a workaround so that the font height matches the requested
pixel height, as the font height in points determined by calculation
results in a larger font than expected.

https://github.com/eclipse-platform/eclipse.platform.swt/issues/1193

Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
The file was modified tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/widgets/timegraph/TimeGraphRender.java (diff)
Commit 242120e2cf3907abecbcde9d88eef2f9275cc1e5 by Patrick Tasse
tmf: Remove workaround for font height in TimeGraphControl.drawMarker()

With the font now matching the requested height, the workaround for the
marker symbol text y-position can be removed.

Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
The file was modified tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/widgets/TimeGraphControl.java (diff)
Commit 2cf4865b5f32d9cd6106bbd8aaa18c6dd4d1e568 by Patrick Tasse
tmf: Adjust font height for time graph name and labels

With the font now matching the requested height, request a slightly
larger (2px) font height for time graph name and labels, to improve
readability. In both cases the text is centered using the actual
rendered text height.

Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
The file was modified tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/widgets/timegraph/TimeGraphRender.java (diff)
The file was modified tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/widgets/TimeGraphControl.java (diff)