Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.8.1
-
Component/s: ICE-Components
-
Labels:None
-
Environment:All
Description
Within the outputted div of every draggable or droppable ice:panelGroup are rendered two hidden input fields, for communicating back to the server:
<input type="hidden" value="" name="iceform:_idiceform:PnlSrs:0:PnlGrpstatus" id="iceform:_idiceform:PnlSrs:0:PnlGrpstatus"/>
<input type="hidden" value="" name="iceform:_idiceform:PnlSrs:0:PnlGrpdropID" id="iceform:_idiceform:PnlSrs:0:PnlGrpdropID"/>
That would really decrease performance, if there was a large table of draggable or droppable objects, because of the many hidden input fields. Instead, I think we should just have 3 or four fields in the form:
dragId, dropId, status (maybe dragStatus and dropStatus, if the two statuses can have different values for the same operation)
<input type="hidden" value="" name="iceform:_idiceform:PnlSrs:0:PnlGrpstatus" id="iceform:_idiceform:PnlSrs:0:PnlGrpstatus"/>
<input type="hidden" value="" name="iceform:_idiceform:PnlSrs:0:PnlGrpdropID" id="iceform:_idiceform:PnlSrs:0:PnlGrpdropID"/>
That would really decrease performance, if there was a large table of draggable or droppable objects, because of the many hidden input fields. Instead, I think we should just have 3 or four fields in the form:
dragId, dropId, status (maybe dragStatus and dropStatus, if the two statuses can have different values for the same operation)
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
Field | Original Value | New Value |
---|---|---|
Salesforce Case | [] | |
Fix Version/s | 1.8.2 [ 10190 ] | |
Assignee Priority | P3 | |
Assignee | Mark Collette [ mark.collette ] |
Salesforce Case | [] | |
Assignee | Mark Collette [ mark.collette ] | Adnan Durrani [ adnan.durrani ] |
Status | Open [ 1 ] | In Progress [ 3 ] |
Link | This issue depends on ICE-4719 [ ICE-4719 ] |
Salesforce Case | [] | |
Assignee Priority | P3 | P2 |
Status | In Progress [ 3 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Link | This issue depends on ICE-4719 [ ICE-4719 ] |
Fix Version/s | 1.8.2-RC1 [ 10210 ] |
Status | Resolved [ 5 ] | Closed [ 6 ] |
Assignee Priority | P2 |