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

Transaction preview/simulation UI should be responsive. #8

Closed
MicahZoltu opened this issue Dec 6, 2022 · 1 comment · Fixed by #25
Closed

Transaction preview/simulation UI should be responsive. #8

MicahZoltu opened this issue Dec 6, 2022 · 1 comment · Fixed by #25

Comments

@MicahZoltu
Copy link
Contributor

If I resize the popup window horizontally, the UI remains fixed in the center, with a lot of the UI elements crammed into a limited amount of space.

Just removing the width/max-width on the body element seems to resolve the issue for the most part. We could do a bit better with layout, but it is an improvement over current.

@MicahZoltu
Copy link
Contributor Author

Setting height: fit-content rather than 400px I think is also more appropriate. Though, really it feels like height shouldn't be present on the element and height: 100% should be replaced with height: fit-content on html,body.

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

Successfully merging a pull request may close this issue.

1 participant