Remove the connection LED because it is not actually required. If the connection is successful, the mixer fader appears and shows the connection. If the connection fails, no fader is visible or the fader disappears after a while. In both cases it is clearly seen that the connection is lost and therefore the connection LED is not required.

This commit is contained in:
Volker Fischer 2014-02-25 15:38:11 +00:00
parent e85b0f93ed
commit 598017e47f
2 changed files with 3 additions and 104 deletions

View file

@ -164,20 +164,6 @@ CClientDlg::CClientDlg ( CClient* pNCliP,
rbtReverbSelR->setWhatsThis ( strRevChanSel ); rbtReverbSelR->setWhatsThis ( strRevChanSel );
rbtReverbSelR->setAccessibleName ( tr ( "Right channel selection for reverberation" ) ); rbtReverbSelR->setAccessibleName ( tr ( "Right channel selection for reverberation" ) );
// connection LED
QString strLEDConnection = tr ( "<b>Connection Status LED:</b> "
"The connection status LED indicator shows the current connection "
"status. If the light is green, a successful connection to the "
"server is established. If the light blinks red right after "
"pressing the connect button, the server address is invalid. "
"If the light turns red and stays red, the connection to the "
"server is lost." );
lblConnection->setWhatsThis ( strLEDConnection );
ledConnection->setWhatsThis ( strLEDConnection );
ledConnection->setAccessibleName ( tr ( "Connection status LED indicator" ) );
// delay LED // delay LED
QString strLEDDelay = tr ( "<b>Delay Status LED:</b> " QString strLEDDelay = tr ( "<b>Delay Status LED:</b> "
"The delay status LED indicator shows the current audio delay " "The delay status LED indicator shows the current audio delay "
@ -242,7 +228,6 @@ CClientDlg::CClientDlg ( CClient* pNCliP,
lbrInputLevelR->setValue ( 0 ); lbrInputLevelR->setValue ( 0 );
// init status LEDs // init status LEDs
ledConnection->Reset();
ledBuffers->Reset(); ledBuffers->Reset();
ledDelay->Reset(); ledDelay->Reset();
@ -673,8 +658,10 @@ void CClientDlg::OnConnectDisconBut()
void CClientDlg::OnDisconnected() void CClientDlg::OnDisconnected()
{ {
// channel is now disconnected, clear mixer board (remove all faders) // channel is now disconnected, clear mixer board (remove all faders) and
// reset the delay LED (since this is only updated on an active connection)
MainMixerBoard->HideAll(); MainMixerBoard->HideAll();
ledDelay->Reset();
UpdateDisplay(); UpdateDisplay();
} }
@ -1003,11 +990,6 @@ void CClientDlg::Connect ( const QString& strSelectedAddress,
TimerBuffersLED.start ( BUFFER_LED_UPDATE_TIME_MS ); TimerBuffersLED.start ( BUFFER_LED_UPDATE_TIME_MS );
TimerPing.start ( PING_UPDATE_TIME_MS ); TimerPing.start ( PING_UPDATE_TIME_MS );
} }
else
{
// show the error as red light
ledConnection->SetLight ( CMultiColorLED::RL_RED );
}
} }
void CClientDlg::Disconnect() void CClientDlg::Disconnect()
@ -1043,7 +1025,6 @@ OnTimerStatus();
// reset LEDs // reset LEDs
ledConnection->Reset();
ledBuffers->Reset(); ledBuffers->Reset();
ledDelay->Reset(); ledDelay->Reset();
ClientSettingsDlg.ResetStatusAndPingLED(); ClientSettingsDlg.ResetStatusAndPingLED();
@ -1054,21 +1035,6 @@ OnTimerStatus();
void CClientDlg::UpdateDisplay() void CClientDlg::UpdateDisplay()
{ {
// update status LEDs
if ( pClient->IsRunning() )
{
if ( pClient->IsConnected() )
{
// connection LED
ledConnection->SetLight ( CMultiColorLED::RL_GREEN );
}
else
{
// connection LED
ledConnection->SetLight ( CMultiColorLED::RL_RED );
}
}
// update settings/chat buttons (do not fire signals since it is an update) // update settings/chat buttons (do not fire signals since it is an update)
if ( chbSettings->isChecked() && !ClientSettingsDlg.isVisible() ) if ( chbSettings->isChecked() && !ClientSettingsDlg.isVisible() )
{ {

View file

@ -112,58 +112,6 @@
</property> </property>
</spacer> </spacer>
</item> </item>
<item>
<layout class="QHBoxLayout">
<item>
<widget class="QLabel" name="lblConnection">
<property name="text">
<string>Connection</string>
</property>
<property name="alignment">
<set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set>
</property>
<property name="wordWrap">
<bool>false</bool>
</property>
</widget>
</item>
<item>
<widget class="CMultiColorLED" name="ledConnection" native="true">
<property name="sizePolicy">
<sizepolicy hsizetype="Fixed" vsizetype="Fixed">
<horstretch>0</horstretch>
<verstretch>0</verstretch>
</sizepolicy>
</property>
<property name="minimumSize">
<size>
<width>14</width>
<height>14</height>
</size>
</property>
<property name="maximumSize">
<size>
<width>14</width>
<height>14</height>
</size>
</property>
</widget>
</item>
</layout>
</item>
<item>
<spacer>
<property name="orientation">
<enum>Qt::Vertical</enum>
</property>
<property name="sizeHint" stdset="0">
<size>
<width>10</width>
<height>0</height>
</size>
</property>
</spacer>
</item>
<item> <item>
<layout class="QHBoxLayout"> <layout class="QHBoxLayout">
<item> <item>
@ -511,12 +459,6 @@
</item> </item>
<item> <item>
<layout class="QHBoxLayout"> <layout class="QHBoxLayout">
<property name="spacing">
<number>6</number>
</property>
<property name="margin">
<number>0</number>
</property>
<item> <item>
<spacer> <spacer>
<property name="orientation"> <property name="orientation">
@ -576,9 +518,6 @@
<property name="spacing"> <property name="spacing">
<number>3</number> <number>3</number>
</property> </property>
<property name="margin">
<number>0</number>
</property>
<item> <item>
<widget class="QLabel" name="lblAudioReverb"> <widget class="QLabel" name="lblAudioReverb">
<property name="text"> <property name="text">
@ -591,12 +530,6 @@
</item> </item>
<item> <item>
<layout class="QHBoxLayout"> <layout class="QHBoxLayout">
<property name="spacing">
<number>6</number>
</property>
<property name="margin">
<number>0</number>
</property>
<item> <item>
<spacer> <spacer>
<property name="orientation"> <property name="orientation">