Skip to content

[stable-3.16] fix(theme): correct dark/light mode check on Win10 - #8162

Merged
nilsding merged 1 commit into
stable-3.16from
backport/8156/stable-3.16
Apr 15, 2025
Merged

[stable-3.16] fix(theme): correct dark/light mode check on Win10#8162
nilsding merged 1 commit into
stable-3.16from
backport/8156/stable-3.16

Conversation

@backportbot

@backportbot backportbot Bot commented Apr 15, 2025

Copy link
Copy Markdown

Backport of PR #8156

@backportbot
backportbot Bot requested review from claucambra and nilsding April 15, 2025 11:41
@backportbot backportbot Bot added this to the 3.16.3 milestone Apr 15, 2025
@nilsding
nilsding enabled auto-merge April 15, 2025 11:41
`Utility::registryGetKeyValue` returns an invalid QVariant should the
key not exist -- which is the case if that switch has never been toggled
before.

--> Fix this by ensuring the QVariant is valid, and only then try to
convert it to a bool.

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding
nilsding force-pushed the backport/8156/stable-3.16 branch from c82f3e9 to c9f0f42 Compare April 15, 2025 12:26
@github-actions

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-8162.zip

SHA256 checksum: f65c99d14a02dbd3b6a47ff2b94e3195ad8dd8acb1341623931efb21276df3f3

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@nilsding
nilsding merged commit 1f13988 into stable-3.16 Apr 15, 2025
@nilsding
nilsding deleted the backport/8156/stable-3.16 branch April 15, 2025 12:59
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant