Enable use of space bar for PTT when in the FreeDV Reporter window.

drowe67 · Jan 21, 2024 · 13d1f23 · 13d1f23
1 parent 943dad8
commit 13d1f23
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/src/gui/dialogs/freedv_reporter.cpp b/src/gui/dialogs/freedv_reporter.cpp
@@ -392,6 +392,11 @@ FreeDVReporterDialog::~FreeDVReporterDialog()
     m_bandFilter->Disconnect(wxEVT_TEXT, wxCommandEventHandler(FreeDVReporterDialog::OnBandFilterChange), NULL, this);
 }
 
+bool FreeDVReporterDialog::isTextMessageFieldInFocus()
+{
+    return m_statusMessage->HasFocus();
+}
+
 void FreeDVReporterDialog::refreshLayout()
 {
     int colOffset = 0;

diff --git a/src/gui/dialogs/freedv_reporter.h b/src/gui/dialogs/freedv_reporter.h
@@ -68,6 +68,8 @@ class FreeDVReporterDialog : public wxDialog
 
         void setBandFilter(FilterFrequency freq);
 
+        bool isTextMessageFieldInFocus();
+
     protected:
 
         // Handlers for events.

diff --git a/src/ongui.cpp b/src/ongui.cpp
@@ -787,7 +787,12 @@ int MainApp::FilterEvent(wxEvent& event)
         (((wxKeyEvent&)event).GetKeyCode() == WXK_SPACE))
         {
             // only use space to toggle PTT if we are running and no modal dialogs (like options) up
-            if (frame->m_RxRunning && frame->IsActive() && wxGetApp().appConfiguration.enableSpaceBarForPTT && !frame->isReceiveOnly()) {
+            bool mainWindowActive = frame->IsActive();
+            bool reporterActiveButNotUpdatingTextMessage = 
+                frame->m_reporterDialog != nullptr && frame->m_reporterDialog->IsActive() && 
+                !frame->m_reporterDialog->isTextMessageFieldInFocus();
+            if (frame->m_RxRunning && (mainWindowActive || reporterActiveButNotUpdatingTextMessage) && 
+                wxGetApp().appConfiguration.enableSpaceBarForPTT && !frame->isReceiveOnly()) {
 
                 // space bar controls rx/rx if keyer not running
                 if (frame->vk_state == VK_IDLE) {