comparison layer/TimeValueLayer.cpp @ 296:ea37c229a578

* Fix #1757772 tempo, dynamic related plug-ins bug -- make auto-align
  only auto-align if there is a unit involved
* Fix #1755366 text layer bug in retrieved session
* Fix input model selection in plugin parameter dialog (was being ignored)
* Use lighter background than the standard widget one for panes (assuming the
  widget background is light but not white) -- similarly darker if dark
* Fix colour reference counting in loaded session in SingleColourLayer
* Reset overview pane colour when switching dark background on or off

 }
 
 bool
 TimeValueLayer::getDisplayExtents(float &min, float &max) const
 {
-    if (!m_model || m_verticalScale == AutoAlignScale) return false;
+    if (!m_model || shouldAutoAlign()) return false;
 
     min = m_model->getValueMinimum();
     max = m_model->getValueMaximum();
     return true;
 }

 {
     min = 0.0;
     max = 0.0;
     log = false;
 
-    if (m_verticalScale == AutoAlignScale) {
+    if (shouldAutoAlign()) {
 
         if (!v->getValueExtents(m_model->getScaleUnits(), min, max, log)) {
             min = m_model->getValueMinimum();
             max = m_model->getValueMaximum();
         } else if (log) {

     }
 
     return val;
 }
 
+bool
+TimeValueLayer::shouldAutoAlign() const
+{
+    if (!m_model) return false;
+    QString unit = m_model->getScaleUnits();
+    return (m_verticalScale == AutoAlignScale && unit != "");
+}
+
 QColor
 TimeValueLayer::getColourForValue(View *v, float val) const
 {
     float min, max;
     bool log;

     long illuminateFrame = -1;
 
     if (v->shouldIlluminateLocalFeatures(this, localPos)) {
 	SparseTimeValueModel::PointList localPoints =
 	    getLocalPoints(v, localPos.x());
-        std::cerr << "TimeValueLayer: " << localPoints.size() << " local points" << std::endl;
+//        std::cerr << "TimeValueLayer: " << localPoints.size() << " local points" << std::endl;
 	if (!localPoints.empty()) illuminateFrame = localPoints.begin()->frame;
     }
 
     int w =
 	v->getXForFrame(frame0 + m_model->getResolution()) -