Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Transaction data not visible on smaller screens #3284

Closed
usame-algan opened this issue Jan 11, 2022 · 2 comments · Fixed by #3309
Closed

Transaction data not visible on smaller screens #3284

usame-algan opened this issue Jan 11, 2022 · 2 comments · Fixed by #3309
Assignees
@usame-algan
Labels
Bug 🐛 Something isn't working

Comments

@usame-algan
Copy link
Member

Description

In some cases the transaction data is cut off on smaller screens

Environment

  • Browser: Chrome/Firefox
  • Wallet: MetaMask
  • Safe: Web
  • Environment:
    • dev

Steps to reproduce

  1. Go to https://safe-team.dev.gnosisdev.com/app/eth:0x4FeE396c8d940B801747CA84632580f917690E87/transactions/0x33eb21a8639f9411ae96a388f41b7af573e118771628b5d57c08f96817349ac9
  2. Observe on smaller viewports / smaller browser window

Expected result

The full transaction data is visible on all viewport sizes

Obtained result

The transaction data is cut off to the right on smaller viewports while the transaction window grows very large

Screenshots

Screenshot 2022-01-11 at 14 34 26
@usame-algan usame-algan added the Bug 🐛 Something isn't working label Jan 11, 2022
@liliya-soroka
Copy link
Member

#1915 - the same issue with a good example of the tx

@usame-algan usame-algan self-assigned this Jan 16, 2022
@usame-algan usame-algan mentioned this issue Jan 17, 2022
Merged
@francovenica
Copy link
Contributor

francovenica commented Jan 17, 2022 •
edited
Loading

Looks good to me.
image

you can compare the tx 162 (a 1inch exchange) of this safe in this PR and prod to see the difference
eth:0x8675B754342754A30A2AeF474D114d8460bca19b

@francovenica francovenica mentioned this issue Feb 1, 2022
Merged
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@usame-algan usame-algan

Labels
Bug 🐛 Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Display MethodDetails content on smaller screens 5afe/safe-react
3 participants
@liliya-soroka @francovenica @usame-algan