Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport] Redemption improvements #585

Merged

Conversation

r-czajkowski
Copy link
Collaborator

Backport of: #556
Backport of: #580
Backport of: #582
Backport of: #583

This PR fixes typos, improves the elapsed time counter and updates copy on the Unmint page.

mswilkison and others added 6 commits July 27, 2023 09:45
We encountered an issue with the <LinkOverlay> component from chakra-ui while
using it in the History table on the tBTC Explorer page. This component was
intended to facilitate navigation for each row, directing the user to the
corresponding etherscan link. However, we noticed an abnormal behavior
specifically on real mobile devices, even though it worked correctly in the
mobile view on Chrome.

On mobile, the overlay for the last item appeared to cover the entire page. As a
result, any other links on the tBTC Explorer page, such as "View on Dune
Analytics" or "Start Minting," unintentionally redirected the user to etherscan.

The cause of this was that the `LinkBox` component uses `position: relative` to
contain the absolute positioned `LinkOverlay`.  Unfortunately `<Tr />`
component, which we passed to the `as property, does not allow `position:
relative`.

As a hacky workaround we are applying `transform: scale(1)` to the tr element.
Currently 3 hours is quite optimistic and it often takes around 5 hours
(experienced that during UATs several times). Here we update a copy
saying that a redemption takes `3-5 hours`.
`transacion` -> `transaction`
The Elapsed Time should be a dynamic countdown. Here we update the
elapsed time every one minute.
Copy link
Contributor

@michalsmiarowski michalsmiarowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@michalsmiarowski michalsmiarowski merged commit dcc50d7 into releases/mainnet/v1.9.0 Jul 27, 2023
2 checks passed
@michalsmiarowski michalsmiarowski deleted the backport-redemptions-improvements branch July 27, 2023 08:17
@github-actions
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants