Skip to content
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

Commonize insets Modifier's #837

Merged
merged 3 commits into from
Sep 25, 2023
Merged

Conversation

MatkovIvan
Copy link
Member

Proposed Changes

+fun Modifier.safeDrawingPadding(): Modifier
+fun Modifier.safeGesturesPadding(): Modifier
+fun Modifier.safeContentPadding(): Modifier
+fun Modifier.systemBarsPadding(): Modifier
+fun Modifier.displayCutoutPadding(): Modifier
+fun Modifier.statusBarsPadding(): Modifier
+fun Modifier.imePadding(): Modifier
+fun Modifier.navigationBarsPadding(): Modifier
+fun Modifier.captionBarPadding(): Modifier
+fun Modifier.waterfallPadding(): Modifier
+fun Modifier.systemGesturesPadding(): Modifier
+fun Modifier.mandatorySystemGesturesPadding(): Modifier

Testing

Test: TBD

Issues Fixed

Fixes JetBrains/compose-multiplatform#3563

@MatkovIvan MatkovIvan reopened this Sep 22, 2023
@MatkovIvan MatkovIvan marked this pull request as ready for review September 25, 2023 07:17
@MatkovIvan MatkovIvan merged commit 46c0eaa into jb-main Sep 25, 2023
3 checks passed
@MatkovIvan MatkovIvan deleted the ivan.matkov/common-padding-modifiers branch September 25, 2023 07:17
@MatkovIvan MatkovIvan restored the ivan.matkov/common-padding-modifiers branch September 25, 2023 09:30
@MatkovIvan MatkovIvan deleted the ivan.matkov/common-padding-modifiers branch September 25, 2023 09:31
igordmn pushed a commit that referenced this pull request Nov 15, 2023
## Proposed Changes

```diff
+fun Modifier.safeDrawingPadding(): Modifier
+fun Modifier.safeGesturesPadding(): Modifier
+fun Modifier.safeContentPadding(): Modifier
+fun Modifier.systemBarsPadding(): Modifier
+fun Modifier.displayCutoutPadding(): Modifier
+fun Modifier.statusBarsPadding(): Modifier
+fun Modifier.imePadding(): Modifier
+fun Modifier.navigationBarsPadding(): Modifier
+fun Modifier.captionBarPadding(): Modifier
+fun Modifier.waterfallPadding(): Modifier
+fun Modifier.systemGesturesPadding(): Modifier
+fun Modifier.mandatorySystemGesturesPadding(): Modifier
```

## Testing

Test: TBD

## Issues Fixed

Fixes JetBrains/compose-multiplatform#3563
igordmn pushed a commit that referenced this pull request Nov 16, 2023
…r's (#837)

## Proposed Changes

```diff
+fun Modifier.safeDrawingPadding(): Modifier
+fun Modifier.safeGesturesPadding(): Modifier
+fun Modifier.safeContentPadding(): Modifier
+fun Modifier.systemBarsPadding(): Modifier
+fun Modifier.displayCutoutPadding(): Modifier
+fun Modifier.statusBarsPadding(): Modifier
+fun Modifier.imePadding(): Modifier
+fun Modifier.navigationBarsPadding(): Modifier
+fun Modifier.captionBarPadding(): Modifier
+fun Modifier.waterfallPadding(): Modifier
+fun Modifier.systemGesturesPadding(): Modifier
+fun Modifier.mandatorySystemGesturesPadding(): Modifier
```

## Testing

Test: TBD

## Issues Fixed

Fixes JetBrains/compose-multiplatform#3563
copybara-service bot pushed a commit to androidx/androidx that referenced this pull request May 15, 2024
It's upstreaming of JetBrains#837.
This API is already available from common since Compose Multiplatform 1.5.10

Test: N/A
Relnote: Make WindowInsetsPadding modifiers available from common source set
Change-Id: I070b2fbfc6dcae41be4bc9bf313d5847065d4053
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants