]> git.lyx.org Git - lyx.git/blobdiff - src/rowpainter.cpp
Reintroduce the code related to InsetEnvSeparator
[lyx.git] / src / rowpainter.cpp
index 0a902b7ef2544bf3b6cdf022fffb883fd748429b..ee4cd18c8a72da9f71ee7787805912eb7e8060cf 100644 (file)
 
 #include "rowpainter.h"
 
-#include "Bidi.h"
 #include "Buffer.h"
 #include "CoordCache.h"
 #include "Cursor.h"
 #include "BufferParams.h"
 #include "BufferView.h"
 #include "Changes.h"
-#include "Encoding.h"
 #include "Language.h"
 #include "Layout.h"
 #include "LyXRC.h"
@@ -54,14 +52,14 @@ namespace lyx {
 using frontend::Painter;
 using frontend::FontMetrics;
 
+
 RowPainter::RowPainter(PainterInfo & pi,
-       Text const & text, pit_type pit, Row const & row, Bidi & bidi, int x, int y)
+       Text const & text, pit_type pit, Row const & row, int x, int y)
        : pi_(pi), text_(text),
          text_metrics_(pi_.base.bv->textMetrics(&text)),
          pars_(text.paragraphs()),
          row_(row), pit_(pit), par_(text.paragraphs()[pit]),
-         pm_(text_metrics_.parMetrics(pit)),
-         bidi_(bidi), change_(pi_.change_),
+         pm_(text_metrics_.parMetrics(pit)), change_(pi_.change_),
          xo_(x), yo_(y), width_(text_metrics_.width()),
          solid_line_thickness_(1.0), solid_line_offset_(1),
          dotted_line_thickness_(1.0), dotted_line_offset_(2)
@@ -90,8 +88,8 @@ RowPainter::RowPainter(PainterInfo & pi,
        //lyxerr << "RowPainter: x: " << x_ << " xo: " << xo_ << " yo: " << yo_ << endl;
        //row_.dump();
 
-       LASSERT(pit >= 0, /**/);
-       LASSERT(pit < int(text.paragraphs().size()), /**/);
+       LBUFERR(pit >= 0);
+       LBUFERR(pit < int(text.paragraphs().size()));
 }
 
 
@@ -162,125 +160,46 @@ void RowPainter::paintInset(Inset const * inset, pos_type const pos)
 }
 
 
-void RowPainter::paintHebrewComposeChar(pos_type & vpos, FontInfo const & font)
-{
-       pos_type pos = bidi_.vis2log(vpos);
-
-       docstring str;
-
-       // first char
-       char_type c = par_.getChar(pos);
-       str += c;
-       ++vpos;
-
-       int const width = theFontMetrics(font).width(c);
-       int dx = 0;
-
-       for (pos_type i = pos - 1; i >= 0; --i) {
-               c = par_.getChar(i);
-               if (!Encodings::isHebrewComposeChar(c)) {
-                       if (isPrintableNonspace(c)) {
-                               int const width2 = pm_.singleWidth(i,
-                                       text_metrics_.displayFont(pit_, i));
-                               dx = (c == 0x05e8 || // resh
-                                     c == 0x05d3)   // dalet
-                                       ? width2 - width
-                                       : (width2 - width) / 2;
-                       }
-                       break;
-               }
-       }
-
-       // Draw nikud
-       pi_.pain.text(int(x_) + dx, yo_, str, font);
-}
-
-
-void RowPainter::paintArabicComposeChar(pos_type & vpos, FontInfo const & font)
-{
-       pos_type pos = bidi_.vis2log(vpos);
-       docstring str;
-
-       // first char
-       char_type c = par_.getChar(pos);
-       c = par_.transformChar(c, pos);
-       str += c;
-       ++vpos;
-
-       int const width = theFontMetrics(font).width(c);
-       int dx = 0;
-
-       for (pos_type i = pos - 1; i >= 0; --i) {
-               c = par_.getChar(i);
-               if (!Encodings::isArabicComposeChar(c)) {
-                       if (isPrintableNonspace(c)) {
-                               int const width2 = pm_.singleWidth(i,
-                                               text_metrics_.displayFont(pit_, i));
-                               dx = (width2 - width) / 2;
-                       }
-                       break;
-               }
-       }
-       // Draw nikud
-       pi_.pain.text(int(x_) + dx, yo_, str, font);
-}
-
-
-void RowPainter::paintChars(pos_type & vpos, FontInfo const & font,
-                           bool hebrew, bool arabic)
+void RowPainter::paintChars(pos_type & vpos, Font const & font)
 {
        // This method takes up 70% of time when typing
        pos_type pos = bidi_.vis2log(vpos);
+       pos_type start_pos = pos;
        // first character
-       char_type prev_char = par_.getChar(pos);
-       vector<char_type> str;
+       char_type c = par_.getChar(pos);
+       docstring str;
        str.reserve(100);
-       str.push_back(prev_char);
+
+       // special case for arabic
+       string const & lang = font.language()->lang();
+       bool const swap_paren = lang == "arabic_arabtex"
+               || lang == "arabic_arabi"
+               || lang == "farsi";
 
        // FIXME: Why only round brackets and why the difference to
        // Hebrew? See also Paragraph::getUChar
-       if (arabic) {
-               char_type c = str[0];
+       if (swap_paren) {
                if (c == '(')
                        c = ')';
                else if (c == ')')
                        c = '(';
-               str[0] = par_.transformChar(c, pos);
        }
+       str.push_back(c);
 
        pos_type const end = row_.endpos();
        FontSpan const font_span = par_.fontSpan(pos);
        // Track-change status.
        Change const & change_running = par_.lookupChange(pos);
 
-       // selected text?
-       bool const selection = (pos >= row_.sel_beg && pos < row_.sel_end)
-               || pi_.selected;
-
        // spelling correct?
        bool const spell_state =
                lyxrc.spellcheck_continuously && par_.isMisspelled(pos);
 
        // collect as much similar chars as we can
        for (++vpos ; vpos < end ; ++vpos) {
-               // Work-around bug #6920
-               // The bug can be reproduced with DejaVu font under Linux.
-               // The issue is that we compute the metrics character by character
-               // in ParagraphMetrics::singleWidth(); but we paint word by word
-               // for performance reason.
-               // Maybe a more general fix would be draw character by character
-               // for some predefined fonts on some platform. In arabic and
-               // Hebrew we already do paint this way.
-               if (prev_char == 'f' || lyxrc.force_paint_single_char)
-                       break;
-
                pos = bidi_.vis2log(vpos);
-               if (pos < font_span.first || pos > font_span.last)
-                       break;
 
-               bool const new_selection = pos >= row_.sel_beg && pos < row_.sel_end;
-               if (new_selection != selection)
-                       // Selection ends or starts here.
+               if (!font_span.inside(pos))
                        break;
 
                bool const new_spell_state =
@@ -299,68 +218,80 @@ void RowPainter::paintChars(pos_type & vpos, FontInfo const & font,
                if (c == '\t')
                        break;
 
-               if (!isPrintableNonspace(c))
-                       break;
-
-               /* Because we do our own bidi, at this point the strings are
-                * already in visual order. However, Qt also applies its own
-                * bidi algorithm to strings that it paints to the screen.
-                * Therefore, if we were to paint Hebrew/Arabic words as a
-                * single string, the letters in the words would get reversed
-                * again. In order to avoid that, we don't collect Hebrew/
-                * Arabic characters, but rather paint them one at a time.
-                * See also http://thread.gmane.org/gmane.editors.lyx.devel/79740
-                */
-               if (hebrew)
-                       break;
-
-               /* FIXME: these checks are irrelevant, since 'arabic' and
-                * 'hebrew' alone are already going to trigger a break.
-                * However, this should not be removed completely, because
-                * if an alternative solution is found which allows grouping
-                * of arabic and hebrew characters, then these breaks may have
-                * to be re-applied.
-
-               if (arabic && Encodings::isArabicComposeChar(c))
+               // When row_.separator == 0, it is possible to print a
+               // string longer than a word in one fell swoop.
+               // Therefore there is no need to break at spaces.
+               if (!isPrintableNonspace(c)
+                   && (c != ' ' || row_.separator > 0))
                        break;
 
-               if (hebrew && Encodings::isHebrewComposeChar(c))
-                       break;
-               */
-
                // FIXME: Why only round brackets and why the difference to
                // Hebrew? See also Paragraph::getUChar
-               if (arabic) {
+               if (swap_paren) {
                        if (c == '(')
                                c = ')';
                        else if (c == ')')
                                c = '(';
-                       c = par_.transformChar(c, pos);
-                       /* see comment in hebrew, explaining why we break */
-                       break;
                }
 
                str.push_back(c);
-               prev_char = c;
        }
 
-       docstring s(&str[0], str.size());
+       // Make pos point to the last character in the string.
+       // Using "pos = bidi_.vis2log(vpos)" does not work for some reason.
+       if (vpos < end)
+               pos = bidi_.vis2log(vpos - 1);
 
-       if (s[0] == '\t')
-               s.replace(0,1,from_ascii("    "));
+       // Now make pos point to the position _after_ the string.
+       // Using vis2log for that is not a good idea in general, we
+       // want logical ordering.
+       if (font.isVisibleRightToLeft())
+               --pos;
+       else
+               ++pos;
+
+       if (str[0] == '\t')
+               str.replace(0,1,from_ascii("    "));
+
+       /* Because we do our own bidi, at this point the strings are
+        * already in visual order. However, Qt also applies its own
+        * bidi algorithm to strings that it paints to the screen.
+        * Therefore, if we were to paint Hebrew/Arabic words as a
+        * single string, the letters in the words would get reversed
+        * again. In order to avoid that, we force LTR drawing.
+        * See also http://thread.gmane.org/gmane.editors.lyx.devel/79740
+        * for an earlier thread on the subject
+        */
+       if (font.isVisibleRightToLeft()) {
+               reverse(str.begin(), str.end());
+               // If the string is reversed, the positions need to be adjusted
+               ++pos;
+               ++start_pos;
+               swap(start_pos, pos);
+       } 
+
+       // at least part of text selected?
+       bool const some_sel = (pos >= row_.sel_beg && start_pos < row_.sel_end)
+               || pi_.selected;
+       // all the text selected?
+       bool const all_sel = (start_pos >= row_.sel_beg && pos < row_.sel_end)
+               || pi_.selected;
 
-       if (!selection && !change_running.changed()) {
-               x_ += pi_.pain.text(int(x_), yo_, s, font);
-               return;
+       if (all_sel) {
+               Font copy = font;
+               copy.fontInfo().setPaintColor(Color_selectiontext);
+               x_ += pi_.pain.text(int(x_), yo_, str, copy);
+       } else if (change_running.changed()) {
+               Font copy = font;
+               copy.fontInfo().setPaintColor(change_running.color());
+               x_ += pi_.pain.text(int(x_), yo_, str, copy);
+       } else if (!some_sel) {
+               x_ += pi_.pain.text(int(x_), yo_, str, font);
+       } else {
+               x_ += pi_.pain.text(int(x_), yo_, str, font, Color_selectiontext,
+                                   max(row_.sel_beg, start_pos) - start_pos,
+                                   min(row_.sel_end, pos) - start_pos);
        }
-
-       FontInfo copy = font;
-       if (change_running.changed())
-               copy.setPaintColor(change_running.color());
-       else if (selection)
-               copy.setPaintColor(Color_selectiontext);
-
-       x_ += pi_.pain.text(int(x_), yo_, s, copy);
 }
 
 
@@ -403,36 +334,14 @@ void RowPainter::paintMisspelledMark(double orig_x, bool changed)
 void RowPainter::paintFromPos(pos_type & vpos, bool changed)
 {
        pos_type const pos = bidi_.vis2log(vpos);
-       Font const orig_font = text_metrics_.displayFont(pit_, pos);
+       Font const font = text_metrics_.displayFont(pit_, pos);
        double const orig_x = x_;
 
-       // usual characters, no insets
-       char_type const c = par_.getChar(pos);
-
-       // special case languages
-       string const & lang = orig_font.language()->lang();
-       bool const hebrew = lang == "hebrew";
-       bool const arabic = lang == "arabic_arabtex" || lang == "arabic_arabi" ||
-                                               lang == "farsi";
-
-       // spelling correct?
-       bool const misspelled =
-               lyxrc.spellcheck_continuously && par_.isMisspelled(pos);
-
-       // draw as many chars as we can
-       if ((!hebrew && !arabic)
-               || (hebrew && !Encodings::isHebrewComposeChar(c))
-               || (arabic && !Encodings::isArabicComposeChar(c))) {
-               paintChars(vpos, orig_font.fontInfo(), hebrew, arabic);
-       } else if (hebrew) {
-               paintHebrewComposeChar(vpos, orig_font.fontInfo());
-       } else if (arabic) {
-               paintArabicComposeChar(vpos, orig_font.fontInfo());
-       }
-
-       paintForeignMark(orig_x, orig_font.language());
+       paintChars(vpos, font);
+       paintForeignMark(orig_x, font.language());
 
-       if (lyxrc.spellcheck_continuously && misspelled) {
+       // Paint the spelling mark if needed.
+       if (lyxrc.spellcheck_continuously && par_.isMisspelled(pos)) {
                // check for cursor position
                // don't draw misspelled marker for words at cursor position
                // we don't want to disturb the process of text editing
@@ -585,20 +494,14 @@ void RowPainter::paintFirst()
                }
        }
 
-       bool const is_seq = text_.isFirstInSequence(pit_);
+       bool const is_first =
+               text_.isFirstInSequence(pit_) || !layout.isParagraphGroup();
        //lyxerr << "paintFirst: " << par_.id() << " is_seq: " << is_seq << endl;
 
-       if (layout.labeltype >= LABEL_STATIC
-               && (layout.labeltype != LABEL_STATIC
-                       || layout.latextype != LATEX_ENVIRONMENT
-                       || is_seq)) {
+       if (layout.labelIsInline()
+                       && (layout.labeltype != LABEL_STATIC || is_first)) {
                paintLabel();
-       } else if (is_seq
-               && (layout.labeltype == LABEL_TOP_ENVIRONMENT
-                       || layout.labeltype == LABEL_BIBLIO
-                       || layout.labeltype == LABEL_CENTERED_TOP_ENVIRONMENT)) {
-               // the labels at the top of an environment.
-               // More or less for bibliography
+       } else if (is_first && layout.labelIsAbove()) {
                paintTopLevelLabel();
        }
 }
@@ -610,48 +513,21 @@ void RowPainter::paintLabel()
        if (str.empty())
                return;
 
-       BufferParams const & bparams = pi_.base.bv->buffer().params();
        bool const is_rtl = text_.isRTL(par_);
        Layout const & layout = par_.layout();
-       ParagraphParameters const & pparams = par_.params();
        FontInfo const font = labelFont();
        FontMetrics const & fm = theFontMetrics(font);
-
        double x = x_;
 
-       // this is special code for the chapter layout. This is
-       // printed in an extra row and has a pagebreak at
-       // the top.
-       if (layout.counter == "chapter") {
-               double spacing_val = 1.0;
-               if (!pparams.spacing().isDefault()) {
-                       spacing_val = pparams.spacing().getValue();
-               } else {
-                       spacing_val = bparams.spacing().getValue();
-               }
-
-               int const labeladdon = int(fm.maxHeight() * layout.spacing.getValue() * spacing_val);
-
-               int const maxdesc = int(fm.maxDescent() * layout.spacing.getValue() * spacing_val)
-                       + int(layout.parsep) * defaultRowHeight();
-
-               if (is_rtl) {
-                       x = width_ - leftMargin() -
-                               fm.width(str);
-               }
-
-               pi_.pain.text(int(x), yo_ - maxdesc - labeladdon, str, font);
+       if (is_rtl) {
+               x = width_ - leftMargin()
+                       + fm.width(layout.labelsep);
        } else {
-               if (is_rtl) {
-                       x = width_ - leftMargin()
-                               + fm.width(layout.labelsep);
-               } else {
-                       x = x_ - fm.width(layout.labelsep)
-                               - fm.width(str);
-               }
-
-               pi_.pain.text(int(x), yo_, str, font);
+               x = x_ - fm.width(layout.labelsep)
+                       - fm.width(str);
        }
+
+       pi_.pain.text(int(x), yo_, str, font);
 }
 
 
@@ -682,7 +558,7 @@ void RowPainter::paintTopLevelLabel()
                + (layout.labelbottomsep * defaultRowHeight()));
 
        double x = x_;
-       if (layout.labeltype == LABEL_CENTERED_TOP_ENVIRONMENT) {
+       if (layout.labeltype == LABEL_CENTERED) {
                if (is_rtl)
                        x = leftMargin();
                x += (width_ - text_metrics_.rightMargin(pm_) - leftMargin()) / 2;
@@ -693,6 +569,7 @@ void RowPainter::paintTopLevelLabel()
        pi_.pain.text(int(x), yo_ - maxdesc - labeladdon, str, font);
 }
 
+
 /** Check if the current paragraph is the last paragraph in a
     proof environment */
 static int getEndLabel(pit_type p, Text const & text)
@@ -788,11 +665,10 @@ void RowPainter::paintLast()
        case END_LABEL_NO_LABEL:
                if (lyxrc.paragraph_markers && size_type(pit_ + 1) < pars_.size()) {
                        docstring const s = docstring(1, char_type(0x00B6));
-                       FontInfo f = FontInfo();
-                       FontMetrics const & fm = theFontMetrics(f);
+                       FontInfo f = FontInfo(text_.layoutFont(pit_));
                        f.setColor(Color_paragraphmarker);
                        pi_.pain.text(int(x_), yo_, s, f);
-                       x_ += fm.width(s);
+                       x_ += theFontMetrics(f).width(s);
                }
                break;
        }
@@ -898,13 +774,11 @@ void RowPainter::paintText()
                                font_span.last = inlineCompletionVPos;
                }
 
+               // Note that this value will only be used in
+               // situations where no ligature of composition of
+               // characters is needed. (see comments alginuses of width_pos).
                const int width_pos = pm_.singleWidth(pos, font);
 
-               if (x_ + width_pos < 0) {
-                       x_ += width_pos;
-                       ++vpos;
-                       continue;
-               }
                Change const & change = par_.lookupChange(pos);
                if (change.changed() && !change_running.changed()) {
                        change_running = change;
@@ -939,6 +813,7 @@ void RowPainter::paintText()
                        int const lwidth = theFontMetrics(labelFont())
                                .width(layout.labelsep);
 
+                       // width_pos is either the width of a space or an inset
                        x_ += row_.label_hfill + lwidth - width_pos;
                }
 
@@ -949,6 +824,7 @@ void RowPainter::paintText()
                if (par_.isSeparator(pos)) {
                        Font const orig_font = text_metrics_.displayFont(pit_, pos);
                        double const orig_x = x_;
+                       // width_pos is the width of a space
                        double separator_width = width_pos;
                        if (pos >= body_pos)
                                separator_width += row_.separator;