Bug 50215 - Too large column separators in a table
Summary: Too large column separators in a table
Status: RESOLVED DUPLICATE of bug 80392
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.0.1   Edit
Hardware: Other Windows CE
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Felipe Heidrich CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-19 10:34 EST by Hubert Lingo CLA
Modified: 2005-06-06 11:25 EDT (History)
2 users (show)

See Also:


Attachments
A preview of the problem. (5.11 KB, image/png)
2004-05-27 05:59 EDT, Hubert Lingo CLA
no flags Details
How the table should be displayed. (8.78 KB, image/png)
2004-05-27 06:03 EDT, Hubert Lingo CLA
no flags Details
A small piece of code written in C showing the too large separators. (23.84 KB, application/octet-stream)
2005-05-09 09:40 EDT, Hubert Lingo CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Hubert Lingo CLA 2004-01-19 10:34:54 EST
The column separators in the header of a table (black lines) are too large. 
Their size is 2 pixels instead of 1.
Comment 1 Hubert Lingo CLA 2004-05-05 06:00:24 EDT
The problem occurs on PocketPC 2003 and Pocket PC 2002.
Comment 2 Hubert Lingo CLA 2004-05-27 05:59:00 EDT
Created attachment 11189 [details]
A preview of the problem.
Comment 3 Hubert Lingo CLA 2004-05-27 06:03:02 EDT
Created attachment 11192 [details]
How the table should be displayed.

See the column spacers.
Comment 4 Steve Northover CLA 2005-04-11 22:59:49 EDT
Chrix, any idea on this one?  We don't do anything special when creating the 
table.
Comment 5 Christophe Cornu CLA 2005-04-12 10:25:04 EDT
I can show Felipe how to run the Pocket PC. It could be be something about the 
arguments we pass to LVM_SETCOLUMN which should not be set or may behave 
differently on Pocket PC.
Comment 6 Steve Northover CLA 2005-04-12 18:25:09 EDT
Ok, show him.
Comment 7 Hubert Lingo CLA 2005-05-09 09:40:50 EDT
Created attachment 20831 [details]
A small piece of code written in C showing the too large separators.

I adapted a small piece of code that displays a table on a Pocket PC. The
column separator seems to be too large also. So it doesn't seem to be a bug.
Comment 8 Steve Northover CLA 2005-05-09 09:52:34 EDT
Thanks Hubert!  I wonder why the separators aren't large for other 
applications.  It must have something to do with the flags that we use in 
CreateWindowEx().
Comment 9 Felipe Heidrich CLA 2005-05-09 14:45:08 EDT
Thanks, Closing.
If by any change you find out a workaround for this problem please let us know.

Comment 10 Hubert Lingo CLA 2005-06-06 06:33:48 EDT
With this flag LVS_NOSORTHEADER, in the CreateWindow() method, it works. 
But now my question is how to change this in SWT. Which style is supposed to
change this behaviour.
Comment 11 Felipe Heidrich CLA 2005-06-06 10:34:25 EDT
I don't think we can use this flag just to fix this bug. 
See Table#widgetStyle()

int widgetStyle () {
	int bits = super.widgetStyle () | OS.LVS_SHAREIMAGELISTS;
	if ((style & SWT.HIDE_SELECTION) == 0) bits |= OS.LVS_SHOWSELALWAYS;
	if ((style & SWT.SINGLE) != 0) bits |= OS.LVS_SINGLESEL;
	/*
	* This code is intentionally commented.  In the future,
	* the FLAT bit may be used to make the header flat and
	* unresponsive to mouse clicks.
	*/
//	if ((style & SWT.FLAT) != 0) bits |= OS.LVS_NOSORTHEADER;
	bits |= OS.LVS_REPORT | OS.LVS_NOCOLUMNHEADER;
	if ((style & SWT.VIRTUAL) != 0) bits |= OS.LVS_OWNERDATA;
	return bits;
}
Comment 12 Hubert Lingo CLA 2005-06-06 10:42:11 EDT
I totally agree with you.
The behaviour of the table, with the LVS_NOSORTHEADER, is totally different.
The code you posted talks by itself ;-)
Comment 13 Steve Northover CLA 2005-06-06 11:23:22 EDT
Ok.
Comment 14 Steve Northover CLA 2005-06-06 11:25:56 EDT
This bug is a dup of 80392.  When we provide the ability to make the header 
unresponsive to mouse clicks, then this flag could be used on Windows CE to 
make the table draw as requested (but also disable clicking on the header).

*** This bug has been marked as a duplicate of 80392 ***