You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2010/12/21 19:33:44 UTC

svn commit: r1051602 - /ofbiz/trunk/framework/images/webapp/images/selectall.js

Author: jleroux
Date: Tue Dec 21 18:33:44 2010
New Revision: 1051602

URL: http://svn.apache.org/viewvc?rev=1051602&view=rev
Log:
Complete r1051462 (We don't need this whole block if there are no descriptions. It looks better in UI: no scrawls around the lookup button, notably when the description is empty) in case of the old style lookup (where a false description <script type="text/javascript">\r\n</script> is passed in because of an empty result in fieldlookup.js for <<if (result.split("ajaxAutocompleteOptions.ftl -->")[1]) {>>

Modified:
    ofbiz/trunk/framework/images/webapp/images/selectall.js

Modified: ofbiz/trunk/framework/images/webapp/images/selectall.js
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/selectall.js?rev=1051602&r1=1051601&r2=1051602&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/selectall.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/selectall.js Tue Dec 21 18:33:44 2010
@@ -447,7 +447,7 @@ function setLookDescription(textFieldId,
             }
         }
       var lookupWrapperEl = jQuery("#" + textFieldId).closest('.field-lookup');
-      if (lookupWrapperEl.length) {
+      if (lookupWrapperEl.length && start != -1) {
           tooltipElement = jQuery("#" + textFieldId + '_lookupDescription')
           if (!tooltipElement.length) {
               tooltipElement = jQuery("<span id='" + textFieldId + "_lookupDescription' class='tooltip'></span>");