[PUSHED] Change in core[libreoffice-3-5]: fdo#49901: SwView::ExecSearch: fix Replace All cursor:

Gerrit gerrit at gerrit.libreoffice.org
Mon Aug 27 11:20:09 PDT 2012


>From Björn Michaelsen <bjoern.michaelsen at canonical.com>:

Björn Michaelsen has submitted this change and it was merged.

Change subject: fdo#49901: SwView::ExecSearch: fix Replace All cursor:
......................................................................


fdo#49901: SwView::ExecSearch: fix Replace All cursor:

Because the "fix" for i#8288 only set the point of the cursor and not
the mark, it can happen that after a Replace All weird things are
selected; fix that (and i#8288 properly) by using the SwCrsrShell stack,
and don't do it when searching in the selection.
This cannot be done in SwCrsrShell itself because ExecSearch sets the
cursor to the beginning/end of the document already.
(regression from cda75d6f358d438f7ed36ed7fa296ac23e3c4a04)

Change-Id: Ie9f8d7da2fa26000a0237c24f8742e47ffa29213
(cherry picked from commit b3233579ccb2528912835deedde37c1d48417d6c)
---
M sw/source/ui/uiview/viewsrch.cxx
1 file changed, 23 insertions(+), 23 deletions(-)

Approvals:
  Björn Michaelsen: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/482
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie9f8d7da2fa26000a0237c24f8742e47ffa29213
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-5
Gerrit-Owner: Michael Stahl <mstahl at redhat.com>
Gerrit-Reviewer: Björn Michaelsen <bjoern.michaelsen at canonical.com>



More information about the LibreOffice mailing list