-
Notifications
You must be signed in to change notification settings - Fork 3k
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
mWeb - Subscription - Control plan modal does not have bottom padding #55113
Comments
Triggered auto assignment to @stephanieelliott ( |
Triggered auto assignment to @MariaHCD ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Offending PR #54771 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Control plan modal does not have bottom padding What is the root cause of that problem?The issue arises because both cards are styled to have the same overall height. However, the Collect card has 7 benefits, which fully utilize the available height, leaving no remaining space at the bottom. On the other hand, the Control card has 6 benefits, resulting in unused vertical space that display as additional bottom padding, creating an inconsistent appearance between the two cards. What changes do you think we should make in order to solve the problem?here
We can modify the condition to include multiple styles by wrapping them in an array, like this:
This ensures that both What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?What alternative solutions did you explore? (Optional)N/A |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.84-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): applausetester+010108kh@applause.expensifail.com
Issue reported by: Applause Internal Team
Device used: iPhone 15 Pro Max / iOS 18.2
App Component: Other
Action Performed:
Expected Result:
Control plan modal will have bottom padding.
Actual Result:
Control plan modal does not have bottom padding on mweb.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6711591_1736572550418.ScreenRecording_01-11-2025_13-10-27_1.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: