compile and cursor positioning fixes

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@2718 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/samples/richedit/wxllist.cpp b/samples/richedit/wxllist.cpp
index 6fc798d84ae55317f72192868cf905dee7f52e5c..83ff44ef42007bddb38cd267427dfea75506cf7a 100644 (file)
--- a/samples/richedit/wxllist.cpp
+++ b/samples/richedit/wxllist.cpp
@@ -2085,47 +2085,24 @@ wxLayoutList::Recalculate(wxDC &dc, CoordType bottom)
 }
 
 void
 }
 
 void

-wxLayoutList::UpdateCursorScreenPos(wxDC &dc,
-                                    bool resetCursorMovedFlag,
-                                    const wxPoint& translate)
+wxLayoutList::UpdateCursorScreenPos(wxDC &dc)

 {
    wxCHECK_RET( m_CursorLine, "no cursor line" );
 
 {
    wxCHECK_RET( m_CursorLine, "no cursor line" );
 

-   if ( m_movedCursor )
-   {
-      // we need to save the current style, in case the layout() of
-      // the line changes it
-      wxLayoutStyleInfo SiBackup = m_CurrentStyleInfo;
-      m_CursorLine->Layout(dc, this,
-                           &m_CursorScreenPos, &m_CursorSize,
-                           m_CursorPos.x,
-                           /* suppress update */ true);
-      ApplyStyle(SiBackup, dc); // restore it
-
-      if ( resetCursorMovedFlag )
-      {
-#ifdef WXLAYOUT_USE_CARET
-            // adjust the caret position
-            wxPoint coords(m_CursorScreenPos);
-            coords += translate;
-
-            // and set it
-            m_caret->Move(coords);
-#endif // WXLAYOUT_USE_CARET
-
-         m_movedCursor = false;
-      }
-   }
+   // we need to save the current style, in case the layout() of the line
+   // changes it
+   wxLayoutStyleInfo SiBackup = m_CurrentStyleInfo;
+   m_CursorLine->Layout(dc, this,
+                        &m_CursorScreenPos, &m_CursorSize,
+                        m_CursorPos.x,
+                        true /* suppress update */);
+   ApplyStyle(SiBackup, dc); // restore it

 }
 
 wxPoint
 wxLayoutList::GetCursorScreenPos(wxDC &dc)
 {
 }
 
 wxPoint
 wxLayoutList::GetCursorScreenPos(wxDC &dc)
 {

-   // this function is called with wxMemoryDC argument from ScrollToCursor(),
-   // for example, so it shouldn't clear "cursor moved" flag - or else the
-   // cursor won't be moved when UpdateCursorScreenPos() is called with the
-   // "real" (i.e. the one used for drawing) wxDC.
-   UpdateCursorScreenPos(dc, false /* don't reset the flag */);
+   UpdateCursorScreenPos(dc);

 
    return m_CursorScreenPos;
 }
 
    return m_CursorScreenPos;
 }


