chore: new swapper tweaks

[0xApotheosis]

Description

Actions some review feedback from #8192

• Makes the inner steps contractable on the trade completion summary (the expandy icon is there, so it should probably behave how the user expects when clicked)
• Changes "last hop" to "second hop" vernacular to better represent the referenced data
• Uses the receive address from the quote, rather than the hook (as we should always have a receive address at the time of trade confirm, and that's the address that will be used in the trade)

Issue (if applicable)

Addresses feedback from #8192

Risk

Small

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

None

Testing

Engineering

With the REACT_APP_FEATURE_NEW_TRADE_FLOW flag enabled, check the changes noted in the description.

Operations

• 🏁 My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)

N/A

[gomesalexandre]

Tested locally, confirmed this does what is says on the box:

• Collapsible bits are indeed collapsible/expandy

https://jam.dev/c/ab1b0924-f4cd-42ec-9fba-2d95897d4180

• Custom/Receive addy is still happy and funds are received on the correct address

https://jam.dev/c/69302e95-0e8e-473b-9eb1-76928a2866ed

Note @0xApotheosis unrelated to this diff but noting we've lost the receive address info i.e note how it is present on the "pre-preview" stage, but as soon as we're at execution screen, this data is not available anymore. We should probably re-surface it there somehow, as it should probably be visible to users at all time? 🤔