Bug 84382 - [memory] Header column in Memory View
Summary: [memory] Header column in Memory View
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 2000
: P5 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
Depends on: 79727
Blocks:
  Show dependency tree
 
Reported: 2005-02-03 15:58 EST by Ken Dyck CLA
Modified: 2019-11-14 02:19 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ken Dyck CLA 2005-02-03 15:58:29 EST
I've received the following enhancement request from one of our users...

  In the memory view, the address row (first row) is fixed. Scrolling 
  up/down keeps this row as the first. The same functionality should 
  occur with the address column. That is, scrolling left/right 
  should keep the address column as the first column. Also the 
  address column should have a solid background to distinguish it 
  from the actual data.

  With this improvement the specific address for a memory cell is 
  easier to determine.
Comment 1 Darin Wright CLA 2005-02-07 09:17:29 EST
Assigning to Samantha for consideration.
Comment 2 Samantha Chan CLA 2005-02-10 13:32:46 EST
The Memory View makes use of the Table SWT widget to display a memory block.
The table widget does not currently support for row headers.  I cannot 
implement this enhancement until the table widget implement this feature.

This is the enhancement request in SWT:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=79727

I will monitor this swt bug and will implement this feature once I get the 
proper support.
Comment 3 Ken Dyck CLA 2005-08-09 09:57:56 EDT
As you might imagine, it is easy to get disoriented in the table renderings once
the address column is scrolled out of view, which is usual when addressable
units are larger than 8-bits. Perhaps, while we wait for support from the SWT
folk to implement the locking columns, we could the display the address in hover
text. 

BTW, is there currently (in Eclipse 3.1) any way to extend the built-in table
renderings to provide hover text? That is, can I do it myself as a plug-in
developer?
Comment 4 Samantha Chan CLA 2005-08-09 17:30:36 EDT
Hi Ken -

That's a good idea, having a hover shows the address of a cell.  

In 3.1, there is no easy way for a plugin developer to create hover support in 
a table rendering without referencing to internal classes.  
AbstractTableRendering does not have any API for clients to figure out where a 
mouse point is located from a hover event.  As a result, if you want to provide 
hover support for this, you would have to reference to internal classes and 
calculate where the mouse is located when hover happens.  

I am going to add an enhancement request to support hover for 3.2.  In the 
meantime, if you really want to provide hover support, try the following:

1.  subclass from AbstractTableRendering
2.  override createControl
3.  After the control is created, create the hover control on the table.
4.  To pop up the hover control on the table, your code needs to listen for 
mouseTrack events on the table control.  (#table.addMouseTrackListener(...)
5.  Then your mouse track listener needs to handle the hover event.

The tricky part is to figure out where the mouse is located when the hover 
event happens.  The event will provide you with the Table widget and the point 
location of the hover.  From the point location, you need to go through the 
table and find the correct table element.  From the table element, you can call 
#TableItem.getData() to get to TableRenderingLine (This is an internal class, 
and I do not recommend you referencing to it as it may change in the 
future. :)  From the TableRenderingLine, you can find the row address.  Then, 
you can use that to calculate the address of the cell using your mouse point.

Hope this helps...
Samantha
Comment 5 Samantha Chan CLA 2005-11-16 11:18:00 EST
Reopen for 3.2.
Comment 6 Samantha Chan CLA 2005-11-16 11:21:56 EST
Hi Ken -
I have added hover support for 3.2.  Please let me know if it is sufficient in 
solving your problem.  

https://bugs.eclipse.org/bugs/show_bug.cgi?id=106567

If you still require header column, I will have to defer until the table widget 
provides the support. 

Thanks...
Samantha
Comment 7 Ken Dyck CLA 2005-11-16 11:58:19 EST
I still think a header column would be the ideal solution, but I also understand
how difficult it would be to implement without support in the SWT components. So
I'd say your solution to Bug 106567 is a reasonable and welcome compromise for
the time being.

Perhaps we should add Bug 79727, the SWT enhancement, to the 'depends on' list
and wait for the SWT folks to decide what they are going to do.
Comment 8 Samantha Chan CLA 2005-11-16 12:02:25 EST
Sounds good, will do.  
In the meantime, I will defer this bug.

Thanks...
Samantha
Comment 9 Samantha Chan CLA 2007-07-04 12:36:41 EDT
reopening
Comment 10 Samantha Chan CLA 2007-07-04 12:39:46 EDT
Unable to implement without proper SWT support.
Comment 11 Lars Vogel CLA 2019-11-14 02:19:27 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

If the bug is still relevant, please remove the "stalebug" whiteboard tag.