]> Shamusworld >> Repos - architektonas/blobdiff - src/dimension.cpp
Fixed loading code, added "Base Unit" dialog.
[architektonas] / src / dimension.cpp
index 203e26ffead1dddb6400d67672fd4d29855f3395..2a989dccec0de1b91fb3d68f8ab13e4d2ebbbcdf 100644 (file)
@@ -92,10 +92,14 @@ I believe they are pixels.
 
        // Calculate whether or not the arrowheads are too crowded to put inside
        // the extension lines. 9.0 is the length of the arrowhead.
-       double t = Vector::Parameter(position, endpoint, endpoint - (unit * 9.0 * size));
+//     double t = Vector::Parameter(position, endpoint, endpoint - (unit * 9.0 * size));
+//     double t = Vector::Parameter(position, endpoint, position + (unit * 9.0 * size));
+       double t = Vector::Parameter(endpoint, position, position + (unit * 9.0 * size));
 //printf("Dimension::Draw(): t = %lf\n", t);
 
-       if (t > 0.5)
+// On the screen, it's acting like this is actually 58%...
+// This is correct, we want it to happen at > 50%
+       if (t > 0.58)
        {
                // Draw main dimension line + arrowheads
                painter->DrawLine(p1, p2);
@@ -113,7 +117,7 @@ I believe they are pixels.
        }
 
        // Draw length of dimension line...
-       painter->SetFont(QFont("Arial", 10.0 * Painter::zoom * size));
+       painter->SetFont(QFont("Arial", 8.0 * Painter::zoom * size));
        Vector v1((p1.x - p2.x) / 2.0, (p1.y - p2.y) / 2.0);
        Point ctr = p2 + v1;
        QString dimText = QString("%1\"").arg(Vector(endpoint - position).Magnitude());
@@ -320,6 +324,25 @@ about keeping track of old states...
 }
 
 
+/*virtual*/ Object * Dimension::Copy(void)
+{
+#warning "!!! This doesn't take care of attached Dimensions !!!"
+/*
+This is a real problem. While having a pointer in the Dimension to this line's points
+is fast & easy, it creates a huge problem when trying to replicate an object like this.
+
+Maybe a way to fix that then, is to have reference numbers instead of pointers. That
+way, if you copy them, ... you might still have problems. Because you can't be sure if
+a copy will be persistant or not, you then *definitely* do not want them to have the
+same reference number.
+*/
+
+       Dimension * d = new Dimension(position, endpoint, dimensionType, parent);
+       d->size = size;
+       return d;
+}
+
+
 // Dimensions are special: they contain exactly *two* points. Here, we check
 // only for zero/non-zero in returning the correct points.
 /*virtual*/ Vector Dimension::GetPointAtParameter(double parameter)