Bug 72930 - [Viewers] TableViewer.reveal() documentation
Summary: [Viewers] TableViewer.reveal() documentation
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: Macintosh All
: P2 minor (vote)
Target Milestone: 3.1 M2   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks:
 
Reported: 2004-08-31 02:32 EDT by Michael Salmon CLA
Modified: 2004-09-22 08:52 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Salmon CLA 2004-08-31 02:32:03 EDT 
The documentation for TableViewer.reveal() does not say that it requires a tableItem and silently 
ignores all other objects.
Comment 1 Tod Creasey CLA 2004-08-31 14:34:20 EDT 
It doesn't require a TableItem. Here is the code

Assert.isNotNull(element);
Widget w = findItem(element);
if (w instanceof TableItem)
	getTable().showItem((TableItem) w);

So it looks for the table item and shows it. If you are referring to findItem 
needing to be a TableItem to reveal it in a table I think this is fine.

BTW thanks for the pointer to the javadoc - it was not referring to the 
structured viewer javadoc correctly and I will fix this.
Comment 2 Tod Creasey CLA 2004-08-31 14:37:54 EDT 
Javadoc fixes released for build >20040831
Comment 3 Tod Creasey CLA 2004-09-22 08:52:52 EDT 
Verified in 20040921