Skip to content

Fix safe insets on full screen#1613

Merged
vegaro merged 10 commits into
mainfrom
safe-insets
Feb 15, 2024
Merged

Fix safe insets on full screen#1613
vegaro merged 10 commits into
mainfrom
safe-insets

Conversation

@vegaro

@vegaro vegaro commented Feb 14, 2024

Copy link
Copy Markdown
Member
Before After
template_1 template_1_fix
template_2 template_2_fix
template_3 template_3_fix
template_4 template_4_fix
template_5 template_5_fix

@vegaro vegaro added the pr:fix A bug fix label Feb 14, 2024
@vegaro vegaro marked this pull request as ready for review February 14, 2024 13:43

@tonidero tonidero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left some comments but this is looking good!

// For extending content behind the Navigation Bar as well
window.addFlags(WindowManager.LayoutParams.FLAG_LAYOUT_NO_LIMITS)

WindowCompat.setDecorFitsSystemWindows(window, false)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm actually wondering if we should make the PaywallActivity in the sdk to paint over the status bar... But that can come in a different PR if we decide to do that.

@vegaro vegaro requested a review from a team February 14, 2024 14:33
vegaro and others added 3 commits February 14, 2024 15:37
…venuecatui/templates/Template4.kt

Co-authored-by: Toni Rico <toni.rico.diez@revenuecat.com>
@vegaro vegaro requested a review from tonidero February 15, 2024 15:11

@tonidero tonidero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great!

@vegaro vegaro enabled auto-merge (squash) February 15, 2024 16:32
@vegaro vegaro merged commit e6c4047 into main Feb 15, 2024
@vegaro vegaro deleted the safe-insets branch February 15, 2024 16:49
tonidero pushed a commit that referenced this pull request Feb 22, 2024
**This is an automatic release.**

### New Features
* Add setDisplayDismissButton to PaywallView (#1608) via Cesar de la
Vega (@vegaro)
### Bugfixes
* Fix runtime crash when using amazon and targetting android 14 (#1622)
via Toni Rico (@tonidero)
* Paywalls: No-op on all view model methods in the Loading paywall
screen (#1617) via Toni Rico (@tonidero)
* Fix safe insets on full screen (#1613) via Cesar de la Vega (@vegaro)
### Other Changes
* Change default paywall background with a lower quality image (#1623)
via Toni Rico (@tonidero)
* Fix load shedder integration tests (#1618) via Cesar de la Vega
(@vegaro)
* Refactor `presentedOfferingIdentifier` into `presentedOfferingContext`
object (#1612) via Toni Rico (@tonidero)
* [External] Update russian and kazakh translations (#1577 by @janbolat)
(#1616) via Toni Rico (@tonidero)

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants