Skip to content

Conversation

@malmstein
Copy link
Contributor

@malmstein malmstein commented Nov 4, 2025

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

  • Install internal flavour
  • Navigate to AI Features
  • Verify “Full Screen Mode” toggle is visible

Play flavour

  • Install Play flavour
  • Navigate to AI Features
  • Verify “Full Screen Mode” toggle is not visible

Copy link
Contributor Author

malmstein commented Nov 4, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@malmstein malmstein force-pushed the feature/david/11-04-duck.ai_fullscreen_mode_ff branch from f4d0082 to 23f4be4 Compare November 4, 2025 15:45
@malmstein malmstein marked this pull request as ready for review November 4, 2025 15:46
@malmstein malmstein force-pushed the feature/david/11-04-duck.ai_fullscreen_mode_ff branch from 23f4be4 to fd6759f Compare November 4, 2025 22:25

suspend fun setShowInAddressBar(showDuckChat: Boolean)

suspend fun setFullScreenModeUserSetting(enabled: Boolean)
Copy link
Contributor

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)
Copy link
Contributor

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()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs tests

Copy link
Contributor

@joshliebe joshliebe left a 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.

@malmstein malmstein merged commit 9d09622 into develop Nov 6, 2025
19 checks passed
@malmstein malmstein deleted the feature/david/11-04-duck.ai_fullscreen_mode_ff branch November 6, 2025 10:00
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