Remove invoice interstitial modal

[MayaRainer]

Closes #1502.

Removes the interstitial modal when clicking on an invoice row, instead taking the user to the invoice page directly.

Before

Screencast_20251105_173455.webm

After

Screencast_20251105_182918.webm

[laugardie]

We could probably remove “View invoice” from the contextual menu since the row is now clickable to open it.

Also, when testing on mobile, I am not able to navigate to the invoice page.

https://github.com/user-attachments/assets/ae830065-f620-444c-a380-19d5ddcc3d6e

[laugardie]

@MayaRainer _a looks good to me!

[MayaRainer]

@laugardie Thanks! @slavingia _a lmk if this is good to merge!

[laugardie]

@MayaRainer just pushed some small UI changes. Rest looks good to me!

I have one small concern regarding the UX when editing the equity (not strictly an issue with this PR, but related). If a contractor adds line items to a new invoice and then realizes they need to update the payment split, clicking that link navigates them away and they lose all the data they just entered. Even if open on a new tab, we need to reload to get the updated equity split. I'm thinking maybe we could just save the form progress temporarily? Keep the info while the invoice hasn't been sent, and then clear it out once they successfully hit send? @ershad

[MayaRainer]

@ershad Gentle bump on this when you have a moment :)