Fixed issues when creating leaded components using drag and drop with

'Snap to Grid' option switched on using a metric grid spacing (e.g. 1mm)

diylc/diylc-core/src/org/diylc/components/AbstractComponent.java

@@ -32,6 +32,7 @@ import java.lang.reflect.Modifier;
 import java.util.Arrays;
 
 import org.diylc.core.IDIYComponent;
+import org.diylc.core.Project;
 import org.diylc.core.annotations.EditableProperty;
 
 /**
@@ -258,5 +259,10 @@ public abstract class AbstractComponent<T> implements IDIYComponent<T> {
   @Override
   public Rectangle2D getCachingBounds() {
     return null;
+  }
+  
+  @Override
+  public void createdIn(Project project) {
+    // TODO Auto-generated method stub
   }
 }

diylc/diylc-core/src/org/diylc/core/IDIYComponent.java

@@ -196,5 +196,12 @@ public interface IDIYComponent<T> extends Serializable, Cloneable {
    * Used only if {@link ComponentDescriptor#enableCache()} is set to true. 
    * @return
    */
-  Rectangle2D getCachingBounds();
+  Rectangle2D getCachingBounds();
+  
+  /**
+   * Notifies the component that it's being created in the context of the specified project  
+   * 
+   * @param project
+   */
+  void createdIn(Project project);
 }

diylc/diylc-core/src/org/diylc/core/measures/AbstractMeasure.java

@@ -32,7 +32,7 @@ import java.text.Format;
  *
  * @param <T>
  */
-public class AbstractMeasure<T extends Enum<? extends Unit>> implements Serializable, Cloneable {
+public abstract class AbstractMeasure<T extends Enum<? extends Unit>> implements Serializable, Cloneable {
 
   private static final long serialVersionUID = 1L;
 

diylc/diylc-core/src/org/diylc/presenter/CalcUtils.java

@@ -48,6 +48,13 @@ public class CalcUtils {
     point.setLocation(x, y);
   }
   
+  public static Size findClosestMultiplierOf(Size factor, Size target, int step) {
+    Size ret = new Size(0d, factor.getUnit());
+    while (ret.convertToPixels() < target.convertToPixels())
+      ret = new Size(ret.getValue() + factor.getValue() * step, factor.getUnit());
+    return ret;
+  }
+  
   public static void snapPointToObjects(Point2D point, Size gridSpacing, IDIYComponent<?> component, List<IDIYComponent<?>> components) {
     for (IDIYComponent<?> c : components) {
       if (c == component)

diylc/diylc-core/src/org/diylc/presenter/InstantiationManager.java

@@ -260,7 +260,8 @@ public class InstantiationManager {
     LOG.info("Instatiating component of type: " + componentType.getInstanceClass().getName());
 
     // Instantiate the component.
-    IDIYComponent<?> component = componentType.getInstanceClass().newInstance();
+    IDIYComponent<?> component = componentType.getInstanceClass().newInstance();
+    component.createdIn(currentProject);
 
     component.setName(createUniqueName(componentType, currentProject.getComponents()));
 

diylc/diylc-core/src/org/diylc/presenter/update.xml

@@ -3290,6 +3290,10 @@
 					<changeType>IMPROVEMENT</changeType>
 					<description>macOS: two finger click shows the popup menu without deselecting a selected component</description>
 				</org.diylc.appframework.update.Change>
+				<org.diylc.appframework.update.Change>
+					<changeType>IMPROVEMENT</changeType>
+					<description>Fixed issues when creating leaded components using drag and drop with 'Snap to Grid' option switched on using a metric grid spacing (e.g. 1mm)</description>
+				</org.diylc.appframework.update.Change>
 				<org.diylc.appframework.update.Change>
 					<changeType>IMPROVEMENT</changeType>
 					<description>Do not allow for components like traces, jumpers and wires to be reduced to nothing by dragging connected components</description>
@@ -3300,7 +3304,7 @@
 				</org.diylc.appframework.update.Change>
 				<org.diylc.appframework.update.Change>
 					<changeType>IMPROVEMENT</changeType>
-					<description>Moving a rigid component (e.g. tube/transistor symbol) which is not snapped to grid makes it snap back to the grid when 'Snap to Grid' is switched on</description>
+					<description>Moving a rigid component (e.g. tube/transistor symbol) which is not snapped to grid makes it snap back to the grid when 'Snap to Grid' option is switched on</description>
 				</org.diylc.appframework.update.Change>		
 				<org.diylc.appframework.update.Change>
 					<changeType>IMPROVEMENT</changeType>

diylc/diylc-library/src/org/diylc/components/AbstractLeadedComponent.java

@@ -52,6 +52,7 @@ import org.diylc.core.annotations.EditableProperty;
 import org.diylc.core.annotations.PositiveNonZeroMeasureValidator;
 import org.diylc.core.measures.Size;
 import org.diylc.core.measures.SizeUnit;
+import org.diylc.presenter.CalcUtils;
 import org.diylc.utils.Constants;
 
 /**
@@ -102,6 +103,13 @@ public abstract class AbstractLeadedComponent<T> extends AbstractTransparentComp
     }    
     points[2] = calculateLabelPosition(points[0], points[1]);
   }
+  
+  @Override
+  public void createdIn(Project project) {
+    double targetSize = CalcUtils.findClosestMultiplierOf(project.getGridSpacing(), DEFAULT_SIZE, 2).convertToPixels();
+    points[0]= new Point2D.Double(-targetSize / 2, 0);
+    points[1] = new Point2D.Double(targetSize / 2, 0);
+  }
 
   protected boolean IsCopperArea() {
     return false;