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.

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

docs: Add SystemThemeHelper page #1330

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

morning4coffe-dev
Copy link
Member

GitHub Issue (If applicable): part of unoplatform/uno#19235, #1324

PR Type

What kind of change does this PR introduce?

What is the new behavior?

PR Checklist

Please check if your PR fulfills the following requirements:

- [`AncestorBinding` and `ItemsControlBinding`](helpers/ancestor-itemscontrol-binding.md): These markup extensions provides relative binding based on ancestor type. If you are familiar with WPF, they are very similar to `{RelativeSource Mode=FindAncestor}`.
- [`CommandExtensions`](helpers/command-extensions.md): Provides Command/CommandParameter attached properties for common scenarios.
- [`InputExtensions`](helpers/input-extensions.md): Provides various attached properties for _input controls_, such as `TextBox` and `PasswordBox`.
- [`ItemsRepeaterExtensions`](helpers/itemsrepeater-extensions.md): Provides selection support for ItemsRepeater.
- [`StatusBar`](helpers/StatusBar-extensions.md): Provides two attached properties on `Page` to controls the visual of the status bar on mobile platforms.
- [`SystemThemeHelper`](helpers/SystemThemeHelper.md): Provides a set of helper methods to check the current operating system theme, and manipulate the application dark/light theme.
Copy link
Contributor

Choose a reason for hiding this comment

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

All of these should be using xref instead of relative routes, no?

We can address that in a separate PR


| Method | Return Type | Description |
|------------------------------|-------------------------|---------------------------------------------------------------------------------------------------------------|
| `GetCurrentOsTheme()` | `ApplicationTheme` | Retrieves the current theme of the operating system, based on the background color. |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `GetCurrentOsTheme()` | `ApplicationTheme` | Retrieves the current theme of the operating system, based on the background color. |
| `GetCurrentOsTheme()` | `ApplicationTheme` | Retrieves the current theme of the operating system. |

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