svn commit: r603893 - in /ofbiz/trunk/framework: common/webcommon/includes/lookup.ftl images/webapp/images/calendar.html

[jleroux]

Author: jleroux
Date: Thu Dec 13 03:40:07 2007
New Revision: 603893

URL: http://svn.apache.org/viewvc?rev=603893&view=rev
Log:
A patch from Daniel Martínez "Make user input change background colours of 
formulars" (https://issues.apache.org/jira/browse/OFBIZ-660) - OFBIZ-660
Note : only when the change comes from calendar or lookup. I will revert if it 
annoys somebody...


Modified:
    ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
    ofbiz/trunk/framework/images/webapp/images/calendar.html

Modified: ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl?rev=603893&r1=603892&r2=603893&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl (original)
+++ ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl Thu Dec 13 
03:40:07 2007
@@ -35,10 +35,15 @@
         if (obj_caller == null) 
             obj_caller = window.opener;
         
-        
+        var bkColor = "yellow";
+        function setSourceColor(src){
+        if(src != null)
+             src.style.backgroundColor = bkColor;
+        }
         // function passing selected value to calling window
         function set_value(value) {
                 if (!obj_caller) return;
+                setSourceColor(obj_caller.target);
                 obj_caller.target.value = value;
                 window.close();
         }
@@ -47,6 +52,7 @@
                 set_value(value);
                 if (!obj_caller.target2) return;
                 if (obj_caller.target2 == null) return;
+                setSourceColor(obj_caller.target2);
                 obj_caller.target2.value = value2;
         }
         function set_multivalues(value) {
@@ -56,6 +62,8 @@
              
                if (arguments.length > 2 ) {
                        for(var i=1; i < arguments.length; i=i+2) {
+                    evalString = "setSourceColor(thisForm." + arguments[i] + 
")";
+                    eval(evalString); 
                                evalString = "thisForm." + arguments[i] + 
".value='" + arguments[i+1] + "'";
                                eval(evalString);
                        }

Modified: ofbiz/trunk/framework/images/webapp/images/calendar.html
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/calendar.html?rev=603893&r1=603892&r2=603893&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/calendar.html (original)
+++ ofbiz/trunk/framework/images/webapp/images/calendar.html Thu Dec 13 
03:40:07 2007
@@ -40,7 +40,12 @@
 var num_id = (re_id.exec(String(window.location))
        ? new Number(RegExp.$1) : 0);
 var obj_caller = (window.opener ? window.opener.calendars[num_id] : null);
-
+// Change background color
+var bkColor = "yellow";
+function setSourceColor(src){
+if(src != null)
+ src.style.backgroundColor = bkColor;
+}
 if (obj_caller && obj_caller.year_scroll) {
        // get same date in the previous year
        var dt_prev_year = new Date(dt_current);
@@ -87,6 +92,7 @@
                        ? obj_caller.gen_tsmp(dt_datetime)
                        : obj_caller.gen_date(dt_datetime)
                );
+        setSourceColor(obj_caller.target);
                window.close();
        }
        else obj_caller.popup(dt_datetime.valueOf());