chart2/source/controller/main/ChartController.cxx           |   26 -----------
 chart2/source/controller/main/ControllerCommandDispatch.cxx |   27 ------------
 chart2/uiconfig/menubar/menubar.xml                         |    1 
 3 files changed, 54 deletions(-)

New commits:
commit 543b6dc7668c8b5958b9adc010ab91855cc47a2a
Author:     Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Sun Jul 6 17:34:15 2025 +0500
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Sun Jul 6 17:20:28 2025 +0200

    Drop "Status Bar" element from Chart's View menu
    
    It does nothing, already in OOo 2.4.3.
    
    Change-Id: I2364426d9bdad6cfe959be79412c4a039757525b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/187445
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/chart2/source/controller/main/ChartController.cxx 
b/chart2/source/controller/main/ChartController.cxx
index c7620c1f7b6a..def6c2736caa 100644
--- a/chart2/source/controller/main/ChartController.cxx
+++ b/chart2/source/controller/main/ChartController.cxx
@@ -1400,31 +1400,6 @@ void SAL_CALL ChartController::dispatch(
         }
 
     }
-    else if( aCommand == "StatusBarVisible" )
-    {
-        // workaround: this should not be necessary.
-        uno::Reference< beans::XPropertySet > xPropSet( m_xFrame, 
uno::UNO_QUERY );
-        if( xPropSet.is() )
-        {
-            uno::Reference< css::frame::XLayoutManager > xLayoutManager;
-            xPropSet->getPropertyValue( u"LayoutManager"_ustr ) >>= 
xLayoutManager;
-            if ( xLayoutManager.is() )
-            {
-                bool bIsVisible( xLayoutManager->isElementVisible( 
u"private:resource/statusbar/statusbar"_ustr ));
-                if( bIsVisible )
-                {
-                    xLayoutManager->hideElement( 
u"private:resource/statusbar/statusbar"_ustr );
-                    xLayoutManager->destroyElement( 
u"private:resource/statusbar/statusbar"_ustr );
-                }
-                else
-                {
-                    xLayoutManager->createElement( 
u"private:resource/statusbar/statusbar"_ustr );
-                    xLayoutManager->showElement( 
u"private:resource/statusbar/statusbar"_ustr );
-                }
-                // @todo: update menu state (checkmark next to "Statusbar").
-            }
-        }
-    }
     else if( aCommand == "ChangeTheme" )
         comphelper::dispatchCommand(u".uno:ChangeTheme"_ustr, getFrame(), 
rArgs);
 }
@@ -1761,7 +1736,6 @@ const o3tl::sorted_vector< std::u16string_view >& 
ChartController::impl_getAvail
         u"NewArrangement",     u"Update",
         u"DefaultColors",      u"BarWidth",             u"NumberOfLines",
         u"ArrangeRow",
-        u"StatusBarVisible",
         u"ChartElementSelector",
 
         // sidebar commands
diff --git a/chart2/source/controller/main/ControllerCommandDispatch.cxx 
b/chart2/source/controller/main/ControllerCommandDispatch.cxx
index b901e865553a..4c97fa1335a5 100644
--- a/chart2/source/controller/main/ControllerCommandDispatch.cxx
+++ b/chart2/source/controller/main/ControllerCommandDispatch.cxx
@@ -60,25 +60,6 @@ using ::com::sun::star::uno::Sequence;
 
 namespace
 {
-bool lcl_isStatusBarVisible( const Reference< frame::XController > & 
xController )
-{
-    bool bIsStatusBarVisible = false;
-    // Status-Bar visible, workaround: this should not be necessary. @todo:
-    // remove when Issue #i68864# is fixed
-    if( xController.is())
-    {
-        Reference< beans::XPropertySet > xPropSet( xController->getFrame(), 
uno::UNO_QUERY );
-        if( xPropSet.is() )
-        {
-            uno::Reference< css::frame::XLayoutManager > xLayoutManager;
-            xPropSet->getPropertyValue( u"LayoutManager"_ustr ) >>= 
xLayoutManager;
-            if ( xLayoutManager.is() )
-                bIsStatusBarVisible = xLayoutManager->isElementVisible( 
u"private:resource/statusbar/statusbar"_ustr );
-        }
-    }
-    return bIsStatusBarVisible;
-}
-
 bool lcl_arePropertiesSame(std::vector<Reference<beans::XPropertySet>>& 
xProperties,
                            const OUString& aPropName)
 {
@@ -953,14 +934,6 @@ void ControllerCommandDispatch::fireStatusEvent(
     }
     else if (m_aCommandAvailability.contains(rURL))
         fireStatusEventForURLImpl( rURL, xSingleListener );
-
-    // statusbar. Should be handled by base implementation
-    // @todo: remove if Issue 68864 is fixed
-    if( rURL.isEmpty() || rURL == ".uno:StatusBarVisible" )
-    {
-        bool bIsStatusBarVisible( lcl_isStatusBarVisible( m_xChartController 
));
-        fireStatusEventForURL( u".uno:StatusBarVisible"_ustr, uno::Any( 
bIsStatusBarVisible ), true, xSingleListener );
-    }
 }
 
 // ____ XDispatch ____
diff --git a/chart2/uiconfig/menubar/menubar.xml 
b/chart2/uiconfig/menubar/menubar.xml
index 53e300045e44..74d431e33e83 100644
--- a/chart2/uiconfig/menubar/menubar.xml
+++ b/chart2/uiconfig/menubar/menubar.xml
@@ -44,7 +44,6 @@
   <menu:menu menu:id=".uno:ViewMenu">
     <menu:menupopup>
       <menu:menuitem menu:id=".uno:AvailableToolbars"/>
-      <menu:menuitem menu:id=".uno:StatusBarVisible"/>
       <menu:menuitem menu:id=".uno:Sidebar" menu:style="text"/>
       <menu:menuseparator/>
       <menu:menuitem menu:id=".uno:DiagramData"/>

Reply via email to