Skip to content

Conversation

@swift-kim
Copy link
Member

The SystemChrome.setSystemUIOverlayStyle method is called when an app restarts, but similarly to #308, the relevant features are not supported by the Tizen platform and thus calling the method will have no effect.

If a particular style is not supported on the platform, selecting it will have no effect.

Copy link
Member

@JSUYA JSUYA left a comment

Choose a reason for hiding this comment

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

Are there any more cases like this?

how about this?

} else if (method == kSetApplicationSwitcherDescriptionMethod || method == kSetSystemUIOverlayStyleMethod) {
    // Not supported on Tizen. Ignore.
    result->Success();
}

I think, since we use the same comment and call Success(), it would be good to combine if statements.

@swift-kim
Copy link
Member Author

Are there any more cases like this?

No, they are all. Please refer to flutter-tizen/embedder#11. Note that SystemChrome.setEnabledSystemUIMode is currently partially implemented.

how about this?

Sure, I will do so.

@bbrto21 bbrto21 merged commit 6c8e921 into flutter-tizen:flutter-3.3.0-tizen Sep 19, 2022
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.

3 participants