Skip to content
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

Closed
shawnborton opened this issue Jun 14, 2024 · 32 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Design External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors NewFeature Something to build that is a new item. Weekly KSv2

Comments

@shawnborton
Copy link
Contributor

shawnborton commented Jun 14, 2024

Right now, our empty states for certain list views look pretty underwhelming:
image

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:
image

image

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:

  • the empty state can have a variety of different row styles in the background
    • table rows to match Search on desktop
    • compact rows to match Search on mobile
    • other generic row styles that our app supports elsewhere
  • we should be able to decide if we want to show a static illustrated header, or if we want to pass in a video or looping GIF
  • we should optionally be able to choose if we want to include a button in the empty state card or not
  • we should make sure we have a good minimum and maximum height for the empty state rows so we don't get any wild, unexpected results on extreme viewport cases

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
  • Upwork Job URL: https://www.upwork.com/jobs/~01809e49334f961ddd
  • Upwork Job ID: 1801589500078964548
  • Last Price Increase: 2024-06-21
Issue OwnerCurrent Issue Owner: @JmillsExpensify
@shawnborton shawnborton self-assigned this Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Current assignee @shawnborton is eligible for the Design assigner, not assigning anyone new.

@filip-solecki
Copy link
Contributor

Hi! I am Filip from SWM an expert agency and I'd like to work on this issue!

@mountiny mountiny added External Added to denote the issue can be worked on by a contributor Daily KSv2 NewFeature Something to build that is a new item. labels Jun 14, 2024
@melvin-bot melvin-bot bot changed the title Create empty state component for list view pages [$250] Create empty state component for list view pages Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01809e49334f961ddd

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Triggered auto assignment to @JmillsExpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added the Weekly KSv2 label Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

@melvin-bot melvin-bot bot removed Daily KSv2 Weekly KSv2 labels Jun 14, 2024
@shawnborton
Copy link
Contributor Author

@filip-solecki these are the possible skeletons that might be included in the empty state component:
image

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.

@melvin-bot melvin-bot bot added the Monthly KSv2 label Jun 17, 2024
Copy link

melvin-bot bot commented Jun 21, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@filip-solecki
Copy link
Contributor

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.

@filip-solecki
Copy link
Contributor

cc @shawnborton @mountiny

@mountiny
Copy link
Contributor

I believe we can add this to the Search page, right? @shawnborton

@filip-solecki
Copy link
Contributor

So I just need confirmed design with illustration/animation/video whatever will be used there

@shawnborton
Copy link
Contributor Author

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.

@shawnborton
Copy link
Contributor Author

In the workspace editor, when you enable Tags or Taxes, we can use this component here too:

CleanShot 2024-06-24 at 17 41 22@2x

@filip-solecki
Copy link
Contributor

Taxes cannot be empty, while enabling we are creating two default tax rates and one has to always be set as default and cannot be deleted, but Categories may be empty so I'll add it also on Tags and Categories.
Just send me all illustrations or whatever has to be in header there.

@shawnborton
Copy link
Contributor Author

@Expensify/design thoughts on doing something generic for these pages, like the folder illustration we currently use there?
CleanShot 2024-06-25 at 09 29 25@2x

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jul 17, 2024
Copy link

melvin-bot bot commented Jul 17, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Jul 17, 2024

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:

Copy link

melvin-bot bot commented Jul 17, 2024

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:

  • [@eVoloshchak] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [@JmillsExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

@dubielzyk-expensify
Copy link
Contributor

Dig it!

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Jul 23, 2024
Copy link

melvin-bot bot commented Jul 24, 2024

Payment Summary

Upwork Job

BugZero Checklist (@JmillsExpensify)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants/1801589500078964548/hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@melvin-bot melvin-bot bot added the Overdue label Jul 25, 2024
@JmillsExpensify
Copy link

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!

@melvin-bot melvin-bot bot removed the Overdue label Jul 26, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Jul 30, 2024
@eVoloshchak
Copy link
Contributor

Regression Test Proposal

  1. Log in to the app
  2. Go to Account Settings -> Workspaces
  3. Select any workspace
  4. Go to More Features
  5. Enable Tags and Categories
  6. Open Categories page and remove all categories
  7. Verify there is an empty page placeholder with the following text "You haven't created any categories"
  8. Go to tags, remove all tags if there are any
  9. Verify there is an empty page placeholder with the following text "You haven't created any tags"

Do we agree 👍 or 👎

@JmillsExpensify
Copy link

Reopening for regression test and payment summary from someone else.

@mallenexpensify
Copy link
Contributor

Contributor+: @eVoloshchak owed $250 via NewDot

Test Case GH

@JmillsExpensify
Copy link

$250 approved for @eVoloshchak

Copy link

melvin-bot bot commented Aug 7, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

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.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Aug 7, 2024
@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-07-24] [$250] Create empty state component for list view pages [HOLD for payment 2024-08-14] [HOLD for payment 2024-07-24] [$250] Create empty state component for list view pages Aug 7, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Aug 7, 2024
Copy link

melvin-bot bot commented Aug 7, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Aug 7, 2024

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:

Copy link

melvin-bot bot commented Aug 7, 2024

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:

  • [@eVoloshchak] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [@JmillsExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Design External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors NewFeature Something to build that is a new item. Weekly KSv2
Projects
Archived in project
Development

No branches or pull requests

8 participants