Skip to content

Combine multiple UIView sources to get window insets#2946

Merged
svastven merged 3 commits intojb-mainfrom
svastven/window-insets
Apr 10, 2026
Merged

Combine multiple UIView sources to get window insets#2946
svastven merged 3 commits intojb-mainfrom
svastven/window-insets

Conversation

@svastven
Copy link
Copy Markdown

@svastven svastven commented Apr 7, 2026

Combine multiple UIView sources to get correct window insets. Fixes support for embedding compose views in UIKit navigation view with UIKit navigation bar and iPad insets for traffic light controls.

Fixes CMP-9931 iOS: WindowInsets.safeDrawing does not include UINavigationBar safe area insets (regression in 1.11.0-alpha04)

Testing

Test CMP-8922 on iPad in combination with sample provided in CMP-9931

This should be tested by QA

Release Notes

Fixes - iOS

  • (prerelease fix) Fix UINavigationBar safe area insets not included in WindowInsets.safeDrawing

@svastven svastven force-pushed the svastven/window-insets branch from fe0c81e to 52217da Compare April 8, 2026 19:10
@svastven svastven requested a review from ASalavei April 8, 2026 19:10
Copy link
Copy Markdown

@ASalavei ASalavei left a comment

Choose a reason for hiding this comment

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

LGTM, waiting for green tests

@svastven svastven force-pushed the svastven/window-insets branch from 52217da to 5f4f652 Compare April 9, 2026 14:54
@svastven svastven requested a review from ASalavei April 9, 2026 19:12
@svastven svastven merged commit 42d25ff into jb-main Apr 10, 2026
17 checks passed
@svastven svastven deleted the svastven/window-insets branch April 10, 2026 09:24
svastven added a commit that referenced this pull request Apr 10, 2026
Combine multiple UIView sources to get correct window insets. Fixes
support for embedding compose views in UIKit navigation view with UIKit
navigation bar and iPad insets for traffic light controls.

Fixes [CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931) iOS:
WindowInsets.safeDrawing does not include UINavigationBar safe area
insets (regression in 1.11.0-alpha04)

Test [CMP-8922](https://youtrack.jetbrains.com/issue/CMP-8922) on iPad
in combination with sample provided in
[CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931)

This should be tested by QA

- _(prerelease fix)_ Fix `UINavigationBar` safe area insets not included
in `WindowInsets.safeDrawing`
svastven added a commit that referenced this pull request Apr 10, 2026
Combine multiple UIView sources to get correct window insets. Fixes
support for embedding compose views in UIKit navigation view with UIKit
navigation bar and iPad insets for traffic light controls.

Fixes [CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931) iOS:
WindowInsets.safeDrawing does not include UINavigationBar safe area
insets (regression in 1.11.0-alpha04)

Test [CMP-8922](https://youtrack.jetbrains.com/issue/CMP-8922) on iPad
in combination with sample provided in
[CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931)

This should be tested by QA

- _(prerelease fix)_ Fix `UINavigationBar` safe area insets not included
in `WindowInsets.safeDrawing`
sekater added a commit that referenced this pull request Apr 10, 2026
…2946) (#2965)

Combine multiple UIView sources to get correct window insets. Fixes
support for embedding compose views in UIKit navigation view with UIKit
navigation bar and iPad insets for traffic light controls.

Fixes [CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931) iOS:
WindowInsets.safeDrawing does not include UINavigationBar safe area
insets (regression in 1.11.0-alpha04)

## Testing

Test [CMP-8922](https://youtrack.jetbrains.com/issue/CMP-8922) on iPad
in combination with sample provided in
[CMP-9931](https://youtrack.jetbrains.com/issue/CMP-9931)

This should be tested by QA

## Release Notes
### Fixes - iOS
- _(prerelease fix)_ Fix `UINavigationBar` safe area insets not included
in `WindowInsets.safeDrawing`
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.

3 participants