some tabular fixes for the problems reported by Helge

[lyx.git] / src / frontends / screen.C

diff --git a/src/frontends/screen.C b/src/frontends/screen.C
index 7b200812250067da8f9bf855c5ccaed8dd07675e..6308730a2024b786afcee8824d1fa9721c2683fc 100644 (file)
--- a/src/frontends/screen.C
+++ b/src/frontends/screen.C
@@ -21,6 +21,7 @@
 #include "BufferView.h"
 #include "buffer.h"
 #include "bufferparams.h"
+#include "coordcache.h"
 #include "cursor.h"
 #include "debug.h"
 #include "language.h"
@@ -121,12 +122,12 @@ SplashScreen::SplashScreen()
 
 
 LyXScreen::LyXScreen()
-       : cursor_visible_(false), greyed_out_(true)
+       : greyed_out_(true), cursor_visible_(false)
 {
        // Start loading the pixmap as soon as possible
        if (lyxrc.show_banner) {
                SplashScreen const & splash = SplashScreen::get();
-               splash.connect(boost::bind(&LyXScreen::greyOut, this));
+               splash.connect(boost::bind(&LyXScreen::checkAndGreyOut, this));
                splash.startLoading();
        }
 }
@@ -137,10 +138,23 @@ LyXScreen::~LyXScreen()
 }
 
 
+void LyXScreen::checkAndGreyOut()
+{
+       if (greyed_out_)
+               greyOut();
+}
+
+
 void LyXScreen::showCursor(BufferView & bv)
 {
-       // this is needed to make sure we copy back the right
-       // pixmap on the hide for the Qt frontend
+       // You are not expected to understand this. This forces Qt
+       // (the problem case) to deal with its event queue. This is
+       // necessary when holding down a key such as 'page down' or
+       // just typing: without this processing of the event queue,
+       // the cursor gets ahead of itself without a selection or
+       // workarea redraw having a chance to keep up. If you think
+       // you can remove this, try selecting text with the mouse
+       // in Qt, or holding Page Down on the User's Guide.
        lyx_gui::sync_events();
 
        if (cursor_visible_)
@@ -167,17 +181,18 @@ void LyXScreen::showCursor(BufferView & bv)
        if (realfont.language() == latex_language)
                shape = BAR_SHAPE;
 
-       int ascent = font_metrics::maxAscent(realfont);
-       int descent = font_metrics::maxDescent(realfont);
-       int h = ascent + descent;
+       LyXFont const font = bv.cursor().getFont();
+       int const asc = font_metrics::maxAscent(font);
+       int const des = font_metrics::maxDescent(font);
+       int h = asc + des;
        int x = 0;
        int y = 0;
        bv.cursor().getPos(x, y);
-       y -= ascent + bv.top_y();
+       y -= asc;
        //lyxerr << "LyXScreen::showCursor x: " << x << " y: " << y << endl;
 
-       // if it doesn't fit entirely on the screen, don't try to show it
-       if (y < 0 || y + h > workarea().workHeight())
+       // if it doesn't touch the screen, don't try to show it
+       if (y + h < 0 || y >= workarea().workHeight())
                return;
 
        cursor_visible_ = true;
@@ -204,82 +219,23 @@ void LyXScreen::toggleCursor(BufferView & bv)
 }
 
 
-bool LyXScreen::fitCursor(BufferView * bv)
-{
-       int const top_y = bv->top_y();
-       int const h = workarea().workHeight();
-       int newtop = top_y;
-       int x, y, asc, desc;
-
-       bv->cursor().getPos(x, y);
-       bv->cursor().getDim(asc, desc);
-       lyxerr << "LyXScreen::fitCursor: x: " << x << " y: " << y
-              << "  top_y: " << top_y << endl;
-
-       bool const big_row = h / 4 < asc + desc && asc + desc < h;
-
-       if (y + desc - top_y >= h) {
-               if (big_row)
-                       newtop = y + desc - h;
-               else
-                       newtop = y - h / 2;
-
-       } else if (top_y > max(y - asc, 0)) {
-               if (big_row)
-                       newtop = y - asc;
-               else {
-                       newtop = y - h / 2;
-                       newtop = min(newtop, top_y);
-               }
-       }
-
-       newtop = max(newtop, 0);
-       if (newtop == top_y)
-               return false;
-
-       bv->top_y(newtop);
-       return true;
-}
-
-
-void LyXScreen::redraw(BufferView & bv)
+void LyXScreen::redraw(BufferView & bv, ViewMetricsInfo const & vi)
 {
-       greyed_out_ = !bv.text();
-
-       if (greyed_out_) {
-               greyOut();
-               return;
-       }
-
+       greyed_out_ = false;
        workarea().getPainter().start();
-
        hideCursor();
-
-       int const y = paintText(bv);
-
-       // maybe we have to clear the screen at the bottom
-       int const y2 = workarea().workHeight();
-       if (y < y2 && bv.text()->isMainText()) {
-               workarea().getPainter().fillRectangle(0, y,
-                       workarea().workWidth(), y2 - y,
-                       LColor::bottomarea);
-       }
-
-       lyxerr << "Redraw screen" << endl;
-
+       paintText(bv, vi);
+       lyxerr[Debug::DEBUG] << "Redraw screen" << endl;
        expose(0, 0, workarea().workWidth(), workarea().workHeight());
-
        workarea().getPainter().end();
-
+       theCoords.doneUpdating();
        showCursor(bv);
 }
 
 
 void LyXScreen::greyOut()
 {
-       if (!greyed_out_)
-               return;
-
+       greyed_out_ = true;
        workarea().getPainter().start();
 
        workarea().getPainter().fillRectangle(0, 0,