[#GROOVY-1878] SwingBuilder / tableLayout -> more than two cells in one row are not displayed

groovy

SwingBuilder / tableLayout -> more than two cells in one row are not displayed

Created: 07/May/07 01:44 AM Updated: 11/Jun/07 03:34 AM

Component/s:

None

Affects Version/s:

1.0

Fix Version/s:

1.1-beta-2

Time Tracking:

Not Specified

File Attachments:	1. patch.txt (0.4 kB)
Environment:	Windows XP Professional sp2 JVM: 1.4.2_10-b03

Description

« Hide

In the attached example button 'B' is not visible.

Regards

Frank

-------

import groovy.swing.SwingBuilder

swing = new SwingBuilder()

frame = swing.frame() {
panel {
tableLayout {
tr {
td {button 'A'}
td {button 'B'}
td {button 'C'}
}
}
}
}
frame.pack()
frame.show()

All

Comments

Work Log

Change History

Sort Order:

[ Permalink | « Hide ]

Frank Harnack added a comment - 08/May/07 02:11 AM

The attached patch seems to solve the problem.

Thanks to my colleague Alexej Stojanov. We fixed the issue together.

Regards

Frank

[ Permalink | « Hide ]

Paul King added a comment - 08/May/07 03:12 PM

Jochen, let me know if you run out of time to apply this. I am happy to add it and do some testing. Just assign to me if you want, Paul.

[ Permalink | « Hide ]

Jochen Theodorou added a comment - 08/May/07 03:54 PM

ok, you can apply the patch

[ Permalink | « Hide ]

Paul King added a comment - 09/May/07 06:39 AM

I applied the patch and it definitely fixed the first issue. It doesn't appear to work properly with colspan however, so I'll see if I can fix that too before committing.

[ Permalink | « Hide ]

Paul King added a comment - 14/May/07 07:23 AM

Patch applied plus I added some tweaks to the colspan calculating code (there wasn't any before) so it works (at least for the simple cases) too now.

[ Permalink | « Hide ]

Paul King added a comment - 10/Jun/07 04:01 AM

No further feedback, assuming fixed.

[ Permalink | « Hide ]

Frank Harnack added a comment - 11/Jun/07 03:34 AM

Hi Paul,

thanks for your bugfix.

Unfortunately we decided not to use SwingBuilder for our little gui and we are to busy to test your additional modifications.
But we want to use SwingBuilder in the future (1.1-beta-2) to make our ant build scripts more user friendly.

Regards

Frank + Alexej