Bug 177213 - [Viewers] Add getColumnDatas() to TableLayout
Summary: [Viewers] Add getColumnDatas() to TableLayout
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks:
 
Reported: 2007-03-13 17:09 EDT by Brad Reynolds CLA
Modified: 2019-09-06 16:07 EDT (History)
1 user (show)

See Also:

Attachments
TableLayout patch (4.26 KB, patch)
2007-03-13 18:35 EDT, Brad Reynolds CLA 		no flags Details | Diff
View All 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 Brad Reynolds CLA 2007-03-13 17:09:38 EDT 
In the current implementation of TableLayout and AbstractColumnLayout the consumer can add ColumnLayoutData instances but they cannot then be retrieved from the layout.  We have an uninteresting use case that I won't go into here that we need access to this data.  I didn't see any reason why this was not exposed other than there wasn't a need.

I'll attach a patch for TableLayout.
Comment 1 Thomas Schindl CLA 2007-03-13 17:15:54 EDT 
You should take a look at the successors of TableLayout in bug #171824. Although it doesn't provide the information either it's much better because it supports resizing.
Comment 2 Brad Reynolds CLA 2007-03-13 17:20:54 EDT 
(In reply to comment #1)
> You should take a look at the successors of TableLayout in bug #171824.
> Although it doesn't provide the information either it's much better because it
> supports resizing.
> 

I have and it's why I referenced AbstractColumnLayout as well in the description.  We want it to only resize once so Abstract/TableColumnLayout doesn't fix out issue.  

But if possible it would be nice to have AbstractColumnLayout return this information as well as the patch I looked at this morning didn't provide this information either.
Comment 3 Thomas Schindl CLA 2007-03-13 17:29:35 EDT 
My vision for this was to attach this information to the Table/TreeColumn instead of adding it to the Layout, this way it would be accessible all over see bug #171844 but I'm out of time for M6. I have just a few minutes ago updated the patch in #171824 make sure you patch is against it.
Comment 4 Brad Reynolds CLA 2007-03-13 17:48:29 EDT 
(In reply to comment #3)
> I have just a few minutes ago
> updated the patch in #171824 make sure you patch is against it.
> 

Will do.  Thanks.
Comment 5 Brad Reynolds CLA 2007-03-13 18:35:55 EDT 
Created attachment 60742 [details]
TableLayout patch
Comment 6 Brad Reynolds CLA 2007-03-13 21:50:45 EDT 
Updating the summary to just be for TableLayout.
Comment 7 Boris Bokowski CLA 2009-11-26 09:48:55 EST 
Hitesh is now responsible for watching bugs in the [Viewers] component area.
Comment 8 Eclipse Webmaster CLA 2019-09-06 16:07:19 EDT 
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.