Bug 369005

Description Patrick Tasse 2012-01-18 15:45:16 EST

In TmfEventsTable the context menu when the user right-clicks in the left margin shows different actions whether the selected item has a bookmark set or not.

The selected item is determined using the current cursor location coordinates. These coordinates are relative to the top left corner of the table, including the header row.

However, in Linux the Table method getItem(Point p) expects a point with coordinates relative to the top left corner of the first table item (excluding the header row) whereas in Windows the coordinates must be relative to the top left corner of the header row.

The current implementation doesn't take this OS-dependent behavior into account, so in Linux the context menu is populated using the wrong table item.

The TableItem method getImageBounds(int index) shows the same OS-dependent behavior.

The solution is to determine the table item for the context menu based on the table selection instead of the cursor coordinates, and to ignore the y coordinates of the image bounds. This makes the implementation of the context menu OS-independent.