Bug 142947 - Make menu expander triangles configurable for icon design
Summary: Make menu expander triangles configurable for icon design
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
6.2.8.2 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
: 107259 (view as bug list)
Depends on:
Blocks: Notebookbar-Theming
  Show dependency treegraph
 
Reported: 2021-06-19 20:47 UTC by Jérôme
Modified: 2023-09-20 19:02 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
a group of the GoupedBar Compact UI (2.02 KB, image/png)
2021-06-19 20:50 UTC, Jérôme
Details
the former example shows no size difference anymore (2.23 KB, image/png)
2023-09-17 09:44 UTC, Jérôme
Details
different size between the dropdown text list and other dropdown menu icons (4.92 KB, image/png)
2023-09-17 09:46 UTC, Jérôme
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jérôme 2021-06-19 20:47:42 UTC
Description:
I noticed that there is a big arrow beside the menu of each group in order to show that a popup textual menu can appear after a click.

Why not using a smaller arrow ? For example this arrow could be the same than the arrow of the icon which provides a popup menu.

The spared space could be used for an additional icon.

Actual Results:
A big arrow use a lot of space in a group of the GrouppedBar compact UI.

Expected Results:
Spare space in order to display one additional icon.


Reproducible: Always


User Profile Reset: No



Additional Info:
I noticed in the versions 6.2.8.2 and 7.1.3.2.
Comment 1 Jérôme 2021-06-19 20:50:11 UTC
Created attachment 173020 [details]
a group of the GoupedBar Compact UI

You can see :
- small arrows on the first line of icons
- a big arrow on the second line.
Comment 2 Jérôme 2021-06-20 06:32:29 UTC
An other possibility would be to use a small character at the end of the title.

For example, we could replace "File" by "File…". Note that a single character gather the 3 dots.
Comment 3 Heiko Tietze 2021-07-20 06:56:25 UTC
Not sure that we have full control of the expander icons.
Comment 4 andreas_k 2021-07-20 07:00:00 UTC
I would also prefer to have the same arrow size everywhere.
Comment 5 Heiko Tietze 2021-07-20 07:06:59 UTC
(In reply to andreas_k from comment #4)
> I would also prefer to have the same arrow size everywhere.

Please resolve NAB/WF as maintainer if your preference for one-sized arrows is strong. If you want more input we can leave the ticket open for a while.
Comment 6 Heiko Tietze 2021-09-13 11:10:51 UTC
The expander triangle is sized to 75% of the control's text height, see SvxMenuEntriesListBox::CreateDropDown() in cui/source/customize/cfg.cxx, while the toolbar is a bit more complex, see void ImplDrawDropdownArrow() in vcl/source/window/toolbox.cxx (changed recently for bug 122118). 

We have to deal with different icon sizes where small icons require smaller expander triangles. The only solution I see is to replace the triangle vectors by a pre-drawn bitmaps, which then depends on the icon size (and can be SVGs too, of course). This would solve bug 136757.

Caolan, is this a reasonable idea?

Making bug 107259 a duplicate of this one. It seems to be the same request though talking also about OS dependency.
Comment 7 Heiko Tietze 2021-09-13 11:11:07 UTC
*** Bug 107259 has been marked as a duplicate of this bug. ***
Comment 8 Caolán McNamara 2021-09-13 11:46:38 UTC
IIRC SvxMenuEntriesListBox::CreateDropDown is only for the arrow shown in the tools, customize dialog in the "menu" and "context menus" pages for the arrow preview in the "assigned comments" lists.

There is sometimes the use of DecoView::DrawSymbol with SPIN_DOWN to draw these kind of things, I would have guessed ImplDrawDropdownArrow used that, but as you say it does its own special thing there. I wonder if the original screenshot is showing "Fichier" in a button using DecoView::DrawSymbol vs ImplDrawDropdownArrow in the above row. Or if its the same thing used in both places but one bigger because more space allocated for the arrow.
Comment 9 Heiko Tietze 2021-09-16 08:09:16 UTC
This code sizes the triangles equally but it depends only on the view setting for toolbars. Small icons on Notebookbars don't fit the size of large on toolbars. The toolbox is not aware of the position (NB, TB or sidebar) and we shouldn't make it dependent. 

-    const tools::Long nSize = rDropDownRect.getWidth() - 2 * nMargin;
+    tools::Long nSize;
+    switch (officecfg::Office::Common::Misc::SymbolSet::get())
+    {
+       case 0: nSize = 10 - 2 * nMargin; break; //S
+       case 1: nSize = 13 - 2 * nMargin; break; //L
+       case 2: nSize = 13 - 2 * nMargin; break; //Auto
+       case 3: nSize = 16 - 2 * nMargin; break; //XL
+    }

My take: WF resp. size the controls containing icons in NB equally.
Comment 10 QA Administrators 2023-09-17 03:05:50 UTC Comment hidden (obsolete)
Comment 11 Jérôme 2023-09-17 09:44:22 UTC
Created attachment 189649 [details]
the former example shows no size difference anymore
Comment 12 Jérôme 2023-09-17 09:46:25 UTC
Created attachment 189650 [details]
different size between the dropdown text list and other dropdown menu icons
Comment 13 Jérôme 2023-09-17 09:58:47 UTC
The problem shown in the former screenshot is fixed.
However, the menu expander triangle is still big for the dropdown text lists.

Version: 7.6.1.2 (X86_64) / LibreOffice Community
Build ID: 4412c0006c0cfe5a5d40cae25a00da8a194aa4c0
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: fr-FR
Calc: threaded
Comment 14 QA Administrators 2023-09-18 03:06:05 UTC Comment hidden (obsolete)
Comment 15 Jérôme 2023-09-18 19:14:15 UTC
A noticed a progress but this bug isn't fixed yet with below version :

Version: 7.6.1.2 (X86_64) / LibreOffice Community
Build ID: 4412c0006c0cfe5a5d40cae25a00da8a194aa4c0
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: fr-FR
Calc: threaded

See comment comment 11, comment 12 and comment 13 for details.
Comment 16 QA Administrators 2023-09-19 03:05:57 UTC Comment hidden (obsolete)
Comment 17 Jérôme 2023-09-19 21:16:54 UTC
This bug is fixed in most cases but not all.

Version: 7.6.1.2 (X86_64) / LibreOffice Community
Build ID: 4412c0006c0cfe5a5d40cae25a00da8a194aa4c0
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: fr-FR
Calc: threaded

See comment comment 11, comment 12 and comment 13 for details.
Comment 18 QA Administrators 2023-09-20 03:06:07 UTC Comment hidden (obsolete)
Comment 19 Jérôme 2023-09-20 19:02:22 UTC
This bug isn't fixed in version 7.6.1.2.
The menu expander triangle is still big for the dropdown text lists.

Version: 7.6.1.2 (X86_64) / LibreOffice Community
Build ID: 4412c0006c0cfe5a5d40cae25a00da8a194aa4c0
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: fr-FR
Calc: threaded

See comment comment 11 and comment 12 for details.