tmf: Fix Select Next/Previous Marker navigation
authorPatrick Tasse <patrick.tasse@gmail.com>
Wed, 2 Mar 2016 00:18:54 +0000 (19:18 -0500)
committerPatrick Tasse <patrick.tasse@gmail.com>
Fri, 18 Mar 2016 06:27:08 +0000 (02:27 -0400)
commitd5997aaf589043220fb3ae785f5a3c5575454ba1
treeae7a1008a05146b3ae425d86d45a1c74bd4c3656
parent494c9b221aad8eb5b7920c537e59edff9f5cce64
tmf: Fix Select Next/Previous Marker navigation

Because the markers are sorted by time, when navigating to the next or
previous marker, the marker start time should always be visible.

The action should not rely on the 'ensureVisible' implementation of the
method setSelectionRangeNotify(), as this method will ensure that the
end time of the selection will be made visible when the start time has
not changed. This causes problems if there are two markers with the same
start time but the end time of the second is out of visible range. Since
the window range is modified, the resulting marker list may no longer
include the marker that should have been next in the navigation order.

Change-Id: I434d454d7411f8a00bf58406ad7acc3128472de7
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/67643
Reviewed-by: Hudson CI
tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/TimeGraphViewer.java
This page took 0.026367 seconds and 5 git commands to generate.