Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.8.1
-
Component/s: ICE-Components
-
Labels:None
-
Environment:All
Description
If you take the component-showcase drag and drop example, which puts a draggable panelGroup within a panelSeries, and then turn off dragGhost, so that dragging the panelGroup results in it actually moving, you'll see that all of the virtual panelGroups move, since the style is being updated, which is not saved per row in the panelSeries.
Activity
Mark Collette
created issue -
Mark Collette
made changes -
Field | Original Value | New Value |
---|---|---|
Link | This issue blocks ICE-4804 [ ICE-4804 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #19155 | Thu Aug 13 09:30:16 MDT 2009 | mark.collette | |
Files Changed | ||||
MODIFY
/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlPanelGroup.java
|
Mark Collette
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Fix Version/s | 1.8.2 [ 10190 ] | |
Resolution | Fixed [ 1 ] |
Joanne Bai
made changes -
Attachment | dragDrop-showcase.JPG [ 11929 ] |
Ken Fyten
made changes -
Fix Version/s | 1.8.2-RC1 [ 10210 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Investigation of the bug started with inspecting the browser DOM with the YUI code versus the non-YUI code. At first I thought it migt be a CSS flow issue where when one DIV is moved, the others might naturally flow from there too. But inspection showed that all of the DIVs were getting the same x,y coordinated. Them flowing is what made them show up in a row, instead of all on top of each otyher. The cssUpdate mechanism looked different between the two. Then I noticed that dragGhost was set in the non-YUI code, which might account for it acting differently. Sure enough, when using the same mode, they both exhibited the same bug. I investigated the decode method on the panelGroup to see if the cssUpdate was somehow being mis-applied to all of the rows, which is when I found that we don't do row state saving of the style properties, and that's why it was being applied to all rows. So I added all the style related properties to the row state saving, and that fixed it.
Subversion 19155
icefaces\component\src\com\icesoft\faces\component\ext\HtmlPanelGroup.java