From d35a4795f16e51f5773d00dce796f6948f82dae2 Mon Sep 17 00:00:00 2001 From: "B. Stack" Date: Fri, 16 Dec 2022 21:08:02 -0500 Subject: add upstream 11.29 --- wx+/context_menu.h | 36 +++++++++++++++++++++++++++++++++++- wx+/no_flicker.h | 45 +++++++++++++++++++++++++++++++++++++++------ 2 files changed, 74 insertions(+), 7 deletions(-) (limited to 'wx+') diff --git a/wx+/context_menu.h b/wx+/context_menu.h index 05dd58b8..d1d2bd9f 100644 --- a/wx+/context_menu.h +++ b/wx+/context_menu.h @@ -10,8 +10,9 @@ #include #include #include -#include #include +#include +#include #include "dc.h" @@ -123,6 +124,39 @@ void fixMenuIcons(wxMenu& menu) if (!menu.Insert(pos, menu.Remove(item))) //detach + reinsert assert(false); } + + +inline +void setClipboardText(const wxString& txt) +{ + wxClipboard& clip = *wxClipboard::Get(); + if (clip.Open()) + { + ZEN_ON_SCOPE_EXIT(clip.Close()); + [[maybe_unused]] const bool rv1 = clip.SetData(new wxTextDataObject(txt)); //ownership passed + [[maybe_unused]] const bool rv2 = clip.Flush(); + assert(rv1 && rv2); + } + else assert(false); +} + + +inline +std::optional getClipboardText() +{ + wxClipboard& clip = *wxClipboard::Get(); + if (clip.Open()) + { + ZEN_ON_SCOPE_EXIT(clip.Close()); + + //if (clip.IsSupported(wxDF_TEXT or wxDF_UNICODETEXT !???)) - superfluous? already handled by wxClipboard::GetData()!? + wxTextDataObject data; + if (clip.GetData(data)) + return data.GetText(); + } + else assert(false); + return std::nullopt; +} } #endif //CONTEXT_MENU_H_18047302153418174632141234 diff --git a/wx+/no_flicker.h b/wx+/no_flicker.h index 7fa4ae23..185ee052 100644 --- a/wx+/no_flicker.h +++ b/wx+/no_flicker.h @@ -107,15 +107,48 @@ void setTextWithUrls(wxRichTextCtrl& richCtrl, const wxString& newText) break; } + //register only once! => use a global function pointer, so that Unbind() works correctly: + using LaunchUrlFun = void(*)(wxTextUrlEvent& event); + static const LaunchUrlFun launchUrl = [](wxTextUrlEvent& event) { wxLaunchDefaultBrowser(event.GetString()); }; + + [[maybe_unused]] const bool unbindOk1 = richCtrl.Unbind(wxEVT_TEXT_URL, launchUrl); if (std::any_of(blocks.begin(), blocks.end(), [](const auto& item) { return item.first == BlockType::url; })) + /**/richCtrl.Bind(wxEVT_TEXT_URL, launchUrl); + + struct UserData : public wxObject + { + explicit UserData(wxRichTextCtrl& rtc) : richCtrl(rtc) {} + wxRichTextCtrl& richCtrl; + }; + using KeyEventsFun = void(*)(wxKeyEvent& event); + static const KeyEventsFun onKeyEvents = [](wxKeyEvent& event) { - //register only once! => use a global function pointer, so that Unbind() works correctly: - using LaunchUrlFun = void(*)(wxTextUrlEvent& event); - static const LaunchUrlFun launchUrl = [](wxTextUrlEvent& event) { wxLaunchDefaultBrowser(event.GetString()); }; + wxRichTextCtrl& richCtrl = dynamic_cast(event.GetEventUserData())->richCtrl; //unclear if we can rely on event.GetEventObject() == richCtrl - [[maybe_unused]] const bool unbindOk = richCtrl.Unbind(wxEVT_TEXT_URL, launchUrl); - richCtrl.Bind(wxEVT_TEXT_URL, launchUrl); - } + //CTRL/SHIFT + INS is broken for wxRichTextCtrl on Windows/Linux (apparently never was a thing on macOS) + if (event.ControlDown()) + switch (event.GetKeyCode()) + { + case WXK_INSERT: + case WXK_NUMPAD_INSERT: + assert(richCtrl.CanCopy()); //except when no selection + richCtrl.Copy(); + return; + } + + if (event.ShiftDown()) + switch (event.GetKeyCode()) + { + case WXK_INSERT: + case WXK_NUMPAD_INSERT: + assert(richCtrl.CanPaste()); //except wxTE_READONLY + richCtrl.Paste(); + return; + } + event.Skip(); + }; + [[maybe_unused]] const bool unbindOk2 = richCtrl.Unbind(wxEVT_KEY_DOWN, onKeyEvents); + /**/ richCtrl. Bind(wxEVT_KEY_DOWN, onKeyEvents, wxID_ANY, wxID_ANY, new UserData(richCtrl) /*pass ownership*/); } } } -- cgit