comparison view/View.cpp @ 1401:28075cc658c9

Scale pen for retina display on macOS as well as for hidpi display elsewhere

         if ((*i)->needsTextLabelHeight()) {
             sortedLayers[getObjectExportId(*i)] = *i;
         }
     }
 
-    int y = ViewManager::scalePixelSize(15) + paint.fontMetrics().ascent();
+    int y = scaleSize(15) + paint.fontMetrics().ascent();
 
     for (std::map<int, Layer *>::const_iterator i = sortedLayers.begin();
          i != sortedLayers.end(); ++i) {
         if (i->second == layer) break;
         y += paint.fontMetrics().height();

     pb->setTextVisible(false);
 
     QPushButton *cancel = new QPushButton(this);
     cancel->setIcon(IconLoader().load("cancel"));
     cancel->setFlat(true);
-    int scaled20 = ViewManager::scalePixelSize(20);
+    int scaled20 = scaleSize(20);
     cancel->setFixedSize(QSize(scaled20, scaled20));
     connect(cancel, SIGNAL(clicked()), this, SLOT(cancelClicked()));
     
     ProgressBarRec pbr;
     pbr.cancel = cancel;

 //    cerr << "View::getZoomLevelIndex: " << z << " -> " << n << " (max)"
 //         << endl;
     return n;
 }
 
+double
+View::scaleSize(double size) const
+{
+    static double ratio = 0.0;
+
+    if (ratio == 0.0) {
+        double baseEm;
+#ifdef Q_OS_MAC
+        baseEm = 17.0;
+#else
+        baseEm = 15.0;
+#endif
+        double em = QFontMetrics(QFont()).height();
+        ratio = em / baseEm;
+
+        SVDEBUG << "View::scaleSize: ratio is " << ratio
+                << " (em = " << em << ")" << endl;
+
+        if (ratio < 1.0) {
+            SVDEBUG << "View::scaleSize: rounding ratio up to 1.0" << endl;
+            ratio = 1.0;
+        }
+    }
+
+    return size * ratio;
+}
+
+double
+View::scalePenWidth(double width) const 
+{
+    if (width <= 0) { // zero-width pen, produce a scaled one-pixel pen
+        width = 1;
+    }
+    double ratio = scaleSize(1.0);
+    return width * sqrt(ratio);
+}
+
+QPen
+View::scalePen(QPen pen) const 
+{
+    return QPen(pen.color(), scalePenWidth(pen.width()));
+}
+
 bool
 View::areLayerColoursSignificant() const
 {
     for (LayerList::const_iterator i = m_layerStack.begin(); i != m_layerStack.end(); ++i) {
         if ((*i)->getLayerColourSignificance() ==

                     i->second.lastCheck = 0;
                     timer->setInterval(2000);
                     timer->start();
                 }
 
-                int scaled20 = ViewManager::scalePixelSize(20);
+                int scaled20 = scaleSize(20);
 
                 cancel->move(0, ph - pb->height()/2 - scaled20/2);
                 cancel->show();
 
                 pb->setValue(completion);

 
         bool illuminateThis =
             (illuminateFrame >= 0 && i->contains(illuminateFrame));
 
         double h = height();
-        double penWidth = PaintAssistant::scalePenWidth(1.0);
+        double penWidth = scalePenWidth(1.0);
         double half = penWidth/2.0;
 
         paint.setPen(QPen(QColor(150, 150, 255), penWidth));
 
         if (translucent && shouldLabelSelections()) {

             paint.drawRect(QRectF(p0, half, p1 - p0, h - penWidth));
         }
 
         if (illuminateThis) {
             paint.save();
-            penWidth = PaintAssistant::scalePenWidth(2.0);
+            penWidth = scalePenWidth(2.0);
             half = penWidth/2.0;
             paint.setPen(QPen(getForeground(), penWidth));
             if (closeToLeft) {
                 paint.drawLine(QLineF(p0, half, p1, half));
                 paint.drawLine(QLineF(p0, half, p0, h - half));