Fix insufficient spacing between card name and issue card button on resize#97018
Draft
MobileMage wants to merge 1 commit into
Draft
Fix insufficient spacing between card name and issue card button on resize#97018MobileMage wants to merge 1 commit into
MobileMage wants to merge 1 commit into
Conversation
Apply a permanent flex gap3 on the feed-selector/header-buttons parent container instead of the conditional mt3 margin, which never applied at medium width because shouldDisplayButtonsInSeparateLine is false there. Fixes Expensify#94988.
Open
7 tasks
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Explanation of Change
On the Expensify Card list page, the gap between the feed selector (Card name) and the header buttons collapses when you resize the window to a medium width. At medium width the buttons stack onto a separate line (
shouldChangeLayout), but the margin that created the gap (styles.mt3) sits behindshouldDisplayButtonsInSeparateLine, which only turns true in the narrow-portrait layout. So at medium width no spacing applies and the Card name and buttons touch.This PR puts a permanent flex
gapon the shared parent that wraps theFeedSelectorand the header buttons. Onestyles.gap3(12px, the same value as the oldmt3) holds the separation in both layouts: horizontal when the row is side-by-side, vertical when it stacks. The stalemt3line is removed.Fixed Issues
$ #94988
PROPOSAL: #94988 (comment)
Tests
Offline tests
No offline-specific behavior. The change is layout only.
QA Steps
Same as Tests.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, 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.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Videos will be added in a few hours.
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari