Danny McClain

Results 178 comments of Danny McClain

I agree with Shawn—2-3 lines seems totally reasonable to me. If we can't pick the amount of lines, I bet we can estimate a character length that gives us something...

Will this live along side the [early adoption discount](https://github.com/Expensify/Expensify/issues/440617)?

Mock looks good to me! Might want to show the mobile version too just for super ultra clarity.

>but with a modal as skinny as this, I don't think it would look very good. Do we have a standard width for these on desktop? Why is the one...

Ok yeah that makes sense! Just wanted to make sure we didn't have a bunch of randomly different sizes 👍

>Wonder if we could just only show valid accounts in the contact picker instead. Big plus one to this. If you can't fully invite a user from the vacation delegate...

Thanks! I do think that looks good. Makes so much more sense that way IMO.

> `updated the index of the Distance rate "New Rate" from "" to "1"` Where is this message coming from? It makes no sense to me and I don't know...

Gonna tag @Expensify/product-pr here in case any of them have any more context on where this weird system message is coming from/why.