Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 280636 - [Table] When in a ScrolledComposite, getColumnOrder() can be wrong after moving a column
Summary: [Table] When in a ScrolledComposite, getColumnOrder() can be wrong after movi...
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.2   Edit
Hardware: All All
: P3 critical (vote)
Target Milestone: 1.3 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr121
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-17 12:32 EDT by Loïc Bertholet CLA
Modified: 2009-07-01 03:05 EDT (History)
0 users

See Also:

Attachments
Snippet View (2.76 KB, text/plain)
2009-06-17 12:32 EDT, Loïc Bertholet CLA 		no flags Details
Proposed patch (925 bytes, patch)
2009-06-19 07:38 EDT, Jordan Yakovchev CLA 		no flags Details | Diff
Proposed patch (941 bytes, patch)
2009-06-23 08:01 EDT, Jordan Yakovchev CLA 		ruediger.herrmann: iplog+
Details | Diff
Show Obsolete (1) 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 Loïc Bertholet CLA 2009-06-17 12:32:47 EDT 
Created attachment 139449 [details]
Snippet View

Environment: Firefox 3.0.8, IE7

Steps to reproduce:
* insert the given View in your application's perspective and launch it
* resize the View and scroll so that you can only see column 15 to 19
* move column 19 between 15 and 16. 
-> Column 19 is displayed between column 0 and 1. table.getColumnOrder() return [0,19,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18].
It seems that the new column position is calculated according to visible one. Here 15 is the first visible column, so 19 is put after 0 (the real first one).

Expected:
Column 19 should be displayed between column 15 and 16 !
Comment 1 Jordan Yakovchev CLA 2009-06-19 07:38:48 EDT 
Created attachment 139627 [details]
Proposed patch

This patch fixes the problem. But unfortunately there is another bug in the column movement(https://bugs.eclipse.org/bugs/show_bug.cgi?id=280905). I hope it will be also fixed soon.
Comment 2 Jordan Yakovchev CLA 2009-06-23 08:01:44 EDT 
Created attachment 139858 [details]
Proposed patch
Comment 3 Rüdiger Herrmann CLA 2009-06-29 14:05:20 EDT 
Fixed in CVS HEAD