From 9462dc8b9e45e47cc93c8950c752fdadebeca681 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 1 Sep 2026 15:25:08 +0000 Subject: [PATCH 1/4] Add keyboard hints to node edit popups Co-authored-by: rahulsavani <743139+rahulsavani@users.noreply.github.com> --- src/gui/efgtooltip.cc | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gui/efgtooltip.cc b/src/gui/efgtooltip.cc index 6a41f9a4e..5739253e6 100644 --- a/src/gui/efgtooltip.cc +++ b/src/gui/efgtooltip.cc @@ -253,6 +253,14 @@ void OutcomeEditorPopup::BuildControls() outerSizer->Add(new wxStaticLine(m_contentPanel), 0, wxEXPAND); + auto *hintText = new wxStaticText( + m_contentPanel, wxID_STATIC, _("Escape cancels and closes this window. Enter accepts.")); + hintText->SetForegroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT)); + wxFont hintFont = hintText->GetFont(); + hintFont.SetPointSize(hintFont.GetPointSize() - 1); + hintText->SetFont(hintFont); + outerSizer->Add(hintText, 0, wxALL, FromDIP(10)); + m_errorText = new wxStaticText(m_contentPanel, wxID_STATIC, wxEmptyString); m_errorText->SetForegroundColour(*wxRED); m_errorText->Wrap(FromDIP(260)); @@ -822,8 +830,10 @@ void AppendMovePopup::BuildControls() outerSizer->Add(new wxStaticLine(m_contentPanel), 0, wxEXPAND); - m_hintText = - new wxStaticText(m_contentPanel, wxID_ANY, _("Tab past the last action to add another")); + m_hintText = new wxStaticText( + m_contentPanel, wxID_ANY, + _("Tab past the last action to add another\nEscape cancels and closes this window. Enter " + "accepts.")); m_hintText->SetForegroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT)); wxFont hintFont = m_hintText->GetFont(); hintFont.SetPointSize(hintFont.GetPointSize() - 1); From 5531da8ca95441480532a06b33e41bbd05fcda2c Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 1 Sep 2026 15:25:42 +0000 Subject: [PATCH 2/4] Polish popup hint punctuation Co-authored-by: rahulsavani <743139+rahulsavani@users.noreply.github.com> --- src/gui/efgtooltip.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gui/efgtooltip.cc b/src/gui/efgtooltip.cc index 5739253e6..f372d312f 100644 --- a/src/gui/efgtooltip.cc +++ b/src/gui/efgtooltip.cc @@ -832,7 +832,7 @@ void AppendMovePopup::BuildControls() m_hintText = new wxStaticText( m_contentPanel, wxID_ANY, - _("Tab past the last action to add another\nEscape cancels and closes this window. Enter " + _("Tab past the last action to add another.\nEscape cancels and closes this window. Enter " "accepts.")); m_hintText->SetForegroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT)); wxFont hintFont = m_hintText->GetFont(); From ced1c5b8a544a05afd326d9fe1a119918505b6be Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 1 Sep 2026 15:40:31 +0000 Subject: [PATCH 3/4] Fix clang-format in outcome hint layout Co-authored-by: rahulsavani <743139+rahulsavani@users.noreply.github.com> --- src/gui/efgtooltip.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gui/efgtooltip.cc b/src/gui/efgtooltip.cc index f372d312f..9a60e0a33 100644 --- a/src/gui/efgtooltip.cc +++ b/src/gui/efgtooltip.cc @@ -253,8 +253,8 @@ void OutcomeEditorPopup::BuildControls() outerSizer->Add(new wxStaticLine(m_contentPanel), 0, wxEXPAND); - auto *hintText = new wxStaticText( - m_contentPanel, wxID_STATIC, _("Escape cancels and closes this window. Enter accepts.")); + auto *hintText = new wxStaticText(m_contentPanel, wxID_STATIC, + _("Escape cancels and closes this window. Enter accepts.")); hintText->SetForegroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT)); wxFont hintFont = hintText->GetFont(); hintFont.SetPointSize(hintFont.GetPointSize() - 1); From 81552c2edba376f57564d73b3472791ef7782623 Mon Sep 17 00:00:00 2001 From: Rahul Savani Date: Tue, 1 Sep 2026 17:29:33 +0100 Subject: [PATCH 4/4] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/gui/efgtooltip.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gui/efgtooltip.cc b/src/gui/efgtooltip.cc index 9a60e0a33..e6781dd19 100644 --- a/src/gui/efgtooltip.cc +++ b/src/gui/efgtooltip.cc @@ -259,6 +259,7 @@ void OutcomeEditorPopup::BuildControls() wxFont hintFont = hintText->GetFont(); hintFont.SetPointSize(hintFont.GetPointSize() - 1); hintText->SetFont(hintFont); + hintText->Wrap(FromDIP(260)); outerSizer->Add(hintText, 0, wxALL, FromDIP(10)); m_errorText = new wxStaticText(m_contentPanel, wxID_STATIC, wxEmptyString);