Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't set focus from wxTimePickerCtrl::SetValue() on GTK

Browse Source 
Match the wxMSW behavior. See #19047
This commit is contained in:
Paul Cornett
2021-01-18 22:24:39 -08:00
parent 72ac21cf18
commit cea8e60dea
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/generic/timectrlg.cpp
View File

@@ -72,6 +72,7 @@ public:
m_btn = new wxSpinButton(ctrl, wxID_ANY, m_btn = new wxSpinButton(ctrl, wxID_ANY,
wxDefaultPosition, wxDefaultSize, wxDefaultPosition, wxDefaultSize,
wxSP_VERTICAL | wxSP_WRAP); wxSP_VERTICAL | wxSP_WRAP);
m_btn->SetCanFocus(false);
m_currentField = Field_Hour; m_currentField = Field_Hour;
m_isFirstDigit = true; m_isFirstDigit = true;
@@ -306,11 +307,13 @@ private:
void OnArrowUp(wxSpinEvent& WXUNUSED(event)) void OnArrowUp(wxSpinEvent& WXUNUSED(event))
{ {
m_text->SetFocus();
ChangeCurrentFieldBy1(Dir_Up); ChangeCurrentFieldBy1(Dir_Up);
} }
void OnArrowDown(wxSpinEvent& WXUNUSED(event)) void OnArrowDown(wxSpinEvent& WXUNUSED(event))
{ {
m_text->SetFocus();
ChangeCurrentFieldBy1(Dir_Down); ChangeCurrentFieldBy1(Dir_Down);
} }
@@ -370,8 +373,6 @@ private:
// Select the currently actively field. // Select the currently actively field.
void HighlightCurrentField() void HighlightCurrentField()
{ {
m_text->SetFocus();
const CharRange range = GetFieldRange(m_currentField); const CharRange range = GetFieldRange(m_currentField);
m_text->SetSelection(range.from, range.to); m_text->SetSelection(range.from, range.to);