[Marble-commits] KDE/kdeedu/marble/src/plugins/render/positionmarker

[Dennis Nienhüser]

SVN commit 1220034 by nienhueser:

Fix the default accuracy color missing a transparency value. Disable the position marker trail by default. In most situations (slow movement or medium or low zoom values) the circles overlap, leading to an overcrowded view. This needs to be improved before it can be the default for all users.

 M  +7 -3      PositionMarker.cpp  
 M  +1 -1      PositionMarkerConfigWidget.ui  


--- trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarker.cpp #1220033:1220034
@@ -48,7 +48,8 @@
       m_aboutDialog( 0 ),
       m_configDialog( 0 ),
       m_settings(),
-      m_visibleTrailPoints( 0 )
+      m_visibleTrailPoints( 0 ),
+      m_showTrail ( false )
 {
     // Create list of rects for trail with increasing size.
     // Last element is not drawn, it only stores the next position.
@@ -343,13 +344,16 @@
         settings.insert( "cursorSize", 1.0 );
     }
     if( !settings.contains( "acColor" ) ) {
-        settings.insert( "acColor", oxygenBrickRed4 );
+        QColor defaultColor = oxygenBrickRed4;
+        bool const smallScreen = MarbleGlobal::getInstance()->profiles() & MarbleGlobal::SmallScreen;
+        defaultColor.setAlpha( smallScreen ? 80 : 40 );
+        settings.insert( "acColor", defaultColor );
     }
     if( !settings.contains( "trailColor" ) ) {
         settings.insert( "trailColor", QColor( 0, 0, 255 ) );
     }
     if( !settings.contains( "showTrail" ) ) {
-        settings.insert( "showTrail", true );
+        settings.insert( "showTrail", false );
     }
 
     m_settings = settings;
--- trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarkerConfigWidget.ui #1220033:1220034
@@ -150,7 +150,7 @@
          <string>Trail</string>
         </property>
         <property name="checked">
-         <bool>true</bool>
+         <bool>false</bool>
         </property>
        </widget>
       </item>