fdo#73151 Make better use of the sidebar

The general idea is to encourage use of the sidebar instead of floating windows (for Navigator, Styles&Formatting and Gallery)

Changes:
* Show the sidebar by default in Writer & Impress
* Remove the Gallery floating window (Gallery now always opens in the sidebar)
* Remove all Gallery, Navigator and Styles&Formatting links from the default toolbar in Writer
(since they have an icon in the sidebar and the sidebar is shown by default now)
* When selecting "More" from the Styles dropdown, the Stylelist opens in the sidebar instead of the floating window

This has all been discussed extensively here: https://bugs.freedesktop.org/show_bug.cgi?id=73151

Change-Id: I3a0461d1472711da7121801000af294b432fccb1
Reviewed-on: https://gerrit.libreoffice.org/11077
Reviewed-by: Thomas Arnhold <thomas@arnhold.org>
Tested-by: Thomas Arnhold <thomas@arnhold.org>

7 files changed, 25 insertions, 48 deletions

diff --git a/sd/source/ui/app/sddll.cxx b/sd/source/ui/app/sddll.cxx
index 69ec0aa530bb..42c3d7840808 100644
--- a/sd/source/ui/app/sddll.cxx
+++ b/sd/source/ui/app/sddll.cxx
@@ -199,7 +199,6 @@ void SdDLL::RegisterControllers()
     SvxColorChildWindow::RegisterChildWindow(false, pMod, SFX_CHILDWIN_TASK);
     SvxSearchDialogWrapper::RegisterChildWindow(false, pMod);
     SvxBmpMaskChildWindow::RegisterChildWindow(false, pMod);
-    GalleryChildWindow::RegisterChildWindow(false, pMod);
     SvxIMapDlgChildWindow::RegisterChildWindow(false, pMod);
     SvxHlinkDlgWrapper::RegisterChildWindow(false, pMod);
     ::sd::SpellDialogChildWindow::RegisterChildWindow(false, pMod);
diff --git a/sd/source/ui/inc/DrawViewShell.hxx b/sd/source/ui/inc/DrawViewShell.hxx
index 37c2ec342ac9..15add1df9891 100644
--- a/sd/source/ui/inc/DrawViewShell.hxx
+++ b/sd/source/ui/inc/DrawViewShell.hxx
@@ -211,7 +211,6 @@ public:
     void            AssignFrom3DWindow();
 
     void            ExecGallery(SfxRequest& rReq);
-    void            GetGalleryState(SfxItemSet& rSet);
 
     void            ExecBmpMask( SfxRequest& rReq );
     void            GetBmpMaskState( SfxItemSet& rSet );
diff --git a/sd/source/ui/slideshow/slideshowimpl.cxx b/sd/source/ui/slideshow/slideshowimpl.cxx
index 606f804f2d8d..cdca55974b51 100644
--- a/sd/source/ui/slideshow/slideshowimpl.cxx
+++ b/sd/source/ui/slideshow/slideshowimpl.cxx
@@ -2484,7 +2484,6 @@ FncGetChildWindowId aShowChildren[] =
     &SvxIMapDlgChildWindow::GetChildWindowId,
     &SvxHlinkDlgWrapper::GetChildWindowId,
     &SfxTemplateDialogWrapper::GetChildWindowId,
-    &GalleryChildWindow::GetChildWindowId,
     &SfxInfoBarContainerChild::GetChildWindowId
 };
 
diff --git a/sd/source/ui/view/drviews2.cxx b/sd/source/ui/view/drviews2.cxx
index 657df96f7066..54765d6500c7 100644
--- a/sd/source/ui/view/drviews2.cxx
+++ b/sd/source/ui/view/drviews2.cxx
@@ -178,6 +178,7 @@
 namespace {
     const char CustomAnimationPanelId[] = "CustomAnimationPanel";
     const char SlideTransitionPanelId[] = "SlideTransitionPanel";
+    const char GalleryPanelId[] = "GalleryPanel";
 }
 
 using namespace ::com::sun::star;
@@ -2721,16 +2722,6 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
         }
         break;
 
-        case SID_GALLERY:
-        {
-            GetViewFrame()->ToggleChildWindow( GalleryChildWindow::GetChildWindowId() );
-            GetViewFrame()->GetBindings().Invalidate( SID_GALLERY );
-
-            Cancel();
-            rReq.Ignore ();
-        }
-        break;
-
         case SID_NAVIGATOR:
         {
             if ( rReq.GetArgs() )
@@ -2746,6 +2737,30 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
         }
         break;
 
