diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2021-06-24 16:02:02 +0300 |
---|---|---|
committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2021-06-28 10:18:14 +0200 |
commit | e51ad28ba22105c60be246f02e7951b986f35150 (patch) | |
tree | f709e71605177ee9c9528f57234486dabd8ecdb0 | |
parent | sw: layout: fix table split loop caused by RemoveFollowFlowLinePending (diff) | |
download | core-e51ad28ba22105c60be246f02e7951b986f35150.tar.gz core-e51ad28ba22105c60be246f02e7951b986f35150.zip |
tdf#143047: When no properties for command, use those from item container
This allows to use the "Label" property defined by user for macro
items, that indeed have no built-in command properties. Thus the
resulting tooltip will be a concatenation of the label and mnemonic
instead of only mnemonic as happened before.
Change-Id: I79d933fecd09ca7d64b7a3e7db0ef194e42c1c55
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117787
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit 8383175bd3c714c163eac40a00b1e8277d4d5307)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117857
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
-rw-r--r-- | framework/source/uielement/toolbarmanager.cxx | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx index 53d456f235c5..69dcd3d7a158 100644 --- a/framework/source/uielement/toolbarmanager.cxx +++ b/framework/source/uielement/toolbarmanager.cxx @@ -1014,6 +1014,8 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine if (( nType == css::ui::ItemType::DEFAULT ) && !aCommandURL.isEmpty() ) { auto aProperties = vcl::CommandInfoProvider::GetCommandProperties(aCommandURL, m_aModuleIdentifier); + if (!aProperties.hasElements()) // E.g., user-provided macro command? + aProperties = aProps; // Use existing info, including user-provided Label OUString aString(vcl::CommandInfoProvider::GetLabelForCommand(aProperties)); ToolBoxItemBits nItemBits = ConvertStyleToToolboxItemBits( nStyle ); |