Skip to content

[Due for payment 2026-10-16] Distance rate - No spacing between currency and amount when inline editing rate #103108

Closed

Description

@applause-bot

If you haven’t already, check out our contributing guidelines for onboarding. To join our Slack channel, fill out this form.


Version Number: 9.5.3-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: #100567
Email or phone of affected tester (no customers): jsdiojsdnidjos@gmail.com
Issue reported by: Applause Internal Team
Bug source: Exploratory - Significant User Experience Deterioration
Device used: Mac 26.5 / Chrome
App Component: Workspace Settings

Action Performed:

Precondition:

  • Enable Distance rates.
  1. Go to staging.new.expensify.com
  2. Go to workspace settings > Distance rates.
  3. Inline edit rate.

Expected Result:

There will be spacing between currency and amount when inline editing rate (similar to inline editing amount on expense report).

Actual Result:

There is no spacing between currency and amount when inline editing rate.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

To be added by Applause Leads

View all open jobs on GitHub

Applause Internal Information
Exported by: Mita Rachim
Bug ID: 7278339

Issue OwnerCurrent Issue Owner: @ahmedGaber93

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    on Oct 6, 2026
  2. applause-bot commented on Oct 6, 2026

    @applause-bot
    Author

    While Applause doesn't embed attachments for this bug, here are the links for quick view:
    https://platform.applause.com/services/links/v1/external/5d480531f6f95691fbca8cecede031c2990455ad862fc3a14fff8744335c819a

  3. melvin-bot commented on Oct 6, 2026

    @melvin-bot

    You have been assigned to this deploy blocker because you recently merged this PR: #100567

  4. melvin-bot commented on Oct 6, 2026

    @melvin-bot

    @JS00001 @Krishna2323 you have been assigned to this deploy blocker because you reviewed the offending PR: #100567

  5. melvin-bot commented on Oct 6, 2026

    @melvin-bot

    @ahmedGaber93 you have been assigned to this deploy blocker because you authored the offending PR: #100567

  6. melvin-bot commented on Oct 6, 2026

    @melvin-bot

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

  7. github-actions commented on Oct 6, 2026

    @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.
  8. MelvinBot commented on Oct 6, 2026

    @MelvinBot
    Contributor

    Deploy Blocker Analysis

    Causing PR: feat: support inline editing for workspace settings tables (high confidence). It added the new InlineNumberEditCell that the distance-rate inline editor uses.

    Root cause: InlineNumberEditCell passes only styles.editableCellSymbolStyle to the symbol (InlineNumberEditCell.tsx:118). The expense-report editor it was modeled on first checks whether the currency format contains a space. If so, it adds styles.pr1 (TotalCell.tsx:146-151, TotalCell.tsx:194). The new cell dropped that check.

    Recommended fix (roll forward, don't revert): Move the hasSymbolSpaceInPreview logic into a shared helper in CurrencyUtils. Use it in both cells:

    symbolTextStyle={[styles.editableCellSymbolStyle, hasSymbolSpace && styles.pr1]}

    This also fixes the same gap in per diem amounts and Expensify Card limits, which use the same cell.

    Blocker status: The bug is cosmetic and only affects admins on web. It shows only for currency/locale pairs whose format includes a space (USD in English has none). Saved values are correct. Reverting would remove inline editing from about 85 files, so a two-line fix from ahmedGaber93 is the cheaper path. Demoting is also reasonable.

    Investigation details

    view run

  9. 2 remaining items

  10. JS00001 commented on Oct 6, 2026

    @JS00001
    Contributor

    not a blocker, demoting

  11. changed the title [-]Distance rate - No spacing between currency and amount when inline editing rate[/-] [+][Due for payment 2026-10-16] Distance rate - No spacing between currency and amount when inline editing rate[/+] on Oct 9, 2026
  12. melvin-bot commented on Oct 9, 2026

    @melvin-bot

    @ahmedGaber93 / @Krishna2323

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.5.6-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2026-10-16. 🎊

    The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the Contributor+ Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]. If no checklist is needed for this issue, you can click: [no checklist button]

  13. Krishna2323 commented on Oct 11, 2026

    @Krishna2323
    Contributor

    Closing this one, fix is on prod.

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.DailyKSv2Engineering

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions