Bug 157235 - OS highlight colour not picked up for row/column and cursor, red used instead (gen/x11 VCL plugin)
Summary: OS highlight colour not picked up for row/column and cursor, red used instead...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.6.1.2 release
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: UI-Theming Cell-Selection
  Show dependency treegraph
 
Reported: 2023-09-14 10:18 UTC by Stéphane Guillou (stragu)
Modified: 2023-09-18 11:01 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
screenshot of LO Calc 7.5.6.2 (left) vs 24.2 alpha0+ (17.03 KB, image/png)
2023-09-14 10:18 UTC, Stéphane Guillou (stragu)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stéphane Guillou (stragu) 2023-09-14 10:18:24 UTC
Created attachment 189567 [details]
screenshot of LO Calc 7.5.6.2 (left) vs 24.2 alpha0+

In 7.6, the default colour for the active row(s), column(s) and cursor has changed from blue to red, when other VCL plugins make use of the OS/DE's default highlight colour. (Ubuntu/GNOME's default orange is picked up correctly in gtk3, kf5 and qt5 in my tests.)
This is despite the cell range highlight still being a shade of blue.

Some blue also remains in the borders and the corner drag square, making it even more inconsistent. See attachment 189299 [details] (top is 7.5, bottom is 7.6).

Started with the fix for bug 145080:

commit 258686a58f909ab04c7281c05f15882eb400748e
author	Heiko Tietze Wed May 17 16:48:30 2023 +0200
committer	Heiko Tietze Tue May 23 09:44:21 2023 +0200
Resolves tdf#145080 - Use accent color for focused cell
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151887

... which was supposed to be a macOS fix.

Repro on Ubuntu 20.04 with GNOME 3.36.8 with:

Version: 7.6.1.1 (X86_64) / LibreOffice Community
Build ID: c7cda394c5de06de37d8109c310df89a4d4c3a98
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded

and a recent master build:

Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 4942aa1533af25cf102da7516bd5e521c553aa07
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded

Confirmed in bug 156800 for XFCE + x11 as well.
Comment 1 Stéphane Guillou (stragu) 2023-09-14 10:19:11 UTC
Maroon colour is:

RGB: 127 0 0
HEX: #7f0000

Blue colour is:

RGB: 0 0 127
HEX: #00007f

Which I think match our COL_RED and COL_BLUE respectively. I see the commit added:

   maAccentColor               = COL_RED

Maybe related?
Comment 2 Stéphane Guillou (stragu) 2023-09-15 10:52:31 UTC
(@nouvo09: please don't change the Version field to a more recent one, it's supposed to be the earliest version affected.)
Comment 3 Heiko Tietze 2023-09-18 11:01:46 UTC
Not really a regression but missing function. Caolan, should/can the generic VCL pick some *accent* color from the OS? The easy alternative is to default to what we used in the past resp. the highlight color.