+        case SID_SLIDE_TRANSITIONS_PANEL:
+        case SID_CUSTOM_ANIMATION_PANEL:
+        case SID_GALLERY:
+        {
+            // First make sure that the sidebar is visible
+            GetViewFrame()->ShowChildWindow(SID_SIDEBAR);
+
+            OUString panelId;
+            if (nSId == SID_CUSTOM_ANIMATION_PANEL)
+                panelId = CustomAnimationPanelId;
+            else if (nSId == SID_GALLERY)
+                panelId = GalleryPanelId;
+            else if (nSId == SID_SLIDE_TRANSITIONS_PANEL)
+                panelId = SlideTransitionPanelId;
+
+            ::sfx2::sidebar::Sidebar::ShowPanel(
+                panelId,
+                GetViewFrame()->GetFrame().GetFrameInterface());
+
+            Cancel();
+            rReq.Done();
+        }
+        break;
+
         case SID_ANIMATION_OBJECTS:
         {
             if ( rReq.GetArgs() )
@@ -2763,30 +2778,6 @@ void DrawViewShell::FuTemporary(SfxRequest& rReq)
         }
         break;
 
-        case SID_CUSTOM_ANIMATION_PANEL:
-        {
-            // Make the slide transition panel visible in the sidebar.
-            ::sfx2::sidebar::Sidebar::ShowPanel(
-                CustomAnimationPanelId,
-                GetViewFrame()->GetFrame().GetFrameInterface());
-
-            Cancel();
-            rReq.Done ();
-        }
-        break;
-
-        case SID_SLIDE_TRANSITIONS_PANEL:
-        {
-            // Make the slide transition panel visible in the sidebar.
-            ::sfx2::sidebar::Sidebar::ShowPanel(
-                SlideTransitionPanelId,
-                GetViewFrame()->GetFrame().GetFrameInterface());
-
-            Cancel();
-            rReq.Done ();
-        }
-        break;
-
         case SID_3D_WIN:
         {
             if ( rReq.GetArgs() )
diff --git a/sd/source/ui/view/drviews6.cxx b/sd/source/ui/view/drviews6.cxx
index 19e141367553..093be5c9fddf 100644
--- a/sd/source/ui/view/drviews6.cxx
+++ b/sd/source/ui/view/drviews6.cxx
@@ -244,11 +244,6 @@ void DrawViewShell::SetChildWindowState( SfxItemSet& rSet )
         sal_uInt16 nId = SvxBmpMaskChildWindow::GetChildWindowId();
         rSet.Put( SfxBoolItem( SID_BMPMASK, GetViewFrame()->HasChildWindow( nId ) ) );
     }
-    if( SFX_ITEM_AVAILABLE == rSet.GetItemState( SID_GALLERY ) )
-    {
-        sal_uInt16 nId = GalleryChildWindow::GetChildWindowId();
-        rSet.Put( SfxBoolItem( SID_GALLERY, GetViewFrame()->HasChildWindow( nId ) ) );
-    }
     if( SFX_ITEM_AVAILABLE == rSet.GetItemState( SID_IMAP ) )
     {
         sal_uInt16 nId = SvxIMapDlgChildWindow::GetChildWindowId();
diff --git a/sd/source/ui/view/drviews9.cxx b/sd/source/ui/view/drviews9.cxx
index 11912682007d..027cf54b25ca 100644
--- a/sd/source/ui/view/drviews9.cxx
+++ b/sd/source/ui/view/drviews9.cxx
@@ -181,10 +181,6 @@ void DrawViewShell::ExecGallery(SfxRequest& rReq)
     GetDocSh()->SetWaitCursor( false );
 }
 
-void DrawViewShell::GetGalleryState(SfxItemSet& )
-{
-}
-
 /**
  * Edit macros for attribute configuration
  */
diff --git a/sd/source/ui/view/drvwshrg.cxx b/sd/source/ui/view/drvwshrg.cxx
index 3df88208a44d..ea7c9d21ddde 100644
--- a/sd/source/ui/view/drvwshrg.cxx
+++ b/sd/source/ui/view/drvwshrg.cxx
@@ -76,7 +76,6 @@ void DrawViewShell::InitInterface_Impl()
     GetStaticInterface()->RegisterChildWindow(AnimationChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(Svx3DChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxBmpMaskChildWindow::GetChildWindowId());
-    GetStaticInterface()->RegisterChildWindow(GalleryChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxIMapDlgChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxHlinkDlgWrapper::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(::sd::SpellDialogChildWindow::GetChildWindowId());
@@ -102,7 +101,6 @@ void GraphicViewShell::InitInterface_Impl()
     GetStaticInterface()->RegisterChildWindow(SvxColorChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(Svx3DChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxBmpMaskChildWindow::GetChildWindowId());
-    GetStaticInterface()->RegisterChildWindow(GalleryChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxIMapDlgChildWindow::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(SvxHlinkDlgWrapper::GetChildWindowId());
     GetStaticInterface()->RegisterChildWindow(::sd::SpellDialogChildWindow::GetChildWindowId());