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

2 participants