Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 372381 - Setting up an initial order for IColumns
Summary: Setting up an initial order for IColumns
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Scout (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-23 13:29 EST by Adrian Staudt CLA
Modified: 2021-08-19 11:02 EDT (History)
3 users (show)

See Also:
zimmermann: indigo+
zimmermann: juno+


Attachments
Path addes an initial order for IColumns. (3.43 KB, patch)
2012-02-23 13:35 EST, Adrian Staudt CLA
zimmermann: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Staudt CLA 2012-02-23 13:29:41 EST
Build Identifier: 

There is an issue with the order for injected columns.

Lets asume that a table page defines some fixed columns (with java) and some injected columns (which will be added at runtime). Scout shows first of all all java-defined columns and than all injected columns. 
An improvment of this concept will be if the developer would have the possibility to define an order.

The difference is that java-defined columns have an initial order. It is defined by the @Order annotation of the column. But in general is a inital order missing.

Therefore I suggest to added an initial order index for columns. So that it will be possible to mix the occurrence of java-defined columns and injected columns.

Reproducible: Always
Comment 1 Adrian Staudt CLA 2012-02-23 13:35:48 EST
Created attachment 211526 [details]
Path addes an initial order for IColumns.

IColumn defines methods for getting and setting an inital order index.
The AbstractTable considers that initial order index.
Comment 2 Andi Bur CLA 2012-03-12 10:07:32 EDT
Thanks for the patch, Adrian.

Applied a modified version:
o the property is called viewOrder instead of initialOrderIndex
o the view order can be changed at any time and the ColumnSet is re-validated on every update
o resetting the column order still checks a column's visibility and whether it is displayable at all
Comment 3 Matthias Zimmermann CLA 2012-05-23 10:56:15 EDT
Comment on attachment 211526 [details]
Path addes an initial order for IColumns.

added missing iplog flag
Comment 4 Matthias Zimmermann CLA 2013-07-08 17:36:47 EDT
Bugfixes shipped on Scout 3.7 branch