@@ -2136,8 +2113,7 @@ wxLayoutList::GetCursorScreenPos(wxDC &dc)
 */
 void
 wxLayoutList::Layout(wxDC &dc, CoordType bottom, bool forceAll,
 */
 void
 wxLayoutList::Layout(wxDC &dc, CoordType bottom, bool forceAll,

-                     wxPoint *cpos = NULL,
-                     wxPoint *csize = NULL)
+                     wxPoint *cpos, wxPoint *csize)

 {
    // first, make sure everything is calculated - this might not be
    // needed, optimise it later
 {
    // first, make sure everything is calculated - this might not be
    // needed, optimise it later


@@ -2184,7 +2160,7 @@ wxLayoutList::Layout(wxDC &dc, CoordType bottom, bool forceAll,
 }
 
 wxPoint
 }
 
 wxPoint

-wxLayoutList::GetScreenPos(wxDC &dc, const wxPoint &cpos, wxPoint *csize = NULL)
+wxLayoutList::GetScreenPos(wxDC &dc, const wxPoint &cpos, wxPoint *csize)

 {
    wxPoint pos = cpos;
    Layout(dc, -1, false, &pos, csize);
 {
    wxPoint pos = cpos;
    Layout(dc, -1, false, &pos, csize);


@@ -2320,6 +2296,13 @@ wxLayoutList::GetSize(void) const
 void
 wxLayoutList::DrawCursor(wxDC &dc, bool active, wxPoint const &translate)
 {
 void
 wxLayoutList::DrawCursor(wxDC &dc, bool active, wxPoint const &translate)
 {

+   if ( m_movedCursor )
+   {
+      UpdateCursorScreenPos(dc);
+
+      m_movedCursor = false;
+   }
+

    wxPoint coords(m_CursorScreenPos);
    coords += translate;
 
    wxPoint coords(m_CursorScreenPos);
    coords += translate;
 


@@ -2334,7 +2317,9 @@ wxLayoutList::DrawCursor(wxDC &dc, bool active, wxPoint const &translate)
    wxLogStatus("Cursor is at (%d, %d)", m_CursorPos.x, m_CursorPos.y);
 #endif
 
    wxLogStatus("Cursor is at (%d, %d)", m_CursorPos.x, m_CursorPos.y);
 #endif
 

-#ifndef WXLAYOUT_USE_CARET
+#ifdef WXLAYOUT_USE_CARET
+   m_caret->Move(coords);
+#else // !WXLAYOUT_USE_CARET

    dc.SetBrush(*wxBLACK_BRUSH);
    dc.SetLogicalFunction(wxXOR);
    dc.SetPen(wxPen(*wxBLACK,1,wxSOLID));
    dc.SetBrush(*wxBLACK_BRUSH);
    dc.SetLogicalFunction(wxXOR);
    dc.SetPen(wxPen(*wxBLACK,1,wxSOLID));

diff --git a/samples/richedit/wxllist.h b/samples/richedit/wxllist.h
index 5f7a9191a5d685720d0d0b344ba186f2d82ea9b3..d0d7c94ae61aa97cf028005fd558ad815bc6bb78 100644 (file)
--- a/samples/richedit/wxllist.h
+++ b/samples/richedit/wxllist.h
@@ -948,7 +948,7 @@ public:
        @return The cursor position on the DC.
    */
    wxPoint GetScreenPos(wxDC &dc, const wxPoint &cpos, wxPoint *csize = NULL);
        @return The cursor position on the DC.
    */
    wxPoint GetScreenPos(wxDC &dc, const wxPoint &cpos, wxPoint *csize = NULL);

-   
+

    /** Calculates new sizes for everything in the list, like Layout()
        but this is needed after the list got changed.
        @param dc the wxDC to draw on
    /** Calculates new sizes for everything in the list, like Layout()
        but this is needed after the list got changed.
        @param dc the wxDC to draw on


@@ -968,16 +968,11 @@ public:
        @return cursor position in pixels
    */
    wxPoint GetCursorScreenPos(wxDC &dc);
        @return cursor position in pixels
    */
    wxPoint GetCursorScreenPos(wxDC &dc);

+

    /** Calculates the cursor position on the screen.
        @param dc the dc to use for cursor position calculations
    /** Calculates the cursor position on the screen.
        @param dc the dc to use for cursor position calculations

-       @param resetCursorMovedFlag: if true, reset "cursor moved" flag
-       @param translate optional translation of cursor coords on screen
-       

    */
    */

-   void UpdateCursorScreenPos(wxDC &dc,
-                              bool resetCursorMovedFlag = true,
-                              const wxPoint& translate = wxPoint(0,
-                                                                 0));
+   void UpdateCursorScreenPos(wxDC &dc);

 
    /** Draws the cursor.
        @param active If true, draw a bold cursor to mark window as
 
    /** Draws the cursor.
        @param active If true, draw a bold cursor to mark window as

diff --git a/samples/richedit/wxlwindow.cpp b/samples/richedit/wxlwindow.cpp
index 5df8db37c5f5e1e9af0a2032300fbf910b339d08..e893d1adac82d01404a7a096c9bcb948d8c7b0a1 100644 (file)
--- a/samples/richedit/wxlwindow.cpp
+++ b/samples/richedit/wxlwindow.cpp
@@ -288,11 +288,16 @@ wxLayoutWindow::OnMouse(int eventId, wxMouseEvent& event)
       x0 *= dx; y0 *= dy;
 
       wxPoint offset(-x0+WXLO_XOFFSET, -y0+WXLO_YOFFSET);
       x0 *= dx; y0 *= dy;
 
       wxPoint offset(-x0+WXLO_XOFFSET, -y0+WXLO_YOFFSET);

-      m_llist->UpdateCursorScreenPos(dc, true, offset);

 
       if(m_CursorVisibility == -1)
          m_CursorVisibility = 1;
 
 
       if(m_CursorVisibility == -1)
          m_CursorVisibility = 1;
 

+      if(m_CursorVisibility != 0)
+      {
+         // draw a thick cursor for    editable windows with focus
+         m_llist->DrawCursor(dc, m_HaveFocus && IsEditable(), offset);
+      }
+

       // VZ: this should be unnecessary because mouse can only click on a
       //     visible part of the canvas
 #if 0
       // VZ: this should be unnecessary because mouse can only click on a
       //     visible part of the canvas
 #if 0


@@ -620,10 +625,11 @@ wxLayoutWindow::OnPaint( wxPaintEvent &WXUNUSED(event))
 void
 wxLayoutWindow::DoPaint(const wxRect *updateRect)
 {
 void
 wxLayoutWindow::DoPaint(const wxRect *updateRect)
 {

-#ifndef __WXMSW__
+   // Causes bad flicker under wxGTK!!!
+#ifdef __WXGTK__

    InternalPaint(updateRect);
 #else
    InternalPaint(updateRect);
 #else

-   Refresh(FALSE, updateRect); // Causes bad flicker under wxGTK!!!
+   Refresh(FALSE); //, updateRect);

 
    if ( !::UpdateWindow(GetHwnd()) )
       wxLogLastError("UpdateWindow");
 
    if ( !::UpdateWindow(GetHwnd()) )
       wxLogLastError("UpdateWindow");


@@ -722,10 +728,9 @@ wxLayoutWindow::InternalPaint(const wxRect *updateRect)
    m_llist->InvalidateUpdateRect();
    if(m_CursorVisibility != 0)
    {
    m_llist->InvalidateUpdateRect();
    if(m_CursorVisibility != 0)
    {

-      m_llist->UpdateCursorScreenPos(dc, true, offset);
+      // draw a thick cursor for editable windows with focus

       m_llist->DrawCursor(*m_memDC,
       m_llist->DrawCursor(*m_memDC,

-                          m_HaveFocus && IsEditable(), // draw a thick
-                          // cursor for    editable windows with focus
+                          m_HaveFocus && IsEditable(),

                           offset);
    }
 
                           offset);
    }