-
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
[HOLD for payment 2024-08-14] [HOLD for payment 2024-07-24] [$250] Create empty state component for list view pages #43747
Comments
Current assignee @shawnborton is eligible for the Design assigner, not assigning anyone new. |
Hi! I am Filip from SWM an expert agency and I'd like to work on this issue! |
Job added to Upwork: https://www.upwork.com/jobs/~01809e49334f961ddd |
Triggered auto assignment to @JmillsExpensify ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
@filip-solecki these are the possible skeletons that might be included in the empty state component: The Search Desktop skeleton row already exists, though I am proposing that we slightly update it to have a background color as part of this PR. In that same PR, we are making the Search Mobile skeleton rows which don't exist yet. Chat rows already exist. But generic table rows do not exist to my knowledge, so we'll need to make those skeletons as part of this. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Is there a confirmed design for any page where we can implement the new empty state component? I almost completed my work, but currently, there's no place to implement it for testing. |
I believe we can add this to the Search page, right? @shawnborton |
So I just need confirmed design with illustration/animation/video whatever will be used there |
Yup, search page will be a great place to test it out. You can start implementing it with a generic illustration (pick any existing one) and I'll follow up soon with a good one to use. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.7-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-07-24. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Dig it! |
Payment Summary
BugZero Checklist (@JmillsExpensify)
|
Payment summary above is filled out and correct. @eVoloshchak please propose a regression test and we can get your payment approved once I receive it in NewDot. Thanks! |
Regression Test Proposal
Do we agree 👍 or 👎 |
Reopening for regression test and payment summary from someone else. |
Contributor+: @eVoloshchak owed $250 via NewDot Test Case GH |
$250 approved for @eVoloshchak |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.17-2 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-08-14. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Right now, our empty states for certain list views look pretty underwhelming:
We want to make these empty states feel nicer by using a subtle hint that the page will eventually contain data by using a faded row pattern, while also creating space for the middle of the page to show an informative or educational modal. The component we want to build is something like this:
Here is a video of the component in action, and you can find it in our brand guidelines as well:
CleanShot.2024-06-13.at.09.51.54.mp4
The idea is this:
Once this component is in a robust state, we'll plan to implement it on the Search page as well as some other upcoming projects like Workspace feeds.
cc @mountiny @filip-solecki @Expensify/design
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @JmillsExpensifyThe text was updated successfully, but these errors were encountered: