Bug 133214 - UI: Icons of the tabbed toolbar aren't positioned properly on MacOS
Summary: UI: Icons of the tabbed toolbar aren't positioned properly on MacOS
Status: RESOLVED DUPLICATE of bug 125511
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
6.3 all versions
Hardware: All macOS (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: implementationError
: 134153 (view as bug list)
Depends on:
Blocks: macOS-UI-polish Notebookbar-Tabbed
  Show dependency treegraph
 
Reported: 2020-05-21 07:35 UTC by Telesto
Modified: 2023-03-18 19:11 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot Mac (28.71 KB, image/png)
2020-05-21 07:36 UTC, Telesto
Details
First version of a patch. (1.52 KB, patch)
2020-12-11 19:03 UTC, Sven Lüppken
Details
Screenshot of GTK UI (180.25 KB, image/png)
2021-01-21 20:28 UTC, Thorsten Wagner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2020-05-21 07:35:20 UTC
Description:
UI: Icons of the tabbed toolbar aren't positioned properly on MacOS

Steps to Reproduce:
1. Open Writer on MacOS
2. Enable tabbed toolbar

Actual Results:
Not proper position

Expected Results:
Right position. no overlap


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.0.0.0.alpha1
Build ID: 6a03b2a54143a9bc0c6d4c7f1...
CPU threads: 4; OS: Mac OS X 10.12.6; UI render: GL; VCL: osx; 
Locale: nl-NL (nl_NL.UTF-8); UI: en-US
Calc: threaded
Comment 1 Telesto 2020-05-21 07:36:37 UTC
Created attachment 161060 [details]
Screenshot Mac
Comment 2 Roman Kuznetsov 2020-05-24 11:31:53 UTC
confirm in

Version: 7.0.0.0.alpha1+
Build ID: b1e396d86655a0131498a4691dd8069ea76c3477
CPU threads: 4; OS: Mac OS X 10.15.4; UI render: default; VCL: osx; 
Locale: ru-RU (ru_RU.UTF-8); UI: en-US
Calc: threaded
Comment 3 Dieter 2020-07-23 11:34:43 UTC
*** Bug 134153 has been marked as a duplicate of this bug. ***
Comment 4 andreas_k 2020-10-31 21:47:58 UTC
I don't have an Mac, but from my point of view I would hide notebookbarshortcuts.xml and also the menubar on the right conrner on MacOS, cause MacOS has a menubar this commands are note needed in general.
Comment 5 mrei 2020-11-16 08:04:20 UTC
I am missing the 6.2 bundle at the moment, but I took a peek at the 6.1 version, where the tabbed toolbar does not exist yet.
It looks like this is something that is occurring since being put in.

This is also affecting other LibreOffice components like Calc and Impress.
Comment 6 Sven Lüppken 2020-12-11 19:03:13 UTC
Hi! :)

I looked into this and saw two things:

1) The layout is off, as described in this bug
2) The text color of the tabs is wrong starting with macOS Big Sur: Prior to macOS 11, the buttons had a blue background with white text. Now it's a white background with black text. I changed the text color to black.

I implemented a very simple fix (attached to this bug, I can push it to Gerrit though) which looks great on Big Sur, but will look broken in older versions of macOS. Is there a way for me to figure out which macOS version is used by the user?

Thanks!
Sven
Comment 7 Sven Lüppken 2020-12-11 19:03:45 UTC
Created attachment 168076 [details]
First version of a patch.
Comment 8 Sven Lüppken 2020-12-15 15:52:29 UTC
Ok, this patch is "too" simple. I'm working on something else. :)
Comment 9 Thorsten Wagner 2021-01-21 20:27:39 UTC
Sven, I suggest not to fix issue within macOS specific VCL code (vcl/osx and vcl/quartz) directories. Please find attached a screenshot from a GTK UI:

(1) Different tabs are simple text labels similar to Windows UIs.

(2) With current macOS implementation dialog tabs are used. These tabs are positioned centered on the top border of the window. I guess this causes area for icons above to be calculated too small.

My proposal:

(1) Identify code to be used to draw notebookbar outside macOS specific code and replace usage of native widgets (tab items in this case) by using stock widgets (independent from GTK, QT, Windows or macOS backends) - take care not to cause side effects to other backends.

(2) Doublecheck whether toolbar is drawn similar to GTK.

If the issue still exists thereafter, reasons of miscalculation have to be identified. In any case I suggest not to use HITheme tabs for notebookbar.

BTW: Wrong text colors within tabs on macOS Big Sur have been fixed in the meantime.
Comment 10 Thorsten Wagner 2021-01-21 20:28:19 UTC
Created attachment 169080 [details]
Screenshot of GTK UI
Comment 11 eisa01 2023-03-18 19:01:11 UTC

*** This bug has been marked as a duplicate of bug 125511 ***