anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-11-22 18:08:10 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix crash in table selection

Browse Source 
git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@9735 a592a061-630c-0410-9148-cb99ea01b6c8
This commit is contained in:
Alfredo Braunstein 2005-03-21 18:21:13 +00:00
parent 33f008d7a1
commit c605824a66
3 changed files with 37 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/insets/ChangeLog
View File

@ -1,3 +1,10 @@
2005-03-21 Alfredo Braunstein <abraunst@lyx.org>
* insettabular.[Ch]: rename setPos -> setCursorFromCoordinates and
make it *not* descend into insets (fix a crash). Move the part
needed from editXY to a new function getNearestCell
2005-02-22 Jean-Marc Lasgouttes <lasgouttes@lyx.org>
* insetinclude.C (InsetInclude): when constructing from another

44
src/insets/insettabular.C
View File

@ -426,16 +426,6 @@ void InsetTabular::edit(LCursor & cur, bool left)
}
InsetBase * InsetTabular::editXY(LCursor & cur, int x, int y) const
{
//lyxerr << "InsetTabular::editXY: " << this << endl;
cur.selection() = false;
cur.push(const_cast<InsetTabular&>(*this));
return setPos(cur, x, y);
//int xx = cursorx_ - xo() + tabular.getBeginningOfTextInCell(cur.idx());
}
void InsetTabular::doDispatch(LCursor & cur, FuncRequest & cmd)
{
lyxerr << "# InsetTabular::dispatch: cmd: " << cmd << endl;
@ -450,7 +440,7 @@ void InsetTabular::doDispatch(LCursor & cur, FuncRequest & cmd)
if (cmd.button() == mouse_button::button1) {
cur.selection() = false;
setPos(cur, cmd.x, cmd.y);
setCursorFromCoordinates(cur, cmd.x, cmd.y);
cur.resetAnchor();
bvcur = cur;
break;
@ -474,7 +464,7 @@ void InsetTabular::doDispatch(LCursor & cur, FuncRequest & cmd)
if (cmd.button() == mouse_button::button1) {
// only accept motions to places not deeper nested than the real anchor
if (bvcur.anchor_.hasPart(cur)) {
setPos(cur, cmd.x, cmd.y);
setCursorFromCoordinates(cur, cmd.x, cmd.y);
bvcur.setCursor(cur);
bvcur.selection() = true;
} else
@ -1136,9 +1126,30 @@ int dist(InsetOld const & inset, int x, int y)
} //namespace anon
InsetBase * InsetTabular::setPos(LCursor & cur, int x, int y) const
InsetBase * InsetTabular::editXY(LCursor & cur, int x, int y) const
{
lyxerr << "# InsetTabular::setPos() x=" << x << " y=" << y << endl;
//lyxerr << "InsetTabular::editXY: " << this << endl;
cur.selection() = false;
cur.push(const_cast<InsetTabular&>(*this));
cur.idx() = getNearestCell(x, y);
resetPos(cur);
return cell(cur.idx())->text_.editXY(cur, x, y);
//int xx = cursorx_ - xo() + tabular.getBeginningOfTextInCell(cur.idx());
}
void InsetTabular::setCursorFromCoordinates(LCursor & cur, int x, int y) const
{
//lyxerr << "# InsetTabular::setCursorFromCoordinates()\n" << cur << endl;
cur.idx() = getNearestCell(x, y);
resetPos(cur);
return cell(cur.idx())->text_.setCursorFromCoordinates(cur, x, y);
}
InsetTabular::idx_type InsetTabular::getNearestCell(int x, int y) const
{
lyxerr << "# InsetTabular::getNearestCell() x=" << x << " y=" << y << endl;
idx_type idx_min = 0;
int dist_min = std::numeric_limits<int>::max();
for (idx_type i = 0; i < nargs(); ++i) {
@ -1150,10 +1161,7 @@ InsetBase * InsetTabular::setPos(LCursor & cur, int x, int y) const
}
}
}
cur.idx() = idx_min;
//lyxerr << "# InsetTabular::setPos()\n" << cur << endl;
resetPos(cur);
return cell(cur.idx())->text_.editXY(cur, x, y);
return idx_min;
}

5
src/insets/insettabular.h
View File

@ -153,7 +153,7 @@ private:
void drawCellLines(Painter &, int x, int y, row_type row,
idx_type cell) const;
///
InsetBase * setPos(LCursor & cur, int x, int y) const;
void setCursorFromCoordinates(LCursor & cur, int x, int y) const;
///
void moveNextCell(LCursor & cur);
@ -183,6 +183,9 @@ private:
/// are we operating on several cells?
bool tablemode(LCursor & cur) const;
/// return the cell nearest to x, y
idx_type getNearestCell(int x, int y) const;
///
Buffer const * buffer_;
///