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

[Paywalls V2] Fix footer positioning and bottom padding #4746

Merged

Conversation

joshdholtz
Copy link
Member

Motivation

Fixes some margin and padding issues in the sticky footer

Description

  • Removed a Spacer() that was causing some margin on the bottom to show
  • Fixed internal padding by applying padding of the bottom safe area inset
Before After
image Simulator Screenshot - iPhone 16 - 2025-01-29 at 20 57 34

Copy link

emerge-tools bot commented Jan 30, 2025

1 build decreased size

Name Version Download Change Install Change Approval
Paywalls
com.revenuecat.PaywallsTester
1.0 (1) 10.8 MB ⬇️ 295 B 40.8 MB ⬇️ 132 B N/A

Paywalls 1.0 (1)
com.revenuecat.PaywallsTester

⚖️ Compare build
📦 Install build
⏱️ Analyze build performance

Total install size change: ⬇️ 132 B
Total download size change: ⬇️ 295 B

Largest size changes

Item Install Size Change
DYLD.String Table ⬇️ -2.6 kB
Other ⬆️ 2.5 kB
View Treemap

Image of diff


🛸 Powered by Emerge Tools

@joshdholtz joshdholtz changed the base branch from main to paywalls-v2/add-full-cover-option-to-paywalls-tester January 30, 2025 03:24
Copy link

emerge-tools bot commented Jan 30, 2025

📸 Snapshot Test

2 modified, 169 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
RevenueCat
com.revenuecat.PaywallsTester
0 0 2 0 169 0 ⏳ Needs approval

🛸 Powered by Emerge Tools

@joshdholtz joshdholtz force-pushed the paywalls-v2/add-full-cover-option-to-paywalls-tester branch from eec3175 to cc82b4f Compare January 30, 2025 03:46
@joshdholtz joshdholtz force-pushed the paywalls-v2/fix-footer-bottom-positioning-and-padding branch from 962c5e2 to daa202d Compare January 30, 2025 03:47
Copy link
Member

@JayShortway JayShortway left a comment

Choose a reason for hiding this comment

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

LGTM!

.edgesIgnoringSafeArea(.top)
})
// This is normal
Copy link
Member

Choose a reason for hiding this comment

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

This is fine 🐶 ☕ 🔥

@joshdholtz joshdholtz force-pushed the paywalls-v2/add-full-cover-option-to-paywalls-tester branch from cc82b4f to 1e2e423 Compare January 30, 2025 10:54
@joshdholtz joshdholtz force-pushed the paywalls-v2/fix-footer-bottom-positioning-and-padding branch from daa202d to d15816a Compare January 30, 2025 10:59
@joshdholtz joshdholtz marked this pull request as ready for review January 30, 2025 11:00
@joshdholtz joshdholtz force-pushed the paywalls-v2/add-full-cover-option-to-paywalls-tester branch from 1e2e423 to 8abb7fa Compare January 30, 2025 11:06
Base automatically changed from paywalls-v2/add-full-cover-option-to-paywalls-tester to main January 30, 2025 11:35
@joshdholtz joshdholtz force-pushed the paywalls-v2/fix-footer-bottom-positioning-and-padding branch from d15816a to 16f911b Compare January 30, 2025 11:35
@joshdholtz joshdholtz merged commit cf8ae65 into main Jan 30, 2025
9 of 10 checks passed
@joshdholtz joshdholtz deleted the paywalls-v2/fix-footer-bottom-positioning-and-padding branch January 30, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants