Skip to content

Revert "Reduce left margin on the multi-expense report view" - #99076

Merged
puneetlath merged 1 commit into
mainfrom
revert-97817-claude-reduceReportViewLeftMargin
Aug 20, 2026
Merged

Revert "Reduce left margin on the multi-expense report view"#99076
puneetlath merged 1 commit into
mainfrom
revert-97817-claude-reduceReportViewLeftMargin

Conversation

@Beamanator

Copy link
Copy Markdown
Contributor

Reverts #97817

Fixes Issues
$ #99035

@Beamanator
Beamanator requested review from a team as code owners August 20, 2026 14:46
@melvin-bot
melvin-bot Bot requested review from JmillsExpensify and arosiclair and removed request for a team August 20, 2026 14:46
@melvin-bot

melvin-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

@arosiclair Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ Changes either increased or maintained existing code coverage, great job!

Files with missing lines Coverage Δ
src/styles/variables.ts 100.00% <ø> (ø)
...gation/helpers/calculateSuperWideRHPWidth/index.ts 0.00% <0.00%> (ø)
... and 7 files with indirect coverage changes

@puneetlath
puneetlath merged commit 36f6a4c into main Aug 20, 2026
45 of 46 checks passed
@puneetlath
puneetlath deleted the revert-97817-claude-reduceReportViewLeftMargin branch August 20, 2026 15:07
@github-actions

Copy link
Copy Markdown
Contributor

🚧 puneetlath has triggered a test Expensify/App build. You can view the workflow run here.

@OSBotify

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

OSBotify pushed a commit that referenced this pull request Aug 20, 2026
…portViewLeftMargin

Revert "Reduce left margin on the multi-expense report view"

(cherry picked from commit 36f6a4c)

(cherry-picked to staging by Beamanator)
@OSBotify OSBotify added the CP Staging marks PRs that have been CP'd to staging label Aug 20, 2026
@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/Beamanator in version: 9.4.57-1 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 cancelled 🔪
🍎 iOS 🍎 cancelled 🔪

@MelvinBot

Copy link
Copy Markdown
Contributor

🤖 No help site changes required.

This PR is a pure revert of #97817. It only touches internal layout/styling code for the Right Hand Panel (RHP) width calculation on the multi-expense report view:

  • src/libs/Navigation/helpers/calculateSuperWideRHPWidth/index.ts — reverts the width formula
  • src/styles/variables.ts — removes the superWideRHPLeftMargin variable

These are visual spacing changes with no user-facing feature, setting, tab, or workflow behavior. Help site articles under App/docs/articles document features and workflows, not pixel-level layout/margin calculations, and a search of the articles for the relevant terms (e.g. "left margin", "multi-expense report view", "RHP") returned no matches. No draft docs PR was created.

@Beamanator, please review the linked help site PR and confirm it reflects the current behavior. Then mark the linked help site PR Ready for review


view run

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.4.57-3 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/Beamanator in version: 9.4.58-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.4.58-4 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

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

Labels

CP Staging marks PRs that have been CP'd to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants