Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337314 - Add view to show the value of an object
Summary: Add view to show the value of an object
Status: RESOLVED FIXED
Alias: None
Product: MAT
Classification: Tools
Component: GUI (show other bugs)
Version: 1.1   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Andrew Johnson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-16 09:21 EST by Andrew Johnson CLA
Modified: 2011-03-09 11:57 EST (History)
0 users

See Also:


Attachments
An example of how the value tab might look (10.40 KB, image/png)
2011-02-16 09:28 EST, Andrew Johnson CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Johnson CLA 2011-02-16 09:21:25 EST
http://www.eclipse.org/forums/index.php?t=msg&th=203917

The name resolver extensions show the value of objects in trees/tables but that isn't ideal for viewing. Another view would be useful.
Comment 1 Andrew Johnson CLA 2011-02-16 09:28:48 EST
Created attachment 189095 [details]
An example of how the value tab might look
Comment 2 Andrew Johnson CLA 2011-02-16 09:29:23 EST
One way would be to add a tab to the Inspector view.

StyledText is the best control - it has a white background, matching the other tabs, unlike Text which is grey. TextViewer is too complicated.

Getting copy to clipboard working is slightly harder - currently setGlobalActionHandler is called twice, for the statics and attributes table. This handler should select the active control - the topTableViewer with focus or the active tab. Multi-line select is also useful for the tables/trees.
Comment 3 Andrew Johnson CLA 2011-02-16 09:53:55 EST
This seems straightforward and useful, so I'll implement it, but leave it active for a while for comments.
Comment 4 Krum Tsvetkov CLA 2011-03-09 09:07:31 EST
I also find this useful, the way you implemented it. It is a nice increment to what we already had - the one liner and the save value. I think we can close this.
If there are some further improvements requested we can still implement them track them separately (and I'm sure one can thing of further improvements - HEX viewer, or even an editor which changes the hprof file). :-)
Comment 5 Andrew Johnson CLA 2011-03-09 11:57:58 EST
The feature works, so we can finish this.