Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert dark mode regressions on Windows 10 #7463

Merged
merged 11 commits into from
Nov 19, 2024
Merged

Revert dark mode regressions on Windows 10 #7463

merged 11 commits into from
Nov 19, 2024

Conversation

claucambra
Copy link
Collaborator

@claucambra claucambra commented Nov 5, 2024 •
edited
Loading

Closes #7269

@claucambra claucambra self-assigned this Nov 5, 2024
@claucambra claucambra force-pushed the bugfix/revert-windows-dark-regressions branch 3 times, most recently from 06d2cca to a7e2afd Compare November 5, 2024 08:28
@claucambra
Copy link
Collaborator Author

/backport to stable-3.14

@claucambra claucambra added this to the 3.14.5 milestone Nov 5, 2024
@claucambra claucambra force-pushed the bugfix/revert-windows-dark-regressions branch from a7e2afd to dbf4d1d Compare November 5, 2024 08:49
mgallien
mgallien requested changes Nov 8, 2024
View reviewed changes
Copy link
Collaborator

@mgallien mgallien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did not work
still getting this
image
settings dialog stays light mode
image

mgallien
mgallien reviewed Nov 8, 2024
View reviewed changes
src/libsync/theme.cpp Outdated Show resolved Hide resolved
@claucambra claucambra force-pushed the bugfix/revert-windows-dark-regressions branch from d3cf8e6 to f913087 Compare November 10, 2024 13:40
@Rello Rello modified the milestones: 3.14.5, 3.15.0 Nov 11, 2024
@mgallien mgallien force-pushed the bugfix/revert-windows-dark-regressions branch from f913087 to 579109d Compare November 12, 2024 13:17
stertingen
stertingen reviewed Nov 12, 2024
View reviewed changes
src/gui/main.cpp Outdated Show resolved Hide resolved
@mgallien mgallien force-pushed the bugfix/revert-windows-dark-regressions branch 2 times, most recently from 151db0d to 29c57e6 Compare November 13, 2024 14:35
@mgallien mgallien force-pushed the bugfix/revert-windows-dark-regressions branch 2 times, most recently from 88830e9 to d47ef20 Compare November 19, 2024 13:01
@mgallien
use QWidget Fusion style on Windows 10: enable dark mode
194f918 
Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien
create a log category for OCC::Theme class
bce6111 
Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien
implement a drak mode palette and use it when needed
1399a32 
for now unable to dynamically switch between light and dark mode

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien
on Windows 10 follow color scheme changes and apply them
1a9ba18 
Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien
some improvements for account network settings
011fcd0 
Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the bugfix/revert-windows-dark-regressions branch from d47ef20 to 011fcd0 Compare November 19, 2024 13:09
@mgallien mgallien merged commit 13e07fa into master Nov 19, 2024
9 of 14 checks passed
@mgallien mgallien deleted the bugfix/revert-windows-dark-regressions branch November 19, 2024 13:13
@backportbot backportbot bot mentioned this pull request Nov 19, 2024
Merged
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-7463-011fcd04f7b441fc8020ee92c1b2ae7bc88b442b-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

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

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
8.1% Coverage on New Code (required ≥ 80%)
24 New Code Smells (required ≤ 0)
B Maintainability Rating on New Code (required ≥ A)
D Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

mgallien added a commit that referenced this pull request Nov 19, 2024
@mgallien
Merge pull request #7463 from nextcloud/bugfix/revert-windows-dark-re…
9034e8a 
…gressions

Revert dark mode regressions on Windows 10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stertingen stertingen stertingen left review comments

@mgallien mgallien mgallien approved these changes

@camilasan camilasan Awaiting requested review from camilasan

Assignees

@claucambra claucambra

Labels
os: 🚪 Windows
Projects
None yet
Milestone
3.15.0
Development

Successfully merging this pull request may close these issues.

[Bug]: Nextcloud UI (3.14.1) is broken on Windows 10 (21H2 19044.4894) using dark mode
5 participants
@claucambra @nextcloud-desktop-bot @mgallien @stertingen @Rello