Bug 503095 - [Regression] With dark desktop theme, transaction form becomes white-on-white when entering new transaction
Summary: [Regression] With dark desktop theme, transaction form becomes white-on-white...
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: ux-ui (other bugs)
Version First Reported In: git (master)
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
: 503242 (view as bug list)
Depends on:
Blocks:
 
Reported: 2025-04-21 08:23 UTC by Paul Worrall
Modified: 2025-05-04 06:48 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.2
Sentry Crash Report:


Attachments
screenshot (303.89 KB, image/png)
2025-04-21 08:23 UTC, Paul Worrall
Details
Screenshot showing KMyMoney using Breeze Dark theme (43.12 KB, image/png)
2025-04-21 09:07 UTC, Thomas Baumgart
Details
Screenshot showing colour of transaction form in kmymoney-master-4220-linux-gcc-x86_64.AppImage (91.25 KB, image/png)
2025-04-21 10:32 UTC, Paul Worrall
Details
Same bug on Windows 10 (11.46 KB, image/png)
2025-04-23 20:07 UTC, Tcpsorcerer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Worrall 2025-04-21 08:23:46 UTC
Created attachment 180468 [details]
screenshot

STEPS TO REPRODUCE
1. Select Breeze Dark desktop theme
2. Configure KMM to show the transaction form at the bottom of the ledger view
2. Open an account ledger
3. Select "New Transaction"

OBSERVED RESULT
Background of transaction form turns white (See screenshot)

EXPECTED RESULT
Transaction form has a dark background (like it has before selecting "New Transaction"

SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.3.4
KDE Frameworks Version: 6.13.0
Qt Version: 6.9.0
Graphics Platform: Wayland
Graphics Processor: AMD CEDAR

ADDITIONAL INFORMATION
Can reproduce this with kmymoney-master-4269-linux-gcc-x86_64.AppImage and with 5.1.95-5dfa71fd6 built from git
Comment 1 Thomas Baumgart 2025-04-21 09:07:07 UTC
Created attachment 180469 [details]
Screenshot showing KMyMoney using Breeze Dark theme

I tried hard to duplicate it with Breeze dark on stock openSUSE Leap 15.6 running with X11. It looks perfectly OK to me. The selection to show the transaction form should not make a difference, as it is hidden as soon as you open an editor. How does the editor look like when you edit an existing transaction?
Comment 2 Paul Worrall 2025-04-21 10:23:30 UTC
(In reply to Thomas Baumgart from comment #1)
I get the same issue when I edit an existing transaction

In case it helps:  I do not get the issue with kmymoney-master-4220-linux-gcc-x86_64.AppImage, on that the transaction form has a cyan background which is fine because you can still read the labels.
Comment 3 Paul Worrall 2025-04-21 10:32:29 UTC
Created attachment 180472 [details]
Screenshot showing colour of transaction form in kmymoney-master-4220-linux-gcc-x86_64.AppImage

Here's a screenshot showing the colour in kmymoney-master-4220-linux-gcc-x86_64.AppImage
Comment 4 Tcpsorcerer 2025-04-23 20:07:14 UTC
Created attachment 180584 [details]
Same bug on Windows 10

Similar issue observed on windows 10
Comment 5 Tcpsorcerer 2025-04-23 20:16:14 UTC
(In reply to Tcpsorcerer from comment #4)
> Created attachment 180584 [details]
> Same bug on Windows 10
> 
> Similar issue observed on windows 10

Forgot to add this is the Scheduled Transactions entry window
Comment 6 Thomas Baumgart 2025-04-24 05:59:02 UTC
*** Bug 503242 has been marked as a duplicate of this bug. ***
Comment 7 Thomas Baumgart 2025-04-26 13:21:33 UTC
Git commit a80a95c8fb848488e4c545ef56afe69aa2681e24 by Thomas Baumgart.
Committed on 26/04/2025 at 10:55.
Pushed by tbaumgart into branch 'master'.

Fix widget handling in transaction and split editor

Added an invisible column to the model which if not present
has the side effect when the left most column of the journal
or split model is moved right that the QTableView paints a
grid line in the area that is covered by the editor widget.

Synced the painting logic in journal and split delegate.

When first used, this will reset some settings of the
views to their defaults which was needed due to the
new column.
FIXED-IN: 5.2

M  +1    -0    kmymoney/mymoney/splitmodel.cpp
M  +6    -1    kmymoney/mymoney/splitmodel.h
M  +1    -0    kmymoney/mymoney/storage/journalmodel.cpp
M  +6    -1    kmymoney/mymoney/storage/journalmodel.h
M  +10   -1    kmymoney/settings/kmymoney.upd
M  +5    -14   kmymoney/views/journaldelegate.cpp
M  +1    -0    kmymoney/views/kpayeesview_p.h
M  +1    -0    kmymoney/views/ksearchtransactiondlg.cpp
M  +1    -0    kmymoney/views/ktagsview.cpp
M  +1    -0    kmymoney/views/ktransactionselectdlg.cpp
M  +19   -15   kmymoney/views/ledgerviewpage.cpp
M  +1    -1    kmymoney/views/ledgerviewpage.ui
M  +74   -73   kmymoney/views/splitdelegate.cpp
M  +2    -0    kmymoney/views/splitview.cpp

https://invent.kde.org/office/kmymoney/-/commit/a80a95c8fb848488e4c545ef56afe69aa2681e24
Comment 8 Paul Worrall 2025-05-01 08:14:24 UTC
Still getting this in Appimage 4297
Comment 9 Paul Worrall 2025-05-03 06:59:45 UTC
Fixed in kmymoney-master-4295-linux-gcc-x86_64.AppImage
Comment 10 Paul Worrall 2025-05-04 06:48:17 UTC
(In reply to Paul Worrall from comment #9)
> Fixed in kmymoney-master-4295-linux-gcc-x86_64.AppImage

Sorry, that should have been kmymoney-master-4301-linux-gcc-x86_64.AppImage