-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Record Selector styling #2185
Record Selector styling #2185
Conversation
(Re-assigning review to Rajat because I see that Pavish currently has several other PRs to review.) |
This comment was marked as off-topic.
This comment was marked as off-topic.
@seancolsen The consistency fixes PR contains some styling changes which affect the record selector. Do you think you could base your PR on top of it, and resolve conflicts? This would be much faster than the other way around. Also, I expect the consistency fix PR to be merged tomorrow. |
Thanks for pointing that out, @pavish. Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@seancolsen I saw 2 weird things happening here:
- Layout shift when searching inside the record selector: https://www.loom.com/share/29ca246255cd4ba298fdf9308bca0529
- In the below screenshot the
Acquisition Date
has no matches but the results are still shown because of theid
andAcquisition price
match. It might make sense but it looked a bit weird to me at first look.
@rajatvijay This is ready for re-review now.
|
@seancolsen LGTM 🚀 |
@rajatvijay I had to dismiss your review to get this merged, please remember to use the "approve" functionality. Otherwise, auto-merge won't work. |
Fixes #2104
Before this PR
Figma design
After this PR
Notable differences vs Figma
Checklist
Update index.md
).master
branch of the repositoryDeveloper Certificate of Origin
Developer Certificate of Origin