X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FRow.h;h=9039ff8860e9d40c2b52cd7b817de8bec8229f8b;hb=bc47054be88467f01fd2f6e437d555743bd8a7c9;hp=4f2bd744d4ecd4056950b34c5bf507233fa69b4b;hpb=6258cebb77d63928313cae14bbba5d7f092d0a5e;p=lyx.git diff --git a/src/Row.h b/src/Row.h index 4f2bd744d4..9039ff8860 100644 --- a/src/Row.h +++ b/src/Row.h @@ -29,6 +29,18 @@ namespace lyx { class DocIterator; class Inset; +/** + * FIXME: Change Row object to operate only on integers and not doubles. + * + * This use of double is only useful to distribute the extra + * horizontal space between separators in justified text. If we do + * integer arithmetic, then it is possible to have two groups of + * separators, with size s or s+1. Then strings can be drawn without + * cutting at separators in justfied text, as it is done in + * non-justified text. This will improve performance. + */ + + /** * An on-screen row of text. A paragraph is broken into a RowList for * display. Each Row contains a tokenized description of the contents @@ -63,19 +75,26 @@ public: : type(t), pos(p), endpos(p + 1), inset(0), extra(0), font(f), change(ch), final(false) {} - // returns total width of element, including separator overhead - double width() const { return dim.wid + extra; }; - // returns position in pixels (from the left) of position - // \param i in the row element + // Return total width of element, including separator overhead + double full_width() const { return dim.wid + extra; }; + /** Return position in pixels (from the left) of position + * \param i in the row element. + */ double pos2x(pos_type const i) const; - /** Return character position that is the closest to * pixel position \param x. The value \param x is - * rounded to the actual pixel position. If \param - * short is true, the pixel value is rounded by - * default. + * adjusted to the actual pixel position. */ - pos_type x2pos(double &x, bool low = false) const; + pos_type x2pos(int &x) const; + /** Break the element if possible, so that its width is + * less then \param w. Returns true on success. + */ + bool breakAt(int w); + + // Returns the position on left side of the element. + pos_type left_pos() const; + // Returns the position on right side of the element. + pos_type right_pos() const; // The kind of row element Type type; @@ -146,7 +165,7 @@ public: Dimension & dimension() { return dim_; } /// int height() const { return dim_.height(); } - /// + /// The width of the row, including the left margin, but not the right one. int width() const { return dim_.wid; } /// int ascent() const { return dim_.asc; } @@ -186,6 +205,10 @@ public: /// bool empty() const { return elements_.empty(); } /// + Element & front() { return elements_.front(); } + /// + Element const & front() const { return elements_.front(); } + /// Element & back() { return elements_.back(); } /// Element const & back() const { return elements_.back(); } @@ -196,11 +219,11 @@ public: /** * if row width is too large, remove all elements after last * separator and update endpos if necessary. If all that - * rename is a large word, cut it to \param width. + * remains is a large word, cut it to \param width. * \param body_pos minimum amount of text to keep. * \param width maximum width of the row */ - void shorten_if_needed(pos_type const body_pos, int const width); + void shortenIfNeeded(pos_type const body_pos, int const width); /** * If last element of the row is a string, compute its width @@ -212,7 +235,7 @@ public: * Find sequences of right-to-left elements and reverse them. * This should be called once the row is completely built. */ - void reverseRTL(); + void reverseRTL(bool rtl_par); friend std::ostream & operator<<(std::ostream & os, Row const & row); @@ -220,8 +243,8 @@ public: double separator; /// width of hfills in the label double label_hfill; - /// the x position of the row (left margin) - double x; + /// the left margin position of the row + int left_margin; /// the right margin of the row int right_margin; ///