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

Gerrit gerrit at gerrit.libreoffice.org
Mon Aug 27 00:22:15 PDT 2012


>From Miklos Vajna <vmiklos at suse.cz>:

Miklos Vajna 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:
  Miklos Vajna: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/481
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-6
Gerrit-Owner: Michael Stahl <mstahl at redhat.com>
Gerrit-Reviewer: Miklos Vajna <vmiklos at suse.cz>



More information about the LibreOffice mailing list