Created attachment 161641 [details] Calc request to optimize the visual layout of the calc specific toolbar 1. The Number Box has no space to the border (see other toolbar elements) 2. The height of the calc specific toolbar should be a bit higher or at least there should be some space on top and bottom of the Number Box and Input Line. (green circles) 3. I don't think that there are separators needed cause left and right are boxes with vertical lines 4. The arrow to increase the Input Line would fit better if it will be inside of the Input Line field. As example see Excel's layout
Yes, let's have 2px or so margin. Thorsten, one for you?
Space should be possible, but it seems to be a Windows-only issue (see additional screenshot from macOS attached). Removing separators should be possible as well. Moving the arrow inside input line will cause problems with other widget sets (e.g. GTK, Carbon/Cocoa) as they are using thinner scrollbars than Windows. This would bring back the original issue with an input line appropriate for different widget set. I will look at spacing but keep arrow outside input line as is. As I'm capable to build on macOS and Linux only, who is able to build and test on Windows?
Created attachment 161780 [details] Additional screenshot from macOS
Created attachment 161784 [details] QT backend compare font name and name box they should be in one vertical line. I think it's the same on osx. In gtk3 it's the same.
In addition it would be nice if the Name Box widget has the same width than the Font Name widget (which is above the Name Box widget).
And as I wrote I'm for remove the two separators see first attachment.
(In reply to Thorsten Wagner from comment #2) > ... who is able to build and test on Windows? Afraid you have to submit and wait for Andreas to test the nightly build. Thanks for looking into the issue.
I will test on the daily build when the patch is in master on windows, gtk3 and qt backend. Thanks for your support Thorsten.
Committed a frist patch to gerrit: https://gerrit.libreoffice.org/c/core/+/96263 Patch contains the following changes: (1) Margin of 2 px added to top and bottom of all control widgets within toolbars if their height exceeds height of other widgets within bar (this will cause margins within Calc formulabar for QT's, GTK's, and Windows widgets, spacing on macOS will remain unchanged as widgets are not that large) (2) Separators removed, Space between position widget and buttons to control inputline inserted to show association of buttons (3) Width of position widget set to width of style widget within formatting toolbar Remaining issue: (1) Screenshot macOS: Formatting toolbar on top of Calc inputbar has drag area on left causing an indent of style widget; comboboxes are not aligned due to drag area (2) Screenshot Linux, QT 1: Undocking of toolbars seem to be disabled with QT. Nevertheless there is a small indent on toolbars with (disabled) drag areas; comboboxes are not aligned due to (disabled) drag area (3) Screenshot Linux, QT 2: Notebookbar has the same indent as Calc's formulabar (4) Screenshot Linux, GTK: Undocking of toolbars seem to be disabled with GTK. Nevertheless there is a small indent on toolbars with (disabled) drag areas; comboboxes are not aligned due to (disabled) drag area As left margins with notebookbar are correct, indent of Calc's formulabar position widget seems to be correct too. Suggestion: Remove left margins on toolbars with disabled drag area (as separate patch)
Created attachment 161959 [details] Screenshot from macOS
Created attachment 161960 [details] Screenshot from Linux with QT 1
Created attachment 161961 [details] Screenshot from Linux with QT 2
Created attachment 161962 [details] Screenshot from Linux with GTK
BTW: Why are drag handles disabled on Linux/QT and Linux/GTK while being enabled on macOS?
(In reply to Thorsten Wagner from comment #9) > Committed a frist patch to gerrit: > > https://gerrit.libreoffice.org/c/core/+/96263 > > Patch contains the following changes: > > (1) Margin of 2 px added to top and bottom of all control widgets within > toolbars if their height exceeds height of other widgets within bar (this > will cause margins within Calc formulabar for QT's, GTK's, and Windows > widgets, spacing on macOS will remain unchanged as widgets are not that > large) work fine on all screenshots. so + 1 > (2) Separators removed, Space between position widget and buttons to control > inputline inserted to show association of buttons + 1 for remove separators. The space between Name Box and Function wizzard is to big. > (3) Width of position widget set to width of style widget within formatting > toolbar + 1 thanks a lot > Remaining issue: > > (1) Screenshot macOS: Formatting toolbar on top of Calc inputbar has drag > area on left causing an indent of style widget; comboboxes are not aligned > due to drag area as you wrote, should be fixed toolbar. > (2) Screenshot Linux, QT 1: Undocking of toolbars seem to be disabled with > QT. Nevertheless there is a small indent on toolbars with (disabled) drag > areas; comboboxes are not aligned due to (disabled) drag area > > (3) Screenshot Linux, QT 2: Notebookbar has the same indent as Calc's > formulabar yes that look really good. > (4) Screenshot Linux, GTK: Undocking of toolbars seem to be disabled with > GTK. Nevertheless there is a small indent on toolbars with (disabled) drag > areas; comboboxes are not aligned due to (disabled) drag area > > As left margins with notebookbar are correct, indent of Calc's formulabar > position widget seems to be correct too. > > Suggestion: Remove left margins on toolbars with disabled drag area (as > separate patch) as remove left margins on fixed toolbars are not really a good idea see attachment 161641 [details] on windows I recommend to add left margin to formulabar which will than fit prefect on os-x, qt1, gtk3 and windows when the other toolbars are fixed (which should be the default see bug I don't know which number), but didn't fit perfect on tabbed notebookbars and non fixed toolbars. But both are not the default, so on default it should work.
Expand formula bar button has an right margin. Please remove it so the arrow look at least align in the gray area. If the arrow can be replaced by an icon from the icon theme, than it would be even more perfect, cause than the designers can design them according to the icon theme. You only have to tell us the icon name. But first step would be to remove the right margin.
Created attachment 161967 [details] Screenshot from macOS + additional tuning Compare with attachment 161959 [details]: Additinal fine tuning: 1. horizontal line on top of headers was removed. 2. expand formular Bar button has size of scrollbar width but is there center align. 3. split window "line" button has height of headers and is easier to grap. 4. 2. and 3. symbols come from the icon theme 5. Name box is align with Font Name (when toolbars are fixed) 6. space between function name and function wizzard is not that big as in the other screenshot.
(In reply to andreas_k from comment #17) > 2. expand formular Bar button has size of scrollbar width but is there > center align. Exactly this was the first approach back in 2016 (see screenshots in tdf#127066). Placing arrow above scrollbar was dropped because there are very thin scollbars especially on GTK. That is the cause for the right margin too. Margin causes arrow clearly associated with inputline. Design used for Excel is only possible with scrollbars wide enough: Default KDE5 design is appropriate, macOS design is still appropriate, Windows design is appropriate, but many GTK themes are not appropriate.
*** Bug 133057 has been marked as a duplicate of this bug. ***
Created attachment 161973 [details] Extended input line at least when the input line is extended it look strange with the two scrallbars. I don't have an good idea for the Collapse formular Bar button. You already wrote that it's not possible to draw this collapse button inside the Input line field. At least the right margin could be removed so it look align and the space between the scrallbars is not so big. If someone has a better idea, ...
(In reply to Thorsten Wagner from comment #10..13) > Screenshot from <> top/bottom space LGTM; left margin inconsistent with other toolbars (except Notebookbar). Thanks for taking care.
Added a current screenshot from macOS 2: (1) Indent of position combobox aligned with style combobox (2) Replaced space between position combobox with separator again (like in Excel), because separator causes a further indent to get next icon aligned with size combobox form formatting toolbar; second separator remains deleted Will look at the right side now, not sure to find a better solution as is, because right margin seems to be caused by toolbar implementation
Created attachment 161995 [details] Screenshot from macOS 2
(In reply to Thorsten Wagner from comment #23) > Created attachment 161995 [details] > Screenshot from macOS 2 great
Committed an updated patchset to gerrit: https://gerrit.libreoffice.org/c/core/+/96263 Added screenshots from Linux/Qt and Linux/GTK as well. Right margin is caused by toolbar implementation. It is as wide as left margin. Changing this will cause poor modifications to VCL. So for the time beeing I suggest to review patch now.
Created attachment 162000 [details] Screenshot from Linux with QT 3
Created attachment 162001 [details] Screenshot from Linux with GTK 2
Gerrit failed due to a reason not related to patch. Change has been abandoned and committed again do Gerrit, verified +1 now: https://gerrit.libreoffice.org/c/core/+/96403
Thorsten Wagner committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f24e64e8b55b695565109a6742598bcc90e4d9d0 tdf#133692: Spacing within Calc formulabar reworked It will be available in 7.1.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Thorsten Wagner committed a patch related to this issue. It has been pushed to "libreoffice-7-0": https://git.libreoffice.org/core/commit/4366fdf8cb7b18bfc8f4665aa09f35e20467d3a5 tdf#133692: Spacing within Calc formulabar reworked It will be available in 7.0.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Great job, Thorsten!
Created attachment 162140 [details] Screenshot from windows after the patch thanks a lot Thorston is there any change to remove the top horizontal line of the headers bar? Like shown in attachment 161967 [details]. the expand formular bar can it be themable, so can it use an icon from the icon theme?
Andreas, I will take a look a the line. As I am very busy right now, it will take some time ... The expand icon is still an image icon with a symbol assigned to it currently. Using an image from an icon set is possible easily.
*** Bug 85517 has been marked as a duplicate of this bug. ***