Kaydet (Commit) 6c5839d3 authored tarafından Mike Kaganski's avatar Mike Kaganski

tdf#117332: select replaced text to properly replace it next time

... othervise the next drag event will insert the new range immediately
after the just inserted, resulting in invalid reference.

Regression after commit 42aece94

  don't replace content of formula field when selecting range, tdf#58635

Change-Id: I7dfdc1ce041ebadf4406dbb605676d87cd489efa
Reviewed-on: https://gerrit.libreoffice.org/64574Reviewed-by: 's avatarMike Kaganski <mike.kaganski@collabora.com>
Tested-by: 's avatarMike Kaganski <mike.kaganski@collabora.com>
üst 63ff8602
...@@ -674,7 +674,13 @@ void ScCondFormatDlg::SetReference(const ScRange& rRef, ScDocument*) ...@@ -674,7 +674,13 @@ void ScCondFormatDlg::SetReference(const ScRange& rRef, ScDocument*)
OUString aRefStr(rRef.Format(nFlags, mpViewData->GetDocument(), OUString aRefStr(rRef.Format(nFlags, mpViewData->GetDocument(),
ScAddress::Details(mpViewData->GetDocument()->GetAddressConvention(), 0, 0))); ScAddress::Details(mpViewData->GetDocument()->GetAddressConvention(), 0, 0)));
if (pEdit != mpEdRange) if (pEdit != mpEdRange)
{
Selection sel = pEdit->GetSelection();
sel.Justify(); // in case of RtL selection
sel.Max() = sel.Min() + aRefStr.getLength();
pEdit->ReplaceSelected(aRefStr); pEdit->ReplaceSelected(aRefStr);
pEdit->SetSelection(sel); // to replace it again with next drag event
}
else else
pEdit->SetRefString( aRefStr ); pEdit->SetRefString( aRefStr );
updateTitle(); updateTitle();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment