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

don't call SetInsertionPointToEnd() on read-only combobox in wxGridCellChoiceEditor::Reset() neither (this had been already fixed for StartEdit() before) (#8843)

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@54413 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2008-06-29 00:19:44 +00:00
parent 929b79014a
commit b64845917a
1 changed files with 14 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
src/generic/grid.cpp
View File

@@ -1598,19 +1598,7 @@ void wxGridCellChoiceEditor::BeginEdit(int row, int col, wxGrid* grid)
m_startValue = grid->GetTable()->GetValue(row, col);
if (m_allowOthers)
{
Combo()->SetValue(m_startValue);
Combo()->SetInsertionPointEnd();
}
else // the combobox is read-only
{
// find the right position, or default to the first if not found
int pos = Combo()->FindString(m_startValue);
if (pos == wxNOT_FOUND)
pos = 0;
Combo()->SetSelection(pos);
}
Reset(); // this updates combo box to correspond to m_startValue
Combo()->SetFocus();
@@ -1638,8 +1626,19 @@ bool wxGridCellChoiceEditor::EndEdit(int row, int col,
void wxGridCellChoiceEditor::Reset()
{
Combo()->SetValue(m_startValue);
Combo()->SetInsertionPointEnd();
if (m_allowOthers)
{
Combo()->SetValue(m_startValue);
Combo()->SetInsertionPointEnd();
}
else // the combobox is read-only
{
// find the right position, or default to the first if not found
int pos = Combo()->FindString(m_startValue);
if (pos == wxNOT_FOUND)
pos = 0;
Combo()->SetSelection(pos);
}
}
void wxGridCellChoiceEditor::SetParameters(const wxString& params)