[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdemultimedia/kmix
From:       Christian Esken <esken () kde ! org>
Date:       2003-11-30 6:27:32
[Download RAW message or body]

CVS commit by esken: 

Don't show "Advanced" button ig there are no "advanced" switches (approved by Helio)


  M +7 -1      kmixerwidget.cpp   1.61


--- kdemultimedia/kmix/kmixerwidget.cpp  #1.60:1.61
@@ -161,4 +161,5 @@ KMixerWidget::createDeviceWidgets()
    MixSet mixSet = m_mixer->getMixSet();
    MixDevice *mixDevice = mixSet.first();
+   bool hasSwitches = false; // so we know whether showing the "advanced" checkbox \
makes sense  for ( ; mixDevice != 0; mixDevice = mixSet.next())
    {
@@ -197,4 +198,5 @@ KMixerWidget::createDeviceWidgets()
          {
             m_devSwitchLayout->addWidget( mdw, row, col );
+            hasSwitches = true;
             col++;
 
@@ -233,5 +235,8 @@ KMixerWidget::createDeviceWidgets()
 
                 balanceAndDetail->addSpacing( 10 );
+                if ( hasSwitches )
                 balanceAndDetail->addWidget( hideShowDetail);
+                else
+                  hideShowDetail->hide();
                 balanceAndDetail->addWidget( m_balanceSlider );
                 balanceAndDetail->addWidget( mixerName );
@@ -265,4 +270,5 @@ KMixerWidget::updateSize(bool force)
 {
         //kdDebug() << "KMixerWidget::updateSize(): (" << \
layout()->minimumSize().width() << "," << layout()->minimumSize().height() << ")\n"; \
+   // !!! The whole updateSize stuff must be reworked after KDE3.2 ... currently it \
is only working by chance  layout()->activate();
    setMinimumWidth( layout()->minimumSize().width() );


[prev in list] [next in list] [prev in thread] [next in thread]