sd/source/ui/inc/ViewShellManager.hxx | 4 ++++ sd/source/ui/view/ViewShellManager.cxx | 24 +++++++++++++++++++----- sd/source/ui/view/drviews1.cxx | 9 +++++++++ sd/source/ui/view/outlview.cxx | 11 ++++++++++- 4 files changed, 42 insertions(+), 6 deletions(-)
New commits: commit fae7fed54206b98cc3aac24abcdac384d3bd9437 Author: Marco Cecchetti <mrcek...@gmail.com> Date: Wed Nov 5 20:15:32 2014 +0100 fix bnc#624546 (fdo#83733) slide pane cut/copy/paste in outline view Problem: When I'm in outline mode, if I select a slide in the left slide preview pane, ctrl-c, then select another slide I would like it after and hit ctrl-v; it refuses to cut/paste. Analysis: This issue is due to the fact that the outline view always grabs focus when activated and a view is activated whenever is pushed to the sfx shell stack even if it is not the new top-most active view shell (see `ViewShellManager`, `SfxViewShell`, `SfxDispacther`). Solution: Make the `OutlineViewShell` grabs focus only if it is the top-most active view shell. Rationale: When `OutlineViewShell::Activate` is invoked, instead of removing the focus grabbing action completely, we check if the `OutlineViewShell` instance is the the top-most view shell and perform the focus grabbing action only in such a case. This change required to have also the `DrawViewShell` grabbing focus on activation (only when it is the top-most view shell). In order to implement this solution I needed to add a new method (and data member) to the `ViewShellManager` class. I named it `GetTopViewShell`. This method returns a pointer to the top-most active view shell of the internal stack. The returned pointer is updated in the `UpdateShellStack` method, before the sfx shell stack is updated. For more details see : https://gist.github.com/mcecchetti/15b3ebc505d6582ea0db (cherry picked from commit 967a386bccb15b99915a1e878e42450fbe9a2d0e) Signed-off-by: Andras Timar <andras.ti...@collabora.com> Conflicts: sd/source/ui/view/ViewShellManager.cxx Change-Id: I619a406864f50f0e62dee3fcb9ac5d46e3d48272 diff --git a/sd/source/ui/inc/ViewShellManager.hxx b/sd/source/ui/inc/ViewShellManager.hxx index 73419c9..6d7baf3 100644 --- a/sd/source/ui/inc/ViewShellManager.hxx +++ b/sd/source/ui/inc/ViewShellManager.hxx @@ -164,6 +164,10 @@ public: */ SfxShell* GetTopShell (void) const; + /** Return the top-most active view shell on the internal shell stack. + */ + SfxShell* GetTopViewShell (void) const; + /** Use this class to safely lock updates of the view shell stack. */ class UpdateLock diff --git a/sd/source/ui/view/ViewShellManager.cxx b/sd/source/ui/view/ViewShellManager.cxx index 3be7dd0..78b8b8b 100644 --- a/sd/source/ui/view/ViewShellManager.cxx +++ b/sd/source/ui/view/ViewShellManager.cxx @@ -126,6 +126,7 @@ public: void MoveToTop (const SfxShell& rParentShell); SfxShell* GetShell (ShellId nId) const; SfxShell* GetTopShell (void) const; + SfxShell* GetTopViewShell (void) const; void Shutdown (void); void InvalidateAllSubShells (const SfxShell* pParentShell); @@ -203,6 +204,8 @@ private: bool mbFormShellAboveParent; SfxShell* mpTopShell; + SfxShell* mpTopViewShell; + void UpdateShellStack (void); @@ -384,8 +387,13 @@ SfxShell* ViewShellManager::GetTopShell (void) const return NULL; } - - +SfxShell* ViewShellManager::GetTopViewShell (void) const +{ + if (mbValid) + return mpImpl->GetTopViewShell(); + else + return NULL; +} void ViewShellManager::Shutdown (void) { @@ -429,7 +437,8 @@ ViewShellManager::Implementation::Implementation ( mpFormShell(NULL), mpFormShellParent(NULL), mbFormShellAboveParent(true), - mpTopShell(NULL) + mpTopShell(NULL), + mpTopViewShell(NULL) { (void)rManager; } @@ -813,8 +822,10 @@ SfxShell* ViewShellManager::Implementation::GetTopShell (void) const return mpTopShell; } - - +SfxShell* ViewShellManager::Implementation::GetTopViewShell (void) const +{ + return mpTopViewShell; +} void ViewShellManager::Implementation::LockUpdate (void) { @@ -865,6 +876,9 @@ void ViewShellManager::Implementation::UpdateShellStack (void) // 1. Create the missing shells. CreateShells(); + // Update the pointer to the top-most active view shell. + mpTopViewShell = maActiveViewShells.begin()->mpShell; + // 2. Create the internal target stack. ShellStack aTargetStack; diff --git a/sd/source/ui/view/drviews1.cxx b/sd/source/ui/view/drviews1.cxx index 6ceeca8..de25ee4 100644 --- a/sd/source/ui/view/drviews1.cxx +++ b/sd/source/ui/view/drviews1.cxx @@ -91,6 +91,15 @@ namespace sd { void DrawViewShell::Activate(bool bIsMDIActivate) { ViewShell::Activate(bIsMDIActivate); + + // When the mode is switched to normal the main view shell grabs focus. + // This is done for getting cut/copy/paste commands on slides in the left + // pane (slide sorter view shell) to work properly. + SfxShell* pTopViewShell = this->GetViewShellBase().GetViewShellManager()->GetTopViewShell(); + if (pTopViewShell && pTopViewShell == this) + { + this->GetActiveWindow()->GrabFocus(); + } } void DrawViewShell::UIActivating( SfxInPlaceClient* pCli ) diff --git a/sd/source/ui/view/outlview.cxx b/sd/source/ui/view/outlview.cxx index 4436bdd..67aae83 100644 --- a/sd/source/ui/view/outlview.cxx +++ b/sd/source/ui/view/outlview.cxx @@ -64,6 +64,7 @@ #include "strings.hrc" #include "EventMultiplexer.hxx" #include "ViewShellBase.hxx" +#include "ViewShellManager.hxx" #include "undo/undoobjects.hxx" #include "undo/undomanager.hxx" #include "stlsheet.hxx" @@ -205,7 +206,15 @@ OutlineView::~OutlineView() void OutlineView::ConnectToApplication (void) { - mrOutlineViewShell.GetActiveWindow()->GrabFocus(); + // When the mode is switched to outline the main view shell grabs focus. + // This is done for getting cut/copy/paste commands on slides in the left + // pane (slide sorter view shell) to work properly. + SfxShell* pTopViewShell = mrOutlineViewShell.GetViewShellBase().GetViewShellManager()->GetTopViewShell(); + if (pTopViewShell && pTopViewShell == &mrOutlineViewShell) + { + mrOutlineViewShell.GetActiveWindow()->GrabFocus(); + } + Application::AddEventListener(LINK(this, OutlineView, AppEventListenerHdl)); } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits