Kaydet (Commit) 598d033c authored tarafından Maxim Monastirsky's avatar Maxim Monastirsky

Fix AddonMenu icon

Another victim of tdf#96059. CommandInfoProvider can only
handle .uno:Something, not slot:12345.

Also, given that we're using the .uno command name, we can
just get the label from officecfg, dropping the local
framework string resource.

Change-Id: Ia0c32bacb67699311f6bbbec0f350321802f75a0
üst 3a72afd0
......@@ -24,7 +24,6 @@
#define RID_MENU_START 10000
#define RID_IMAGE_START 15000
#define STR_MENU_ADDONS (RID_STR_START+0)
#define STR_MENU_HEADFOOTALL (RID_STR_START+2)
#define STR_UPDATEDOC (RID_STR_START+3)
#define STR_CLOSEDOC_ANDRETURN (RID_STR_START+4)
......
......@@ -19,11 +19,6 @@
#include <classes/resource.hrc>
String STR_MENU_ADDONS
{
Text [ en-US ] = "Add-Ons" ;
};
String STR_MENU_HEADFOOTALL
{
Text [ en-US ] = "All" ;
......
......@@ -1232,45 +1232,23 @@ void MenuBarManager::FillMenuManager( Menu* pMenu, const Reference< XFrame >& rF
if ( pPopup->GetItemType( nCount-1 ) != MenuItemType::SEPARATOR )
pPopup->InsertSeparator();
// Use resource to load popup menu title
OUString aAddonsStrRes(FwkResId(STR_MENU_ADDONS));
pPopup->InsertItem( ITEMID_ADDONLIST, aAddonsStrRes );
pPopup->InsertItem( ITEMID_ADDONLIST, OUString() );
pPopup->SetPopupMenu( ITEMID_ADDONLIST, pSubMenu );
// Set item command for popup menu to enable it for GetImageFromURL
OUString aNewItemCommand = "slot:" + OUString::number( ITEMID_ADDONLIST );
pPopup->SetItemCommand( ITEMID_ADDONLIST, aNewItemCommand );
pPopup->SetItemCommand( ITEMID_ADDONLIST, ".uno:Addons" );
}
else
pSubMenu.disposeAndClear();
}
MenuBarManager* pSubMenuManager;
if ( nItemId == ITEMID_ADDONLIST )
{
AddonMenu* pSubMenu = dynamic_cast< AddonMenu* >( pPopup.get() );
if ( pSubMenu )
{
MenuBarManager* pSubMenuManager = new MenuBarManager( m_xContext, m_xFrame,
m_xURLTransformer,pSubMenu, false );
AddMenu(pSubMenuManager,aItemCommand,nItemId);
(pSubMenuManager->m_aMenuItemCommand).clear();
// Set image for the addon popup menu item
if ( bItemShowMenuImages && !pPopup->GetItemImage( ITEMID_ADDONLIST ))
{
Image aImage = vcl::CommandInfoProvider::GetImageForCommand(aItemCommand, rFrame);
if ( !!aImage )
pPopup->SetItemImage( ITEMID_ADDONLIST, aImage );
}
}
}
pSubMenuManager = new MenuBarManager( m_xContext, m_xFrame, m_xURLTransformer, pPopup, false );
else
{
MenuBarManager* pSubMenuMgr = new MenuBarManager( m_xContext, rFrame, m_xURLTransformer,
rDispatchProvider, aModuleIdentifier,
pPopup, false, m_bHasMenuBar );
AddMenu(pSubMenuMgr,aItemCommand,nItemId);
}
pSubMenuManager = new MenuBarManager( m_xContext, rFrame, m_xURLTransformer,
rDispatchProvider, aModuleIdentifier,
pPopup, false, m_bHasMenuBar );
AddMenu(pSubMenuManager, aItemCommand, nItemId);
}
}
else if ( pMenu->GetItemType( i ) != MenuItemType::SEPARATOR )
......
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