-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feature: PausableComposition feature flag in CCGP #4546
base: master
Are you sure you want to change the base?
Conversation
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.
I made some suggestions to use active voice a bit more. Please let me know what you think about combining the notes :)
@@ -194,6 +194,27 @@ and functions that are implicitly not skippable (inline functions and functions | |||
> | |||
{style="warning"} | |||
|
|||
### PausableComposition | |||
|
|||
> The feature flag affects behavior only with a version of Compose runtime that supports pausable composition, |
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.
Why not combine the info and warning into one warning at the top? If the warning is at the bottom of the text then readers can miss it if they don't read through all the content. Or think the warning is applicable to the next section. WDYT? :)
Co-authored-by: Sarah Haggarty <[email protected]>
Mentioning the PausableComposition flag in the Compose Compiler Gradle plugin reference: https://youtrack.jetbrains.com/issue/KT-71227/Compose-Add-PausableComposition-feature-flag-to-the-Compose-Gradle-Plugin