]> git.lyx.org Git - lyx.git/blobdiff - src/bufferview_funcs.cpp
RTL bugfix: inside texted, use ParagraphMetrics::insetDimension() instead of Inset...
[lyx.git] / src / bufferview_funcs.cpp
index 2dda2498e4da73079cf974ad6046acc4a3bbcc24..3f132de962d585d3c1366832038f27a22a5988cc 100644 (file)
@@ -16,7 +16,6 @@
 
 #include "bufferview_funcs.h"
 
-#include "Author.h"
 #include "Buffer.h"
 #include "BufferParams.h"
 #include "BufferView.h"
 #include "CoordCache.h"
 #include "gettext.h"
 #include "Language.h"
-#include "LColor.h"
+#include "Color.h"
 #include "Lexer.h"
-#include "Row.h"
-#include "Paragraph.h"
-#include "ParagraphParameters.h"
-#include "ParIterator.h"
 
-#include "frontends/Alert.h"
+#include "frontends/alert.h"
 
 #include "insets/InsetCommand.h"
 #include "insets/InsetText.h"
 
 #include <sstream>
 
-
-namespace lyx {
-
-using support::bformat;
-
 using std::istringstream;
 using std::ostringstream;
 using std::string;
@@ -52,11 +42,15 @@ using std::vector;
 using std::find;
 
 
+namespace lyx {
+
+using support::bformat;
+
 namespace bv_funcs {
 
 // Set data using font and toggle
 // If successful, returns true
-bool font2string(LyXFont const & font, bool const toggle, string & data)
+bool font2string(Font const & font, bool const toggle, string & data)
 {
        string lang = "ignore";
        if (font.language())
@@ -81,7 +75,7 @@ bool font2string(LyXFont const & font, bool const toggle, string & data)
 
 // Set font and toggle using data
 // If successful, returns true
-bool string2font(string const & data, LyXFont & font, bool & toggle)
+bool string2font(string const & data, Font & font, bool & toggle)
 {
        istringstream is(data);
        Lexer lex(0,0);
@@ -98,26 +92,26 @@ bool string2font(string const & data, LyXFont & font, bool & toggle)
 
                if (token == "family") {
                        int const next = lex.getInteger();
-                       font.setFamily(LyXFont::FONT_FAMILY(next));
+                       font.setFamily(Font::FONT_FAMILY(next));
 
                } else if (token == "series") {
                        int const next = lex.getInteger();
-                       font.setSeries(LyXFont::FONT_SERIES(next));
+                       font.setSeries(Font::FONT_SERIES(next));
 
                } else if (token == "shape") {
                        int const next = lex.getInteger();
-                       font.setShape(LyXFont::FONT_SHAPE(next));
+                       font.setShape(Font::FONT_SHAPE(next));
 
                } else if (token == "size") {
                        int const next = lex.getInteger();
-                       font.setSize(LyXFont::FONT_SIZE(next));
+                       font.setSize(Font::FONT_SIZE(next));
 
                } else if (token == "emph" || token == "underbar" ||
                           token == "noun" || token == "number") {
 
                        int const next = lex.getInteger();
-                       LyXFont::FONT_MISC_STATE const misc =
-                               LyXFont::FONT_MISC_STATE(next);
+                       Font::FONT_MISC_STATE const misc =
+                               Font::FONT_MISC_STATE(next);
 
                        if (token == "emph")
                            font.setEmph(misc);
@@ -130,7 +124,7 @@ bool string2font(string const & data, LyXFont & font, bool & toggle)
 
                } else if (token == "color") {
                        int const next = lex.getInteger();
-                       font.setColor(LColor::color(next));
+                       font.setColor(Color::color(next));
 
                } else if (token == "language") {
                        string const next = lex.getString();
@@ -161,25 +155,51 @@ Point coordOffset(BufferView const & bv, DocIterator const & dit,
 {
        int x = 0;
        int y = 0;
+       int lastw = 0;
 
-       // Contribution of nested insets
-       for (size_t i = 1; i != dit.depth(); ++i) {
+       // Addup ontribution of nested insets, from inside to outside,
+       // keeping the outer paragraph for a special handling below
+       for (size_t i = dit.depth() - 1; i >= 1; --i) {
                CursorSlice const & sl = dit[i];
                int xx = 0;
                int yy = 0;
+               
+               // get relative position inside sl.inset()
                sl.inset().cursorPos(bv, sl, boundary && ((i+1) == dit.depth()), xx, yy);
+               
+               // Make relative position inside of the edited inset relative to sl.inset()
                x += xx;
                y += yy;
+               
+               // In case of an RTL inset, the edited inset will be positioned to the left
+               // of xx:yy
+               if (sl.text()) {
+                       bool boundary_i = boundary && i + 1 == dit.depth();
+                       bool rtl = bv.textMetrics(sl.text()).isRTL(sl, boundary_i);
+                       if (rtl)
+                               x -= lastw;
+                       // remember width for the case that sl.inset() is positioned in an RTL inset
+                       Dimension const & dim = bv.parMetrics(sl.text(), sl.pit()).
+                               insetDimension(&sl.inset());
+                       lastw = dim.wid;
+               } else {
+                       // remember width for the case that sl.inset() is positioned in an RTL inset
+                       Dimension const dim = sl.inset().dimension(bv);
+                       lastw = dim.wid;
+               }
+               
                //lyxerr << "Cursor::getPos, i: "
                // << i << " x: " << xx << " y: " << y << endl;
        }
 
        // Add contribution of initial rows of outermost paragraph
        CursorSlice const & sl = dit[0];
-       ParagraphMetrics const & pm = bv.parMetrics(sl.text(), sl.pit());
+       TextMetrics const & tm = bv.textMetrics(sl.text());
+       ParagraphMetrics const & pm = tm.parMetrics(sl.pit());
        BOOST_ASSERT(!pm.rows().empty());
        y -= pm.rows()[0].ascent();
 #if 1
+       // FIXME: document this mess
        size_t rend;
        if (sl.pos() > 0 && dit.depth() == 1) {
                int pos = sl.pos();
@@ -195,8 +215,20 @@ Point coordOffset(BufferView const & bv, DocIterator const & dit,
        for (size_t rit = 0; rit != rend; ++rit)
                y += pm.rows()[rit].height();
        y += pm.rows()[rend].ascent();
-       x += dit.bottom().text()->cursorX(bv, dit.bottom(), boundary && dit.depth() == 1);
-
+       
+       TextMetrics const & bottom_tm = bv.textMetrics(dit.bottom().text());
+       
+       // Make relative position from the nested inset now bufferview absolute.
+       int xx = bottom_tm.cursorX(dit.bottom(), boundary && dit.depth() == 1);
+       x += xx;
+       
+       // In the RTL case place the nested inset at the left of the cursor in 
+       // the outer paragraph
+       bool boundary_1 = boundary && 1 == dit.depth();
+       bool rtl = bottom_tm.isRTL(dit.bottom(), boundary_1);
+       if (rtl)
+               x -= lastw;
+       
        return Point(x, y);
 }
 
@@ -204,19 +236,12 @@ Point coordOffset(BufferView const & bv, DocIterator const & dit,
 Point getPos(BufferView const & bv, DocIterator const & dit, bool boundary)
 {
        CursorSlice const & bot = dit.bottom();
-       CoordCache::ParPosCache::const_iterator cache_it = 
-               bv.coordCache().getParPos().find(bot.text());
-       if (cache_it == bv.coordCache().getParPos().end())
+       TextMetrics const & tm = bv.textMetrics(bot.text());
+       if (!tm.has(bot.pit()))
                return Point(-1, -1);
 
-       CoordCache::InnerParPosCache const & cache = cache_it->second;
-       CoordCache::InnerParPosCache::const_iterator it = cache.find(bot.pit());
-       if (it == cache.end()) {
-               //lyxerr << "cursor out of view" << std::endl;
-               return Point(-1, -1);
-       }
        Point p = coordOffset(bv, dit, boundary); // offset from outer paragraph
-       p.y_ += it->second.y_;
+       p.y_ += tm.parMetrics(bot.pit()).position();
        return p;
 }
 
@@ -225,31 +250,31 @@ Point getPos(BufferView const & bv, DocIterator const & dit, bool boundary)
 // FIXME: This does not work within mathed!
 CurStatus status(BufferView const * bv, DocIterator const & dit)
 {
-       CoordCache::InnerParPosCache const & cache =
-               bv->coordCache().getParPos().find(dit.bottom().text())->second;
-
-       if (cache.find(dit.bottom().pit()) != cache.end())
-               return CUR_INSIDE;
-       else if (dit.bottom().pit() < bv->anchor_ref())
+       // FIXME: it's be better to have something like TextMetrics::status().
+       TextMetrics const & tm = bv->textMetrics(dit.bottom().text());
+       int par_pos = tm.parPosition(dit.bottom().pit());
+       if (par_pos < 0)
                return CUR_ABOVE;
-       else
+       else if (par_pos > bv->workHeight())
                return CUR_BELOW;
+               
+       return CUR_INSIDE;
 }
 
 namespace {
 
 bool findNextInset(DocIterator & dit,
-                  vector<InsetBase_code> const & codes,
+                  vector<Inset_code> const & codes,
                   string const & contents)
 {
        DocIterator tmpdit = dit;
 
        while (tmpdit) {
-               InsetBase const * inset = tmpdit.nextInset();
+               Inset const * inset = tmpdit.nextInset();
                if (inset
                    && find(codes.begin(), codes.end(), inset->lyxCode()) != codes.end()
                    && (contents.empty() ||
-                       static_cast<InsetCommand const *>(inset)->getContents() == contents)) {
+                   static_cast<InsetCommand const *>(inset)->getContents() == contents)) {
                        dit = tmpdit;
                        return true;
                }
@@ -262,7 +287,7 @@ bool findNextInset(DocIterator & dit,
 } // namespace anon
 
 
-bool findInset(DocIterator & dit, vector<InsetBase_code> const & codes,
+bool findInset(DocIterator & dit, vector<Inset_code> const & codes,
               bool same_content)
 {
        string contents;
@@ -272,7 +297,7 @@ bool findInset(DocIterator & dit, vector<InsetBase_code> const & codes,
                return false;
 
        if (same_content) {
-               InsetBase const * inset = tmpdit.nextInset();
+               Inset const * inset = tmpdit.nextInset();
                if (inset
                    && find(codes.begin(), codes.end(), inset->lyxCode()) != codes.end()) {
                        contents = static_cast<InsetCommand const *>(inset)->getContents();
@@ -294,13 +319,13 @@ bool findInset(DocIterator & dit, vector<InsetBase_code> const & codes,
 }
 
 
-void findInset(DocIterator & dit, InsetBase_code code, bool same_content)
+void findInset(DocIterator & dit, Inset_code code, bool same_content)
 {
-       findInset(dit, vector<InsetBase_code>(1, code), same_content);
+       findInset(dit, vector<Inset_code>(1, code), same_content);
 }
 
 
-void gotoInset(BufferView * bv, vector<InsetBase_code> const & codes,
+void gotoInset(BufferView * bv, vector<Inset_code> const & codes,
               bool same_content)
 {
        Cursor tmpcur = bv->cursor();
@@ -314,9 +339,9 @@ void gotoInset(BufferView * bv, vector<InsetBase_code> const & codes,
 }
 
 
-void gotoInset(BufferView * bv, InsetBase_code code, bool same_content)
+void gotoInset(BufferView * bv, Inset_code code, bool same_content)
 {
-       gotoInset(bv, vector<InsetBase_code>(1, code), same_content);
+       gotoInset(bv, vector<Inset_code>(1, code), same_content);
 }