Community
Participate
Working Groups
I would like to have a constructor in DecoratingLabelProvider that takes ITableLabelProvider as an argument and uses getColumnText and getColumnImage methods. Right now there seems to be no way to do this. Even though this sounds like a enhancement request, it is kind of critical to have .
I think with the new Viewer-API this is easily possible but we needed to revert the DecoratingLabelProvider. See bug #171612, we will add this a provider in 3.4. Since then I think you can simply use the one before we reverted. I'll attach the code but because I've never worked myself with DecoratingLabelProvider it could be that I'm completely mistaken.
Created attachment 63667 [details] Implemenation ontop of the new API
Why is this critical? You are asking for new API, and it is after the API freeze. Is there no way you can implement the desired functionality yourself, until we can provide it in 3.4?
Speaking of the new API, it is difficult to understand without whole picture, but [how] does it address the ITableLabelProvider?
It is critical in a sense that the TableTree and its viewer is deprecated. Those classes were unable to display images in the first column, which was bad. So I decided I 'd move away from them to tree with columns. That class (and viewer) can display images in any column, but those images come up undecorated, because the DecoratingLabelprovider that I use in my application is not supported together with ITableLabelProvider (this one must be used with tree+columns). In short, I cannot provide required look-and-feel using current implementation. Being an owner of a huge UI component, I feel it is critical.
Alex, can you try using the class attached by Tom? (You would have to move it to one of your packages.) It's definitely worth a try. As for the new API, you can now do something like the following to set up column-specific label providers: TreeViewerColumn column = new TreeViewer(treeViewer, SWT.NONE); column.setLabelProvider(new SomeLabelProvider()); column.setEditingSupport(new EditingSupport() { ... }); column.getColumn().setWidth(200); column.getColumn().setText("TableHeader");
Tom, do you want to take this one?
*** Bug 79651 has been marked as a duplicate of this bug. ***
I'd love to try it, but I am still on 3.2, and it's a bit difficult to switch right now.
taken :-)
I used the same method (see Tom's attachment) to bypass this bug, but instead of extending CellLabelProvider, my class extends ColumnlabelProvider. this way i don't need to redefine the update(ViewerCell cell) method. The annoying thing in those workarounds is that I can't move my DecoratingColumnLabelProvider to another package, because of the visibility of ViewerLabel#setHasPendingDecorations()
(In reply to comment #11) > I used the same method (see Tom's attachment) to bypass this bug, but instead > of extending CellLabelProvider, my class extends ColumnlabelProvider. this way > i don't need to redefine the update(ViewerCell cell) method. > The annoying thing in those workarounds is that I can't move my > DecoratingColumnLabelProvider to another package, because of the visibility of > ViewerLabel#setHasPendingDecorations() > What about using reflection? I think putting your class in the org.eclipse.jface-package will fail with signed bundles.
I don't build signed bundles, only a standalone RCP application. It seems to work fine even with product exportation so I will stand this way
There is another method to Override to enable tooltips in Tom's solution: @Override public String getToolTipText(Object element) { if(provider instanceof CellLabelProvider) { return ((CellLabelProvider) provider).getToolTipText(element); } return null; }
missed the M4 cut
Has somebody of the ones attached to this bug a snippet for me to test the implementation else I'll have to write my own.
Created attachment 88319 [details] Updated to fixes from 181215 and support for CellLabelProvider methods
any news on this becoming a possibility?
Missed for 3.5 - but any one is free to test this and report back findings with the above implementation
multi change because of intenion of stepping back as platform-ui committer
Hi, What is the status on this bug? What is the integration/release plan for this? I am working on the Mylyn reviews subproject & I would like this to be included as I want to use this functionality in our plugin to be able to display tooltips on a tree viewer. Thanks, Sebastien Dubois
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.
