Skip to content

Split - No bottom margin for date split option #78048

Closed

Description

@vincdargento

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: v9.2.82-0
Reproducible in staging?: Yes
Reproducible in production?: N/A - new feature, doesn't exist in prod
If this was caught during regression testing, add the test name, ID and link from BrowserStack: #74652
Email or phone of affected tester (no customers): testpayment935@gmail.com
Issue reported by: Applause Internal Team
Bug source: Pull Request QA execution
Device used: Windows 11 pro /Chrome
App Component: Money Requests

Action Performed:

  1. Navigate to staging.new.expensify.com
  2. Create workspace and add expense
  3. Go to expense detail page and split the workspace using date ( have at least 10 days apart)
  4. Notice the bottom margin between save button and last split item

Expected Result:

There should be bottom margin between save button and last split item ( Amount and percentage option behavior)

Actual Result:

There is no bottom margin between save button and last split item

Workaround:

Unknown

Platforms:

  • Android: App
  • Android: mWeb Chrome
  • iOS: App
  • iOS: mWeb Safari
  • iOS: mWeb Chrome
  • Windows: Chrome
  • MacOS: Chrome / Safari

Screenshots/Videos

bug.mp4

View all open jobs on GitHub

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    on Dec 18, 2025
  2. melvin-bot commented on Dec 18, 2025

    @melvin-bot

    Triggered auto assignment to @bfitzexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

  3. melvin-bot commented on Dec 18, 2025

    @melvin-bot

    Triggered auto assignment to @Beamanator (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  4. github-actions commented on Dec 18, 2025

    @github-actions
    Contributor

    👋 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:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  5. melvin-bot commented on Dec 18, 2025

    @melvin-bot

    💬 A slack conversation has been started in #expensify-open-source

  6. ZhenjaHorbach commented on Dec 18, 2025

    @ZhenjaHorbach
    Contributor

    Minor issue
    Will fix in the next improvement PR

  7. Beamanator commented on Dec 18, 2025

    @Beamanator
    Contributor

    I agree this isn't a blocker 👍

  8. added and removed
    DeployBlockerCashThis issue or pull request should block deployment
    on Dec 18, 2025
  9. Beamanator commented on Dec 18, 2025

    @Beamanator
    Contributor

    @ZhenjaHorbach can you post here which PR this came from just so it's documented? again, this isn't a blocker for sure, just curious to link where it came from

  10. ZhenjaHorbach commented on Dec 18, 2025

    @ZhenjaHorbach
    Contributor

    @ZhenjaHorbach can you post here which PR this came from just so it's documented? again, this isn't a blocker for sure, just curious to link where it came from

    For the dates feature, we have this PR

  11. garrettmknight commented on Jan 2, 2026

    @garrettmknight
    Contributor

    Fixed! Handling payment in a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.EngineeringReviewingHas a PR in reviewWeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions