Conversation
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
@dubielzyk-expensify @jayeshmangwani One of you needs to 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] |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 67c2b27dc0
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Hey 1. Using 2. Unit tests The PR doesn't include any test files, but the author checklist mentions adding unit tests for bug fixes. The Enter key + switch fix in 3. Mention colors look good The 4. Type rename is clean Renaming Overall the changes are focused and address the issues well. |
|
Images look good. I'm gonna run a test build to test further 👍 |
|
🚧 @dubielzyk-expensify has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|


Explanation of Change
This PR fixes four bugs introduced by PR #85197 (High Contrast theme):
mentionText: colors.blue700, mentionBG: colors.blue100todark-contrast.ts:Result:
Enter key doesn't toggle high contrast switch (Preference - Unable to turn on the high contrast mode toggle using enter key #86346): The
disableEnterShortcutguard in BaseSelectionList only checked for button and checkbox roles. When the Switch (role=switch) had focus inside the SelectionList footer, pressing Enter triggered the list's keyboard shortcut instead. Fixed by addingCONST.ROLE.SWITCHtodisableEnterShortcutinBaseSelectionList.tsxGBR badge ignores high contrast theme (Theme - High contrast theme is not applied on GBR on LHN #86349): badgeStrongText used hardcoded colors.white instead of a theme-aware variable. Fixed by changing it to
theme.buttonSuccessText, which resolves to near-white in normal themes and dark green in high-contrast themes.Result:
Fixed Issues
$ #79228 (Original issue)
$ #86339
$ #86346
$ #86349
PROPOSAL:
Tests
Offline tests
QA Steps
// TODO: These must be filled out, or the issue title must include ""[No QA].""
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
Screen.Recording.2026-03-31.at.01.24.34.mov