-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search - Header Pills are touching each other when highlighted, no spacing #48249
Comments
Triggered auto assignment to @RachCHopkins ( |
Triggered auto assignment to @pecanoro ( |
👋 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:
|
We think that this bug might be related to #wave-control |
This is the intended design. cc @luacmartins for confirmation |
ProposalPlease re-state the problem that we are trying to solve in this issue.There is no spacing between the Search Pills What is the root cause of that problem?We are not setting margin for the button here App/src/components/Search/SearchStatusBar.tsx Line 155 in 8c69c58
What changes do you think we should make in order to solve the problem?We should add margin horizontal for these buttons and pay attention to the first and last item in here What alternative solutions did you explore? (Optional) |
I believe this is the intended design. cc @Expensify/design |
Correct, this is not a bug - going to close. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: v9.0.26-1
Reproducible in staging?: Y
Reproducible in production?: N
Issue was found when executing this PR: #47753
Email or phone of affected tester (no customers): applausetester+en@applause.expensifail.com
Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
User expects to have spacing between each pill
Actual Result:
There is no spacing between the Search Pills
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6586081_1724903004159.Search_pills_are_touching_when_highlighted.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: