Oct 16 15:18:27 tomsontom TodC: Have you got a minute?
Oct 16 15:20:59 TodC tomsontom: sure
Oct 16 15:39:46 * tashiro (n=stephan@e180161045.adsl.alicedsl.de) has joined #eclipse-dev
Oct 16 15:40:39 * blupin (i=chatzill@eclipse/developer/Eclipse/blupin) has joined #eclipse-dev
Oct 16 15:43:13 tomsontom blupin: What's state of Viewer Widget Independency?
Oct 16 15:43:49 blupin tomsontom: it seems that the widget-independent TreeViewer is the one that will actually be used.
Oct 16 15:44:14 tomsontom Don't you want to provide one for TableViewer too?
Oct 16 15:44:27 blupin I don't want to do a major refactoring without having a client who can validate it.
Oct 16 15:44:55 blupin Adding API "just in case" is not something we like to do...
Oct 16 15:45:31 tomsontom Then I'll work on an independent TreeViewer or did you already did something?
Oct 16 15:45:57 blupin no I didn't. If you could give it a try that would be great.
Oct 16 15:46:28 blupin BTW I'm not saying that I didn't like what you did for TableViewer, in fact I thought it was good work.
Oct 16 15:47:00 tomsontom I'm not 100% sure Grid really only wants TreeViewer
Oct 16 15:47:28 blupin The point is that growing the API comes at a price (you have to maintain more API forever), and it's only worth doing if you actually gain something.
Oct 16 15:47:28 tomsontom Everybody is forced to implement ITreeContenProvider which might not make sense
Oct 16 15:51:26 tomsontom Ok. I'll go the TreeViewer road for now although not 100% convinced.
Oct 16 15:56:56 blupin tomsontom: I talked to Chris Gross, and he could only promise that they will use TreeViewer. If you find another client who promises to use TableViewer, I would be happy to put it in.
Oct 16 15:57:41 blupin But so far, it seems that a refactored Table/TreeViewer would only benefit Nabula's Grid and CustomTableTree.
Oct 16 15:58:32 blupin I am not aware of any other custom table or tree for which our refactoring would have worked. For example, CompositeTable is too different.
Oct 16 15:59:23 tomsontom Ok accepted
Oct 16 16:00:05 tomsontom As already filed then 159597 and 159600 could be applied to head