Skip to content

[FragmentStrictMode] Detect wrong Fragment container usage - #146

Closed
simonschiller wants to merge 1 commit into
androidx:androidx-mainfrom
simonschiller:feature/wrong-fragment-container
Closed

[FragmentStrictMode] Detect wrong Fragment container usage#146
simonschiller wants to merge 1 commit into
androidx:androidx-mainfrom
simonschiller:feature/wrong-fragment-container

Conversation

@simonschiller

Copy link
Copy Markdown
Contributor

Proposed Changes

  • Detect usage of Fragment containers other than FragmentContainerView

Testing

Test: See FragmentStrictModeTest#detectWrongFragmentContainer

Issues Fixed

Fixes: 181137036

@google-cla google-cla Bot added the cla: yes label Mar 22, 2021
@simonschiller
simonschiller force-pushed the feature/wrong-fragment-container branch from 8fbe22e to 23a24fc Compare March 23, 2021 08:06
@simonschiller
simonschiller force-pushed the feature/wrong-fragment-container branch from 23a24fc to cd84de8 Compare March 25, 2021 07:58
@dlam
dlam requested a review from jbw0033 March 25, 2021 19:11

@jbw0033 jbw0033 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you for doing all of these. We are ready to make the APIs public now. Do you want to a change for removing the RestrictTo() annotations?

@simonschiller

simonschiller commented Mar 30, 2021

Copy link
Copy Markdown
Contributor Author

Happy to help 😄
Sure, I can open a follow-up CL after this one is merged.

@simonschiller
simonschiller deleted the feature/wrong-fragment-container branch March 31, 2021 07:00
harryzz pushed a commit to harryzz/compose-multiplatform-core that referenced this pull request Jul 28, 2026
## Proposed Changes

  - Detect usage of `Fragment` containers other than `FragmentContainerView`

## Testing

Test: See `FragmentStrictModeTest#detectWrongFragmentContainer`

## Issues Fixed

Fixes: 181137036

This is an imported pull request from androidx#146.

Resolves JetBrains#146
Github-Pr-Head-Sha: cd84de8
GitOrigin-RevId: 02bc194
Change-Id: I86d8f2797c4e76957b36c31e91fb474e42c9ee1d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants