diff options
author | Armin Le Grand <alg@apache.org> | 2012-08-09 08:42:27 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2013-05-28 15:55:23 +0100 |
commit | 12a4200e8ff7f045efcc7e9d15a24b15b248c437 (patch) | |
tree | 00363aedf07713ec9ff931d933b86a6b35e20267 /svx/source/svdraw/svdedxv.cxx | |
parent | 64b07d99aa0b31b3d04519afe6afe0cdf36bcd6f (diff) |
Related: #i120498# Enhanced Undo/Redo and user experience...
when editing texts in graphic objects and/or tables
(cherry picked from commit a096725779b210c7a2706e72596fd7c80e049fdd)
Conflicts:
editeng/inc/editeng/editeng.hxx
editeng/inc/editeng/editund2.hxx
editeng/inc/editeng/outliner.hxx
editeng/source/editeng/editundo.cxx
editeng/source/editeng/impedit.hxx
editeng/source/outliner/outliner.cxx
sd/source/core/undo/undomanager.cxx
sd/source/ui/view/drviewse.cxx
svx/Library_svxcore.mk
svx/Package_inc.mk
svx/inc/svx/svdedxv.hxx
svx/source/svdraw/svdedxv.cxx
Change-Id: I40e3ef2dff681f9b6f2f6b5d35507071f8110533
Comment unused variable pNewEditUndoManager to silence the compiler
(cherry picked from commit 568655083af7830e7b9edf56ef862ddf9a99003b)
Change-Id: Ib7179ee6c34ce03a75942978831c3a55968f161f
Removed unused variable
(cherry picked from commit 0bbde4414badfd40234de4a4c9f750194f5d1d5e)
Change-Id: I39e7f25426e8e7d1367102d603b0f6c84d96622f
Diffstat (limited to 'svx/source/svdraw/svdedxv.cxx')
-rw-r--r-- | svx/source/svdraw/svdedxv.cxx | 83 |
1 files changed, 81 insertions, 2 deletions
diff --git a/svx/source/svdraw/svdedxv.cxx b/svx/source/svdraw/svdedxv.cxx index 6134278eddb8..6a5e571269bc 100644 --- a/svx/source/svdraw/svdedxv.cxx +++ b/svx/source/svdraw/svdedxv.cxx @@ -63,6 +63,7 @@ #include <svtools/colorcfg.hxx> #include <vcl/svapp.hxx> #include <svx/sdrpaintwindow.hxx> +#include <svx/sdrundomanager.hxx> //////////////////////////////////////////////////////////////////////////////////////////////////// @@ -87,7 +88,8 @@ void SdrObjEditView::ImpClearVars() } SdrObjEditView::SdrObjEditView(SdrModel* pModel1, OutputDevice* pOut): - SdrGlueEditView(pModel1,pOut) + SdrGlueEditView(pModel1,pOut), + mpOldTextEditUndoManager(0) { ImpClearVars(); } @@ -95,8 +97,10 @@ SdrObjEditView::SdrObjEditView(SdrModel* pModel1, OutputDevice* pOut): SdrObjEditView::~SdrObjEditView() { pTextEditWin = NULL; // so there's no ShowCursor in SdrEndTextEdit - if (IsTextEdit()) SdrEndTextEdit(); + if (IsTextEdit()) + SdrEndTextEdit(); delete pTextEditOutliner; + delete mpOldTextEditUndoManager; } //////////////////////////////////////////////////////////////////////////////////////////////////// @@ -516,6 +520,12 @@ IMPL_LINK(SdrObjEditView,ImpOutlinerCalcFieldValueHdl,EditFieldInfo*,pFI) return 0; } +IMPL_LINK(SdrObjEditView, EndTextEditHdl, SdrUndoManager*, /*pUndoManager*/) +{ + SdrEndTextEdit(); + return 0; +} + sal_Bool SdrObjEditView::SdrBeginTextEdit( SdrObject* pObj, SdrPageView* pPV, Window* pWin, sal_Bool bIsNewObj, SdrOutliner* pGivenOutliner, @@ -742,6 +752,31 @@ sal_Bool SdrObjEditView::SdrBeginTextEdit( if( mxSelectionController.is() ) mxSelectionController->onSelectionHasChanged(); + if(IsUndoEnabled()) + { + SdrUndoManager* pSdrUndoManager = dynamic_cast< SdrUndoManager* >(GetModel()->GetSdrUndoManager()); + + if(pSdrUndoManager) + { + // we have an outliner, undo manager and it's an EditUndoManager, exchange + // the document undo manager and the default one from the outliner and tell + // it that text edit starts by setting a callback if it needs to end text edit mode. + if(mpOldTextEditUndoManager) + { + // should not happen, delete it + delete mpOldTextEditUndoManager; + mpOldTextEditUndoManager = 0; + } + + mpOldTextEditUndoManager = pTextEditOutliner->SetUndoManager(pSdrUndoManager); + pSdrUndoManager->SetEndTextEditHdl(LINK(this, SdrObjEditView, EndTextEditHdl)); + } + else + { + OSL_ENSURE(false, "The document undo manager is not derived from SdrUndoManager (!)"); + } + } + return sal_True; // ran fine, let TextEdit run now } else @@ -795,6 +830,42 @@ SdrEndTextEditKind SdrObjEditView::SdrEndTextEdit(sal_Bool bDontDeleteReally) SdrOutliner* pTEOutliner =pTextEditOutliner; OutlinerView* pTEOutlinerView=pTextEditOutlinerView; Cursor* pTECursorMerker=pTextEditCursorMerker; + SdrUndoManager* pExtraUndoEditUndoManager = 0; + + if(IsUndoEnabled() && GetModel() && pTEObj && pTEOutliner) + { + // change back the UndoManager to the remembered original one + ::svl::IUndoManager* pOriginal = pTEOutliner->SetUndoManager(mpOldTextEditUndoManager); + mpOldTextEditUndoManager = 0; + + if(pOriginal) + { + // check if we got back our document undo manager + SdrUndoManager* pSdrUndoManager = dynamic_cast< SdrUndoManager* >(GetModel()->GetSdrUndoManager()); + + if(pSdrUndoManager && dynamic_cast< SdrUndoManager* >(pOriginal) == pSdrUndoManager) + { + // We are ending text edit; execute all redos to create a complete text change + // undo action for the redo buffer. Also mark this state when at least one redo was + // executed; the created TextChange needs to be undone plus the first real undo + // outside the text edit changes + while(pSdrUndoManager->GetRedoActionCount()) + { + pExtraUndoEditUndoManager = pSdrUndoManager; + pSdrUndoManager->Redo(); + } + + // reset the callback link and let the undo manager cleanup all text edit + // undo actions to get the stack back to the form before the text edit + pSdrUndoManager->SetEndTextEditHdl(Link()); + } + else + { + OSL_ENSURE(false, "´Got UndoManager back in SdrEndTextEdit which is NOT the expected document UndoManager (!)"); + delete pOriginal; + } + } + } if( GetModel() && mxTextEditObj.is() ) { @@ -988,6 +1059,14 @@ SdrEndTextEditKind SdrObjEditView::SdrEndTextEdit(sal_Bool bDontDeleteReally) ((SfxBroadcaster*)pTEObj->GetBroadcaster())->Broadcast(aHint); } + if(pExtraUndoEditUndoManager) + { + // undo the text edit action since it was created as part of a EndTextEdit + // callback from undo itself. This needs to be done after the call to + // FmFormView::SdrEndTextEdit since it gets created there + pExtraUndoEditUndoManager->Undo(); + } + return eRet; } |