diff options
author | Henry Castro <hcastro@collabora.com> | 2021-03-17 07:35:39 -0400 |
---|---|---|
committer | Szymon Kłos <szymon.klos@collabora.com> | 2021-03-17 16:18:24 +0100 |
commit | c8881fa20aa7b194cc2b87b78903d6d2e5780120 (patch) | |
tree | 44d55d6f9efd6b4b010326029157c9571a1e34a7 /cui/source/inc/cfgutil.hxx | |
parent | e693b5b6f5d61f8f4108cc70669ce3e5e87910a6 (diff) |
lok: fix the empty script node type container without "+" symbol expander
According to description in https://bz.apache.org/ooo/show_bug.cgi?id=30923
"In this case I think it's an acceptable solution to display a '+' symbol for all
library nodes, and if a library has no children and the user clicks on the '+'
symbol, the '+' will disappear".
Unfortunately this behaviour is not well driven for final users'
client side, so they misinterpret as a bug.
After tracing the method hasChildNodes(), the log result indicates
that this method does not load the libraries, but it is necessary
to iterate applying the method getChildNodes(), which will cause
to load all libraries and determine if the child has a script
CONTAINER node type to assign the '+' symbol.
Also, I have traced the loading libraries when the Macro Selector
dialog pops up and the result it load once, so I do not think it
is a problem loading a small finite number of libraries today.
Change-Id: I4ae5395b6afa7b7d6ff2b2ec692771e93c46c529
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112636
Reviewed-by: Pedro Silva <pedro.silva@collabora.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Diffstat (limited to 'cui/source/inc/cfgutil.hxx')
-rw-r--r-- | cui/source/inc/cfgutil.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cui/source/inc/cfgutil.hxx b/cui/source/inc/cfgutil.hxx index 20da0fc6f4a6..f6301f4e09bb 100644 --- a/cui/source/inc/cfgutil.hxx +++ b/cui/source/inc/cfgutil.hxx @@ -203,7 +203,7 @@ class CuiConfigGroupListBox void InitModule(); void FillScriptList(const css::uno::Reference< css::script::browse::XBrowseNode >& xRootNode, - const weld::TreeIter* pParentEntry, bool bCheapChildrenOnDemand); + const weld::TreeIter* pParentEntry); void FillFunctionsList(const css::uno::Sequence< css::frame::DispatchInformation >& xCommands); OUString MapCommand2UIName(const OUString& sCommand); |