svn commit: r1680201 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1680201 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

jleroux@apache.org
Author: jleroux
Date: Tue May 19 08:30:47 2015
New Revision: 1680201

URL: http://svn.apache.org/r1680201
Log:
Fixes in another way "Make background-color in setSourceColor function in fieldlookup.js theme dependent" https://issues.apache.org/jira/browse/OFBIZ-6371

I preferred the idea of using the jQuery UI effect function as explained in the Jira. It's then no longer color dependent since jQuery has #ffff99 as default color and this might be overridden easily in themes...
https://api.jqueryui.com/highlight-effect/

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

Modified: ofbiz/trunk/framework/images/webapp/images/fieldlookup.js
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/fieldlookup.js?rev=1680201&r1=1680200&r2=1680201&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/fieldlookup.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/fieldlookup.js Tue May 19 08:30:47 2015
@@ -702,9 +702,10 @@ catch (err) {
 
 function setSourceColor(src) {
     if (src && src != null) {
-        src.css("background-color", "yellow");
+        src.effect("highlight", {}, 3000);
     }
 }
+
 // function passing selected value to calling window, using only in the
 // TimeDuration case
 function set_duration_value(value) {