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 12:33:50 UTC
svn commit: r1051462 -
/ofbiz/trunk/framework/images/webapp/images/selectall.js
Author: jleroux
Date: Tue Dec 21 11:33:50 2010
New Revision: 1051462
URL: http://svn.apache.org/viewvc?rev=1051462&view=rev
Log:
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
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=1051462&r1=1051461&r2=1051462&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/selectall.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/selectall.js Tue Dec 21 11:33:50 2010
@@ -446,15 +446,15 @@ function setLookDescription(textFieldId,
dependentField.val(dependentFieldValue);
}
}
- }
- var lookupWrapperEl = jQuery("#" + textFieldId).closest('.field-lookup');
- if (lookupWrapperEl.length) {
- tooltipElement = jQuery("#" + textFieldId + '_lookupDescription')
- if (!tooltipElement.length) {
- tooltipElement = jQuery("<span id='" + textFieldId + "_lookupDescription' class='tooltip'></span>");
- }
- tooltipElement.html(description);
- lookupWrapperEl.append(tooltipElement);
+ var lookupWrapperEl = jQuery("#" + textFieldId).closest('.field-lookup');
+ if (lookupWrapperEl.length) {
+ tooltipElement = jQuery("#" + textFieldId + '_lookupDescription')
+ if (!tooltipElement.length) {
+ tooltipElement = jQuery("<span id='" + textFieldId + "_lookupDescription' class='tooltip'></span>");
+ }
+ tooltipElement.html(description);
+ lookupWrapperEl.append(tooltipElement);
+ }
}
}