Skip to content

feat: Integrate reactive Dark Mode detection for all desktop OS#368

Merged
leandroBorgesFerreira merged 5 commits intoWriteopia:mainfrom
kdroidFilter:main
May 7, 2025
Merged

feat: Integrate reactive Dark Mode detection for all desktop OS#368
leandroBorgesFerreira merged 5 commits intoWriteopia:mainfrom
kdroidFilter:main

Conversation

@kdroidFilter
Copy link
Copy Markdown
Contributor

feat: Integrate reactive Dark Mode detection for all desktop OS and adaptive top bar on Windows

  • Replace the use of isSystemInDarkTheme with isSystemInDarkMode to handle real-time theme changes across all desktop operating systems.
  • Add Windows top bar configuration using setWindowsAdaptiveTitleBar().
  • Currently, the top bar only follows the system theme and does not sync with the user's explicit preferences.

more information at https://github.com/kdroidFilter/Platform-Tools?tab=readme-ov-file#-dark-mode-detection-module

…daptive top bar on Windows

- Replace the use of isSystemInDarkTheme with isSystemInDarkMode to handle real-time theme changes across all desktop operating systems.
- Add Windows top bar configuration using setWindowsAdaptiveTitleBar().
- Currently, the top bar only follows the system theme and does not sync with the user's explicit preferences.

more information at https://github.com/kdroidFilter/Platform-Tools?tab=readme-ov-file#-dark-mode-detection-module
@leandroBorgesFerreira
Copy link
Copy Markdown
Collaborator

Thanks for your contribution! It will be nice that now the bars in Linux and Windows will follow the system's colour.

@kdroidFilter
Copy link
Copy Markdown
Contributor Author

it seems my library is having problems with pro guards, i will inspect the problem

Bumped platformtoolsDarkmodedetector to 0.2.8 and downgraded AGP to 8.6.1 to ensure compatibility and stability. Changes help maintain synchronization between dependencies.
@kdroidFilter
Copy link
Copy Markdown
Contributor Author

I fixed the problem with Proguard, can we try again?

Comment thread gradle/libs.versions.toml Outdated
@leandroBorgesFerreira
Copy link
Copy Markdown
Collaborator

Hi @kdroidFilter. I think this is just fine. Please remove the downgrade and fix the Ktlint change and we can merge this 🎉 🎉 🎉

Replaced `isSystemInDarkTheme()` with `isSystemInDarkMode()` for dark mode detection using the `platformtools` library. Updated dependency version to `0.2.9` and replaced references in theme configuration and desktop app setup. Ensure compatibility and alignment with the updated library API.
@kdroidFilter
Copy link
Copy Markdown
Contributor Author

it should be good :)

@leandroBorgesFerreira
Copy link
Copy Markdown
Collaborator

Hi @kdroidFilter Somehow I didn't see your message, sorry about it.

@leandroBorgesFerreira
Copy link
Copy Markdown
Collaborator

leandroBorgesFerreira commented May 7, 2025

Hey! Thanks for the help with the colour theme of the desktops apps and congrats on your first contribution!! 🎉 🎉 🎉 🎉 🎉 🎉 🎉

@leandroBorgesFerreira leandroBorgesFerreira merged commit cd91000 into Writeopia:main May 7, 2025
7 checks passed
@kdroidFilter
Copy link
Copy Markdown
Contributor Author

With pleasure, sorry for the wait :(

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants