Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 2.0-Beta2
-
Fix Version/s: 2.0.0
-
Component/s: None
-
Labels:None
-
Environment:ICEfaces
Description
ice.se (singleSubmitExecuteThis) causes focus to be lost when used with compat and h: components.
Using the singleSubmit variant of component-showcase:
icefaces2/branches/ice-6178/
Choose the selection demo. Observe that focus is lost when any of the "select" items are selected.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
namespace.enableSingleSubmit also contains
ice.setFocus(null);
in an attempt to prevent the focus from being changed by the server when the page updates are received. (The user may already have changed focus by this time.) Removing this line results in focus retention on unintended elements.
Using innerHTML is not an option because the entire element (with its attribute) not only its content needs to be copied into the hidden form.
Changed the single submit code to clone the submitting node into the hidden form instead of moving it in temporarely. Also, while cloning, the element specific state (for checkboxes, radio buttons and selects) is copied programatically to fix IE's cloning issues (see reference in ICE-5731 ).
Used 'Date Time Entry' in ace-showcase for testing since all the components included in the page use ice.se for submitting the data.
Multiple selections and checkboxes are not being submitted with all values.
Modifying application.js to use the following allows focus to be retained in component-showcase:
jsf.ajax.request(element, event,
{execute: element.id, render: "@all"});
// ice.se(e, element);
The root cause of loss of focus appears to be the following in submit.js:
//move element from its original place into the single submit form
form.appendChild(element);
Moving the element that is currently focused by the user is likely the source of the problem. Using innerHtml may be an option for cloning the element if DOM APIs are not suitable.