Changeset 409

Timestamp:

10/29/05 14:38:54 (8 years ago)

Author:

gogo

Message:

Fix for #447 - removing border/margin/padding lengths in dialog did not
properly clear styles.

Files:

• trunk/plugins/ImageManager/image-manager.js (modified) (1 diff)

trunk/plugins/ImageManager/image-manager.js

@@ -189 +189 @@
                             case "f_alt"    : img.alt    = value; break;
                             case "f_border" :
-            img.style.borderWidth = /[^0-9]/.test(value) ? value :  (parseInt(value || "0") + 'px');
+          if(value.length)
+          {           
+            img.style.borderWidth = /[^0-9]/.test(value) ? value :  (parseInt(value) + 'px');
             if(img.style.borderWidth && !img.style.borderStyle)
             {
               img.style.borderStyle = 'solid';
             }
-            break;
+          }
+          else
+          {
+            img.style.borderWidth = '';
+            img.style.borderStyle = '';
+          }
+          break;
+          
           case "f_borderColor": img.style.borderColor = value; break;
           case "f_backgroundColor": img.style.backgroundColor = value; break;
-          case "f_padding": img.style.padding =
-                              /[^0-9]/.test(value) ? value :  (parseInt(value || "0") + 'px'); break;
-          case "f_margin": img.style.margin =
-                              /[^0-9]/.test(value) ? value :  (parseInt(value || "0") + 'px'); break;
+            
+          case "f_padding": 
+          {
+            if(value.length)
+            {
+              img.style.padding = /[^0-9]/.test(value) ? value :  (parseInt(value) + 'px'); 
+            }
+            else
+            {
+              img.style.padding = '';
+            }
+          }
+          break;
+          
+          case "f_margin": 
+          {
+            if(value.length)
+            {
+              img.style.margin = /[^0-9]/.test(value) ? value :  (parseInt(value) + 'px'); 
+            }
+            else
+            {
+              img.style.margin = '';
+            }
+          }
+          break;
+          
                             case "f_align"  : img.align  = value; break;
-                                case "f_width"  : img.width = parseInt(value || "0"); break;
-                                case "f_height"  : img.height = parseInt(value || "0"); break;
+            
+          case "f_width" : 
+          {
+            if(!isNaN(parseInt(value))) { img.width  = parseInt(value); } else { img.width = ''; }
+          }
+          break;
+          
+                                  case "f_height":
+          {
+            if(!isNaN(parseInt(value))) { img.height = parseInt(value); } else { img.height = ''; }
+          }
+          break;
                         }