Bug 142608 - Background color of Notebookbar tabbed tabs does not match the background of the bar itself (kf5)
Summary: Background color of Notebookbar tabbed tabs does not match the background of ...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
7.0.6.2 release
Hardware: All Linux (All)
: low trivial
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: KDE, KF5 Notebookbar-Tabbed
  Show dependency treegraph
 
Reported: 2021-06-01 14:54 UTC by Lorenz Hoffmann
Modified: 2023-10-20 09:02 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot of the problem (12.52 KB, image/png)
2021-06-01 14:54 UTC, Lorenz Hoffmann
Details
Screenshot of the UI settings (73.21 KB, image/png)
2021-06-09 09:41 UTC, Lorenz Hoffmann
Details
Screenshot of the patched UI (61.38 KB, image/png)
2022-10-19 19:33 UTC, Rafael Lima
Details
Screenshot gtk3 with https://gerrit.libreoffice.org/c/core/+/141411 v2 and gtk3 Adwaita theme (61.71 KB, application/pdf)
2022-11-10 14:16 UTC, Michael Weghorn
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lorenz Hoffmann 2021-06-01 14:54:43 UTC
Created attachment 172558 [details]
Screenshot of the problem

When using the "in registers" view, the background of those registers is different to that of its content.
Comment 1 Timur 2021-06-08 13:41:21 UTC
Not clear. You need to write all, not assuming it's obvious.
Is this about Tabbed UI?
That inactive tabs are darker? Which is not a bug.
Comment 2 Lorenz Hoffmann 2021-06-08 14:04:24 UTC
No, I mean this thing you click on to select a tab in the top row, like the head if you will, has a different background than its content.
In other words, the background of "Datei" is different to the background of "Neu", "Öffnen", "Dokumentvorlagen" etc.

The result is this weird line between tab header and tab content where the background changes slightly without a borderline. In my opinion, that does not look good.

I hope that describes it better, should have made a marker in the screenshot.
Comment 3 Timur 2021-06-09 09:06:31 UTC
I see it in your screenshot, trivial as marked. 
But again you didn't write what's your OS, some customization like dark mode and what is in Options-View, did you change or left default (also please screenshot). Only written is LO 7.0.5.
That being said, I don't see in Linux and Windows with default settings.
Comment 4 Lorenz Hoffmann 2021-06-09 09:41:15 UTC
Created attachment 172725 [details]
Screenshot of the UI settings
Comment 5 Lorenz Hoffmann 2021-06-09 09:47:25 UTC
The only change I made is that I switched the User Interface to "Tabbed".

My version info from "About LibreOffice":
Version: 7.0.6.2
Build ID: 00(Build:2)
CPU threads: 12; OS: Linux 5.12; UI render: default; VCL: kf5
Locale: de-DE (de_DE.UTF-8); UI: de-DE
7.0.6-1
Calc: threaded

(In the meantime, I updated LibreOffice to 7.0.6.2.)


System information:
Operating System: Manjaro Linux
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.82.0
Qt Version: 5.15.2
Kernel Version: 5.12.8-1-MANJARO
OS Type: 64-bit
Graphics Platform: X11
Processors: 12 × Intel® Core™ i5-10400F CPU @ 2.90GHz
Memory: 15.5 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 1060 6GB/PCIe/SSE2

As you can see, I am running KDE Plasma, using the Breeze theme by default.
Comment 6 Buovjaga 2022-04-27 12:39:34 UTC
Repro, but only with kf5 UI. Not on Windows either.

Arch Linux 64-bit
Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: 1c1427e81c2463fbc40855adfafa360c13108abb
CPU threads: 8; OS: Linux 5.17; UI render: default; VCL: kf5 (cairo+xcb)
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: threaded Jumbo
Built on 27 April 2022
Comment 7 Jan-Marek Glogowski 2022-05-28 12:47:38 UTC
I had a look today and the background is painted by the Qt styling function. Here it's actually the same color then the toolbar / menubar. Even reading Qt code, it's not obvious for me, why / where this color is selected. Maybe it's a bug in the breeze theme engine? There is a lot of stuff in Breeze, which is not available via the QStyle interface, like the white mouse-over text for buttons and obviously several animations. I tried to adapt the state, but nothing changed the background color. LO could overwrite the background setting with the Window background color for the selected tab...
Comment 8 Rafael Lima 2022-10-18 18:27:43 UTC
This is not limited to the Tabbed interface, but also to any tabs in LibreOffice UI.

For instance, in Writer go to Format - Paragraph and notice the color of the selected tab.
Comment 9 Rafael Lima 2022-10-19 19:33:27 UTC
Created attachment 183138 [details]
Screenshot of the patched UI

Actually it's not the Tab that has the wrong color, but the tab pane was using the wrong color (for some reason I could not figure out).

Both the Tab and the Tab contents need to use DialogColor. See the attached image of the UI with the patch.

Here's the patch:
https://gerrit.libreoffice.org/c/core/+/141411
Comment 10 Rafael Lima 2022-10-19 19:34:07 UTC
FTR the dark theme works as well.
Comment 11 Michael Weghorn 2022-11-10 14:16:03 UTC
Created attachment 183520 [details]
Screenshot gtk3 with https://gerrit.libreoffice.org/c/core/+/141411 v2 and gtk3 Adwaita theme

The attached screenshot show how https://gerrit.libreoffice.org/c/core/+/141411 (PS2) affects gtk3 with the Adwaita theme. I'll add a comment to the Gerrit change with some more notes.
Comment 12 Stéphane Guillou (stragu) 2023-09-09 21:23:03 UTC
(In reply to Michael Weghorn from comment #11)
> Created attachment 183520 [details]
> Screenshot gtk3 with https://gerrit.libreoffice.org/c/core/+/141411 v2 and
> gtk3 Adwaita theme
> 
> The attached screenshot show how
> https://gerrit.libreoffice.org/c/core/+/141411 (PS2) affects gtk3 with the
> Adwaita theme. I'll add a comment to the Gerrit change with some more notes.

Michael, which one it the one with the patch? Both say "without".
Comment 13 Rafael Lima 2023-09-10 13:50:09 UTC
FTR I ended up abandoning the patch. I'm setting it back to NEW.

Sorry for taking so long to update the ticket status.
Comment 14 Michael Weghorn 2023-10-20 09:02:58 UTC
(In reply to Stéphane Guillou (stragu) from comment #12)
> > The attached screenshot show how
> > https://gerrit.libreoffice.org/c/core/+/141411 (PS2) affects gtk3 with the
> > Adwaita theme. I'll add a comment to the Gerrit change with some more notes.
> 
> Michael, which one it the one with the patch? Both say "without".

The first screenshot is without, the second one with the patch applied.
(Sorry for the confusion and the late reply.)