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

Fixed a control-arrow bug

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@44559 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Julian Smart
2007-02-23 10:30:40 +00:00
parent b03466749c
commit 0871b34352
1 changed files with 25 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
src/richtext/richtextctrl.cpp
View File

@@ -1556,6 +1556,11 @@ bool wxRichTextCtrl::PageDown(int noPages, int flags)
return false; return false;
} }
static bool wxRichTextCtrlIsWhitespace(const wxString& str)
{
return str == wxT(" ") || str == wxT("\t");
}
// Finds the caret position for the next word // Finds the caret position for the next word
long wxRichTextCtrl::FindNextWordPosition(int direction) const long wxRichTextCtrl::FindNextWordPosition(int direction) const
{ {
@@ -1570,7 +1575,12 @@ long wxRichTextCtrl::FindNextWordPosition(int direction) const
{ {
// i is in character, not caret positions // i is in character, not caret positions
wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i)); wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i));
if (text != wxT(" ") && !text.empty()) wxRichTextLine* line = GetBuffer().GetLineAtPosition(i, false);
if (line && (i == line->GetAbsoluteRange().GetEnd()))
{
break;
}
else if (!wxRichTextCtrlIsWhitespace(text) && !text.empty())
i += direction; i += direction;
else else
{ {
@@ -1581,9 +1591,13 @@ long wxRichTextCtrl::FindNextWordPosition(int direction) const
{ {
// i is in character, not caret positions // i is in character, not caret positions
wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i)); wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i));
wxRichTextLine* line = GetBuffer().GetLineAtPosition(i, false);
if (line && (i == line->GetAbsoluteRange().GetEnd()))
return wxMax(-1, i);
if (text.empty()) // End of paragraph, or maybe an image if (text.empty()) // End of paragraph, or maybe an image
return wxMax(-1, i - 1); return wxMax(-1, i - 1);
else if (text == wxT(" ") || text.empty()) else if (wxRichTextCtrlIsWhitespace(text) || text.empty())
i += direction; i += direction;
else else
{ {
@@ -1604,9 +1618,11 @@ long wxRichTextCtrl::FindNextWordPosition(int direction) const
{ {
// i is in character, not caret positions // i is in character, not caret positions
wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i)); wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i));
if (text.empty()) // End of paragraph, or maybe an image wxRichTextLine* line = GetBuffer().GetLineAtPosition(i, false);
if (text.empty() || (line && (i == line->GetAbsoluteRange().GetStart()))) // End of paragraph, or maybe an image
break; break;
else if (text == wxT(" ") || text.empty()) else if (wxRichTextCtrlIsWhitespace(text) || text.empty())
i += direction; i += direction;
else else
break; break;
@@ -1616,7 +1632,11 @@ long wxRichTextCtrl::FindNextWordPosition(int direction) const
{ {
// i is in character, not caret positions // i is in character, not caret positions
wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i)); wxString text = GetBuffer().GetTextForRange(wxRichTextRange(i, i));
if (text != wxT(" ") /* && !text.empty() */) wxRichTextLine* line = GetBuffer().GetLineAtPosition(i, false);
if (line && line->GetAbsoluteRange().GetStart() == i)
return i-1;
if (!wxRichTextCtrlIsWhitespace(text) /* && !text.empty() */)
i += direction; i += direction;
else else
{ {