leather-io / extension

Leather browser extension
https://leather.io
MIT License
305 stars 143 forks source link

Increase fee dialog + `retrieve-taproot-funds` incorrect background #5074

Open pete-watters opened 8 months ago

pete-watters commented 8 months ago

During testing I noticed that the increase fee dialog has an incorrect background:

Screenshot 2024-03-14 at 09 20 04

I tried fixing it quickly but couldn't.

It's currently a routed dialog with routes for /increase-fee/stx and /increase-fee/btc.

@kyranjamie - what are your thoughts on changing this from being a routed dialog to a non routed dialog?

If you navigate directly to the URL the context of the transaction is not provided anyway

pete-watters commented 8 months ago

@markmhendrickson this relates to the issue you raised here .

In that scenario, increase fee hit an error and couldn't show the error page.

I fixed that but in testing it I found this other issue where the dialog has the incorrect BG

kyranjamie commented 8 months ago

@kyranjamie - what are your thoughts on changing this from being a routed dialog to a non routed dialog?

These are only called from the one place, so isn't it a case of just making sure the containing page has an <Outlet />? If necessary it can be a non-routed modal, but worry that could introduce stale state bugs, whereas the currently the state is ephemeral and passed with route state, I think?

pete-watters commented 8 months ago

OK thanks. I'll try fix it as routed dialogs. The STX increase fee doesn't get passed any state but maybe it should.

I'll investigate and fix properly

pete-watters commented 2 months ago

We have a similar issue with the retrieve-taproot-funds dialog which isn't showing the correct background page

Screenshot 2024-08-20 at 10 23 23