Danny McClain

Results 178 comments of Danny McClain

Sounds good to me. It is a bit cramped right now. (Also down with making "Create expense" the top option)

>I still feel like we should have all of the expense options at the top (especially for new users where the menu is smaller), but curious if anyone agrees? I...

>Lets please get @Expensify/design involved whenever there are changes to how things look in the App. Was this look of the code block in composer vetted before? Apologies if that...

Definitely not against a success/confirmation page—we have lots of them sprinkled throughout the app so that shouldn't be a problem. (Also fine with it being a follow-up/not strictly part of...

I agree with Shawn, let's just make it look like a read-only bit of info (like a read-only push input row as Shawn pointed out). I will note that I...

@ishpaul777 is this what you're looking for? ![CleanShot 2024-03-20 at 09 09 32@2x](https://github.com/Expensify/App/assets/7516624/07350256-d668-4384-8361-53889a196f1b) In Figma, we're using our medium (default) size buttons—they have `16px` of left/right padding and a fixed...

>I don't feel too strongly. Is there any chance that request can take a long time? If so, guess the loading spinner makes sense. I feel the same way. But...

Awesome, appreciate the explanation! I'm down with including it too in that case 👍

[The video that Jon shared](https://github.com/Expensify/App/pull/49539#issuecomment-2375537540) is what I would expect and prefer. But if that's not technically feasible at the moment, I would rather have the bottom tabs than the...

Updated the OP with a new mock based on related Slack threads. Posting the mocks here just for super ultra clarity. ![image](https://github.com/user-attachments/assets/06a13604-fc38-4761-b6ab-36456a0b527f) ![image](https://github.com/user-attachments/assets/abf56323-6d77-4bd2-b082-c384732acc98)