[
https://issues.apache.org/jira/browse/OFBIZ-10012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16351745#comment-16351745 ]
Aditya Sharma commented on OFBIZ-10012:
---------------------------------------
Here is the solution patch.
Passed all the parameters as data-* attribute values.
Improved handling of width, height and position.
Removed checks for showDescription as it is already handled at org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java:2199
Added css as Browser styling which causes cursor to become pointer was not applied for the dynamic element .
Added call to bindObservers fieldlookup.js so that lookup works on dynamic elements of LookForm too.
Added examples for call_fieldlookup3 i.e. Popup lookup field with description and Layered lookup field with autocompletion, description at right and optional target.
Although call_fieldlookup2 and call_fieldlookup3 didn’t worked initially & requires a separate work. The lookup opens but when clicked on value it is not set. This is a old way & not used now. [
https://demo-trunk.ofbiz.apache.org/example/control/FormWidgetExamples]
ofbizUtils.js was being included twice for content component as included in main-decorator also so removed that too.
Added check for form name in the beginning but as form name is extracted directly from element’s form I guess it will pass in most of the cases & the problem will not arise.
Verify:
Lookup Fields: [
https://localhost:8443/example/control/FormWidgetExamples]
Owner Content Id: [
https://localhost:8443/content/control/findContent] Here, it can be also verified that a lookup from a lookup works. Click link for ownerContentId and then click any link for lookup from the lookup.
Thanks Deepak for the guidance provided.
Please review.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)