[No QA] Docs updates for E/A#92546 multi-expense preview per-expense navigation and View button - #97142
[No QA] Docs updates for E/A#92546 multi-expense preview per-expense navigation and View button#97142MelvinBot wants to merge 1 commit into
Conversation
Co-authored-by: wildan-m <wildan-m@users.noreply.github.com>
|
A preview of your ExpensifyHelp changes have been deployed to https://f54e160b.helpdot.pages.dev ⚡️ Updated articles: |
|
|
||
| - Click an individual expense in the preview to open that expense. | ||
| - On a wide screen, the expense opens in the right-hand pane. | ||
| - On a narrow screen (mobile or mWeb), the expense opens on top of the report, so pressing back returns you to the report. |
There was a problem hiding this comment.
Internal/jargon language: "mWeb" is internal terminology and should not appear in a HelpDot article (HELP_AUTHORING_GUIDELINES.md Section 6 — "Avoid internal language"). Use plain user-facing wording, e.g. "On a narrow screen (mobile web or the mobile app)".
Also, "right-hand pane" (line 35) is internal layout jargon. Prefer describing behavior in user terms rather than the internal RHP name.
HelpDot Documentation ReviewOverall AssessmentThis PR makes a small, focused documentation update to Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: This review is based only on the proposed diff, not on unchanged content in the file. |
|
I don't think this needs to be documented. |
Explanation of Change
This documents the behavior changes from #92546 in the multi-expense Report Previews section of the Getting Started with the Spend Page article.
PR #92546 changed how the multi-expense report preview behaves:
What changed in the article
I verified the button label against the code: it uses
common.view→ View.@wildan-m, please review the files changed and confirm they reflect the current behavior. Then post a new comment saying
@MelvinBot mark this PR as ready for review.