App icon indicating copy to clipboard operation
App copied to clipboard

Add drag/drop for receipt empty state

Open JmillsExpensify opened this issue 1 year ago • 31 comments

If a Collect or Control policy is configured with any receipt-related category violation, we’ll update the confirmation screen for manual requests and include a receipt empty state. Tapping the empty state brings up the camera (mobile) or file picker (web). For web, we’ll support drag/drop.

image

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e09455d07e462ca6
  • Upwork Job ID: 1704005352336375808
  • Last Price Increase: 2023-09-19

JmillsExpensify avatar Sep 19 '23 05:09 JmillsExpensify

Current assignee @JmillsExpensify is eligible for the NewFeature assigner, not assigning anyone new.

melvin-bot[bot] avatar Sep 19 '23 05:09 melvin-bot[bot]

Job added to Upwork: https://www.upwork.com/jobs/~01e09455d07e462ca6

melvin-bot[bot] avatar Sep 19 '23 05:09 melvin-bot[bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @aimane-chnaif (Internal)

melvin-bot[bot] avatar Sep 19 '23 05:09 melvin-bot[bot]

No update from me, focused on detailed design for Wave 9: Violations

cead22 avatar Sep 28 '23 01:09 cead22

Still working through the detailed design for Wave 9.

JmillsExpensify avatar Oct 10 '23 23:10 JmillsExpensify

Still working through the detailed.

JmillsExpensify avatar Oct 17 '23 20:10 JmillsExpensify

Still focused on violations implementation first.

JmillsExpensify avatar Oct 24 '23 22:10 JmillsExpensify

Still on HOLD, but Infinite Red started on App#30663

cead22 avatar Nov 08 '23 02:11 cead22

App#30663 is merged so this is off hold. @cdanwards @trevor-coleman can you comment here so I can assign you?

cead22 avatar Nov 22 '23 00:11 cead22

Commenting for assignment

cdanwards avatar Nov 22 '23 14:11 cdanwards

Waiting on PR

JmillsExpensify avatar Nov 29 '23 00:11 JmillsExpensify

@cdanwards any updates on this?

cead22 avatar Nov 29 '23 21:11 cead22

@cdanwards is working on a couple of other items for violations. This is in the queue, but not being worked on yet

cead22 avatar Dec 12 '23 02:12 cead22

All good, I think it's lower priority.

JmillsExpensify avatar Dec 13 '23 05:12 JmillsExpensify

This is in the queue, but not being worked on yet

cead22 avatar Dec 22 '23 18:12 cead22

Still not at the top of the priority list.

JmillsExpensify avatar Jan 03 '24 03:01 JmillsExpensify

Beginning work on this.

cdanwards avatar Jan 10 '24 21:01 cdanwards

One small design bug to fix while we're here:

  • let's update the border color of the thumbnail area to use our colors.border value
  • let's update the background color of the thumbnail area to use our colors.highlightBG value

shawnborton avatar Jan 25 '24 17:01 shawnborton

@cdanwards please tag @Expensify/design in the PR when you create it

cead22 avatar Jan 25 '24 18:01 cead22

Also, the border should not be 2px like it is here: CleanShot 2024-01-25 at 14 42 36@2x

Just 1px stroke for the border.

shawnborton avatar Jan 25 '24 19:01 shawnborton

Also it looks like the image is cut off - I wonder if we need to get you a better image for that? Can you share the image that you are currently using please? Thanks!

shawnborton avatar Jan 25 '24 19:01 shawnborton

@shawnborton sorry this got lost in my GH notifications. This one has already been merged, but here is the svg that we're using for this component

cdanwards avatar Feb 05 '24 16:02 cdanwards

Interesting, the svg itself seems fine as we can see the borders: CleanShot 2024-02-05 at 11 55 19@2x

Any idea why it might be getting cut off?

shawnborton avatar Feb 05 '24 16:02 shawnborton

@shawnborton I'm honestly not sure. It's the right side that you're referring to, correct? There was some difficulty in having the svg display colors correctly, so that could be part of the issue.

cdanwards avatar Feb 05 '24 20:02 cdanwards

Exactly, see how the 1px stroke is slightly cut off?

shawnborton avatar Feb 05 '24 20:02 shawnborton

@shawnborton I found the issue. Going to put together a little PR with a fix for that and also the other corrections you mentioned.

cdanwards avatar Feb 05 '24 21:02 cdanwards

Amazing, thanks! Feel free to tag me in the PR for review.

shawnborton avatar Feb 05 '24 21:02 shawnborton

Screenshot 2024-02-05 at 4 20 38 PM @shawnborton does this look right for the border width and color you mentioned above? The border looks very faint with the combination of those styles. I'm using these values:

  backgroundColor: colors.highlightBG,
  borderColor: colors.border,
  borderWidth: 1,

cdanwards avatar Feb 05 '24 21:02 cdanwards

I think something is happening here, there is actually no BG color being shown at all. If there was, we wouldn't see the illustration come through.

shawnborton avatar Feb 05 '24 21:02 shawnborton

Screenshot 2024-02-05 at 4 26 10 PM There we are. Got the right values!

cdanwards avatar Feb 05 '24 21:02 cdanwards