Bugzilla – Full Text Bug Listing
| Summary: | Create Factory for TableViewerColumn | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Lars Vogel <Lars.Vogel> |
| Component: | UI | Assignee: | Platform-UI-Inbox <Platform-UI-Inbox> |
| Status: | NEW --- | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | Lars.Vogel, marcus.hoepfner |
| Version: | 4.17 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| Whiteboard: | |||
Marcus, WDYT? Interesting. There is a factory for TableColumn already. But not for TableViewerColumn. In comparison TableViewerColumn is not a widget. So this is actually a different game. Maybe it would even be a new plugin? If we start with TableViewerColumn we should also consider all types of viewers (table, tree, combo) for a factory. (In reply to Marcus Höpfner from comment #2) > Interesting. > There is a factory for TableColumn already. But not for TableViewerColumn. > In comparison TableViewerColumn is not a widget. > > So this is actually a different game. > Maybe it would even be a new plugin? I think it is fine to re-use the same plug-in > If we start with TableViewerColumn we should also consider all types of > viewers (table, tree, combo) for a factory. Agree, can you start with the base implementation? I can afterwards start adding factories. |
TableViewerColumn looks like it good use a factory to create it: // create column for description property TableViewerColumn colDescription = new TableViewerColumn(viewer, SWT.NONE); colDescription.setLabelProvider(new ColumnLabelProvider() { @Override public String getText(Object element) { Task todo = (Task) element; return todo.getDescription(); } }); colDescription.getColumn().setWidth(200); colDescription.getColumn().setText("Description");