Danny McClain
Danny McClain
>curious what you think - on the desktop version, should we use the 32px padding for the text/icon/button area? I'm not _intensely_ passionate, but I'm for using 32px padding on...
I like it with 32px! At least for these wider "empty-state-style" ones. And I think the bottom padding is good 🙃
I think showing a fallback image makes sense—but I'm not sure I would show that type of thumbnail. I feel like it kinda makes more sense to show our little...
>We're going to prevent uploading corrupted PDF as scan receipt in https://github.com/Expensify/App/issues/34032. So to be consistent, we should not allow corrupted images. @situchan that makes sense to me. >For no-preview...
I like Shawn's riff too.
One thing I think we need to consider that [Krishna's proposal](https://github.com/Expensify/App/issues/51504#issuecomment-2445925882) made me realize: I think we should auto-update the avatar when the name field is blurred (or _something_ like...
Yeah I was thinking about that too. Seems unlikely they'll actually unfocus the input. Maybe we just update it as soon as the input has a new first character? 🤷
>disclaimer: I am not involved in evaluating proposals :) Haha same! But that is what I had in mind.
Also this is feeling verrrrrry tight to me:  Can we try `20px` between the number inputs and the `Didn't receive a magic code?` message?...
For that case I think we'd do it like this:  But happy for Shawn to weigh in too!