Skip to content

[SystemChrome] setPreferredOrientations implementation added #79

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

Merged

Conversation

pkosko
Copy link

@pkosko pkosko commented Apr 30, 2021

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
(value) => print("preferred orientations were set"));

/// to clear previous limitations
List orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
(value) => print("preferred orientations were CLEARED"));

This PR is related to part of https://github.sec.samsung.net/f-project/home/issues/35:

Even though there's no corresponding Tizen system API for locking the app orientation, can't we somehow manage the desired orientation information in the embedder to use with TizenEmbedderEngine::SetWindowOrientation? I think we should investigate this particular method in more details because many apps may want to use a fixed orientation.

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
@pkosko pkosko requested a review from a team April 30, 2021 11:30
@pkosko
Copy link
Author

pkosko commented May 5, 2021

@pwasowski2 Thank you for you feedback. I've applied all your suggestions.

Copy link

@pwasowski2 pwasowski2 left a comment

Choose a reason for hiding this comment

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

LGTM 👏

@pkosko
Copy link
Author

pkosko commented May 6, 2021

@swift-kim I've renamed member name to tizen_renderer_ as you suggested.

@swift-kim swift-kim merged commit 9dfe48b into flutter-tizen:flutter-2.0.1-tizen May 6, 2021
@pkosko pkosko mentioned this pull request May 12, 2021
12 tasks
swift-kim pushed a commit that referenced this pull request Jun 7, 2021
* [SystemChrome] setPreferredOrientations implementation added

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));

* [SystemChrome] setPreferredOrientations implementation added - review fixes

* [SystemChrome] setPreferredOrientations implementation added - changed member name
swift-kim pushed a commit that referenced this pull request Sep 27, 2021
* [SystemChrome] setPreferredOrientations implementation added

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));

* [SystemChrome] setPreferredOrientations implementation added - review fixes

* [SystemChrome] setPreferredOrientations implementation added - changed member name
swift-kim pushed a commit that referenced this pull request Nov 14, 2021
* [SystemChrome] setPreferredOrientations implementation added

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));

* [SystemChrome] setPreferredOrientations implementation added - review fixes

* [SystemChrome] setPreferredOrientations implementation added - changed member name
swift-kim pushed a commit that referenced this pull request Dec 9, 2021
* [SystemChrome] setPreferredOrientations implementation added

[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));

* [SystemChrome] setPreferredOrientations implementation added - review fixes

* [SystemChrome] setPreferredOrientations implementation added - changed member name
swift-kim pushed a commit that referenced this pull request Dec 17, 2021
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
swift-kim pushed a commit that referenced this pull request Feb 7, 2022
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
swift-kim pushed a commit that referenced this pull request Feb 11, 2022
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
swift-kim pushed a commit that referenced this pull request May 12, 2022
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
swift-kim pushed a commit that referenced this pull request Aug 5, 2022
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
swift-kim pushed a commit that referenced this pull request Sep 1, 2022
Offset is a uint64_t that needs a cast.
swift-kim pushed a commit that referenced this pull request Sep 1, 2022
[Before] not implemented

[After] Using API below allows to narrow some rotations of application UI
e.g.
/// to limit some orientations
List<DeviceOrientation> orientations = [DeviceOrientation.portraitUp, DeviceOrientation.landscapeLeft];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were set"));

/// to clear previous limitations
List<DeviceOrientation> orientations = [];
SystemChrome.setPreferredOrientations(orientations).then(
    (value) => print("preferred orientations were CLEARED"));
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