Merge pull request #440 from drowe67/ms-voicekeyer-ptt-color

Fix bug causing the PTT button to stay red after the voice keyer finishes TX.
drowe67 · Jun 16, 2023 · 195c047 · 195c047
2 parents fd53c75 + a0624f0
commit 195c047
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 3 deletions.
diff --git a/USER_MANUAL.html b/USER_MANUAL.html
@@ -608,6 +608,7 @@ <h2 data-number="17.1" id="v1.8.12-tbd-2023"><span class="header-section-number"
 <ul>
 <li>Clear audio plots when recording or playback starts. (PR #439)</li>
 <li>Clear button now clears the callsign list. (PR #436)</li>
+<li>Fix bug causing the PTT button to stay red after the voice keyer finishes TX. (PR #440)</li>
 </ul></li>
 <li>Build system:
 <ul>

diff --git a/USER_MANUAL.md b/USER_MANUAL.md
@@ -868,6 +868,7 @@ LDPC | Low Density Parity Check Codes - a family of powerful FEC codes
 1. Bugfixes:
     * Clear audio plots when recording or playback starts. (PR #439)
     * Clear button now clears the callsign list. (PR #436)
+    * Fix bug causing the PTT button to stay red after the voice keyer finishes TX. (PR #440)
 2. Build system:
     * Bump Codec2 version to v1.1.1. (PR #437)
 

diff --git a/USER_MANUAL.pdf b/USER_MANUAL.pdf
diff --git a/src/voicekeyer.cpp b/src/voicekeyer.cpp
@@ -77,14 +77,18 @@ void MainFrame::VoiceKeyerProcessEvent(int vk_event) {
         // to Mic In
 
         if (vk_event == VK_SPACE_BAR) {
-            m_btnTogPTT->SetValue(false); togglePTT();
+            m_btnTogPTT->SetValue(false); 
+            m_btnTogPTT->SetBackgroundColour(wxNullColour);
+            togglePTT();
             m_togBtnVoiceKeyer->SetValue(false);
             next_state = VK_IDLE;
             CallAfter([&]() { StopPlayFileToMicIn(); });
         }
 
         if (vk_event == VK_PLAY_FINISHED) {
-            m_btnTogPTT->SetValue(false); togglePTT();
+            m_btnTogPTT->SetValue(false); 
+            m_btnTogPTT->SetBackgroundColour(wxNullColour);
+            togglePTT();
             vk_repeat_counter++;
             if (vk_repeat_counter > vk_repeats) {
                 m_togBtnVoiceKeyer->SetValue(false);
@@ -154,7 +158,9 @@ void MainFrame::VoiceKeyerProcessEvent(int vk_event) {
     default:
         // catch anything we missed
 
-        m_btnTogPTT->SetValue(false); togglePTT();
+        m_btnTogPTT->SetValue(false); 
+        m_btnTogPTT->SetBackgroundColour(wxNullColour);
+        togglePTT();
         m_togBtnVoiceKeyer->SetValue(false);
         next_state = VK_IDLE;
     }