Details
- 
        Type: Bug Bug
- 
        Status: Closed
- 
            Priority: Major Major
- 
            Resolution: Fixed
- 
            Affects Version/s: 2.0-Beta2
- 
            Fix Version/s: 2.0-Beta2
- 
            Component/s: None
- 
            Labels:None
- 
            Environment:Server: Tomcat6
 Browser: FF3.6, IE8 & Opera10.10
- 
                        Assignee Priority:P2
Description
                    Icefaces2 Revision# 22054
This test case may have been failing for long because the the scripts were incorrect. Basically the problem is the same as described originally inICE-2283.
If I have something like this on my page:
<ice:dataTable columnWidths="100px,200px,300px">
<ice:column rendered="true"/>
<ice:column rendered="false"/>
<ice:column rendered="true"/>
</ice:dataTable>
Then I should get this rendered in my DOM:
<table>
<tr>
<td width="100px"/>
<td width="300px"/>
</tr>
</table>
But instead, I get this:
<table>
<tr>
<td width="100px"/>
<td width="200px"/>
</tr>
</table>
Test-case available at "C:\repo\qa\trunk\Regression-Icefaces2\Nightly\ICE-2283".
            
This test case may have been failing for long because the the scripts were incorrect. Basically the problem is the same as described originally in
If I have something like this on my page:
<ice:dataTable columnWidths="100px,200px,300px">
<ice:column rendered="true"/>
<ice:column rendered="false"/>
<ice:column rendered="true"/>
</ice:dataTable>
Then I should get this rendered in my DOM:
<table>
<tr>
<td width="100px"/>
<td width="300px"/>
</tr>
</table>
But instead, I get this:
<table>
<tr>
<td width="100px"/>
<td width="200px"/>
</tr>
</table>
Test-case available at "C:\repo\qa\trunk\Regression-Icefaces2\Nightly\
Activity
| Field | Original Value | New Value | 
|---|---|---|
| Assignee | Deryk Sinotte [ deryk.sinotte ] | 
| Salesforce Case | [] | |
| Assignee Priority | P2 | |
| Assignee | Deryk Sinotte [ deryk.sinotte ] | Mark Collette [ mark.collette ] | 
| Assignee | Mark Collette [ mark.collette ] | Yip Ng [ yip.ng ] | 
| Salesforce Case | [] | |
| Fix Version/s | 2.0-Beta2 [ 10242 ] | 
| Repository | Revision | Date | User | Message | 
| ICEsoft Public SVN Repository | #22096 | Wed Aug 04 14:50:35 MDT 2010 | yip.ng |  | 
| Files Changed | ||||
|  MODIFY
            				/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/renderkit/TableRenderer.java | 
| Repository | Revision | Date | User | Message | 
| ICEsoft Public SVN Repository | #22097 | Wed Aug 04 14:55:09 MDT 2010 | yip.ng |  | 
| Files Changed | ||||
|  MODIFY
            				/icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/renderkit/TableRenderer.java | 
| Attachment | screenshot-01.png [ 12479 ] | 
| Attachment | screenshot-02.png [ 12481 ] | 
| Status | Open [ 1 ] | Resolved [ 5 ] | 
| Resolution | Fixed [ 1 ] | 
| Status | Resolved [ 5 ] | Closed [ 6 ] | 
| Resolution | Fixed [ 1 ] | |
| Status | Closed [ 6 ] | Reopened [ 4 ] | 
| Assignee | Yip Ng [ yip.ng ] | Mircea Toma [ mircea.toma ] | 
| Status | Reopened [ 4 ] | Resolved [ 5 ] | 
| Resolution | Cannot Reproduce [ 5 ] | 
| Resolution | Cannot Reproduce [ 5 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] | 
| Salesforce Case | [] | |
| Assignee Priority | P2 | P1 | 
| Status | Reopened [ 4 ] | Resolved [ 5 ] | 
| Resolution | Fixed [ 1 ] | 
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] | 
| Assignee Priority | P1 | P2 | 
| Repository | Revision | Date | User | Message | 
| ICEsoft Public SVN Repository | #22396 | Wed Sep 22 06:55:39 MDT 2010 | mircea.toma |  | 
| Files Changed | ||||
|  MODIFY
            				/icefaces2/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/CheckboxRenderer.java | 
| Status | Reopened [ 4 ] | Resolved [ 5 ] | 
| Resolution | Fixed [ 1 ] | 
| Status | Resolved [ 5 ] | Closed [ 6 ] | 
 
                         
                        
Assigning to Mark as it looks like something specifically component related. As Mandeep notes, this may not strictly be a regression as the test scripts have been out of sorts. Provided it's a straightforward solution, we should just fix it. If required, we can verify against 1.8 and older 2.0 releases to see how recent it might be.