-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Duck.ai: Fullscreen mode FF #7055
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
Duck.ai: Fullscreen mode FF #7055
Conversation
f4d0082 to
23f4be4
Compare
23f4be4 to
fd6759f
Compare
|
|
||
| suspend fun setShowInAddressBar(showDuckChat: Boolean) | ||
|
|
||
| suspend fun setFullScreenModeUserSetting(enabled: Boolean) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs tests
|
|
||
| suspend fun setShowInAddressBar(showDuckChat: Boolean) | ||
|
|
||
| suspend fun setFullScreenModeUserSetting(enabled: Boolean) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs tests
| withContext(dispatchers.io()) { | ||
| duckChatFeatureRepository.setFullScreenModeUserSetting(enabled) | ||
| cacheUserSettings() | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs tests
joshliebe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested, LGTM. Just missing some tests.

Task/Issue URL: https://app.asana.com/1/137249556945/project/1174433894299346/task/1211834986565997?focus=true
Description
This PR adds a FF for the new full screen mode. No functionality has been added to it.
Steps to test this PR
Internal flavour
Play flavour