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

fix: small layout bug #2553

Closed
wants to merge 1 commit into from
Closed

fix: small layout bug #2553

wants to merge 1 commit into from

Conversation

bonomat
Copy link
Contributor

@bonomat bonomat commented May 21, 2024

Before After
image image

@bonomat bonomat requested review from holzeis and luckysori May 21, 2024 05:44
@bonomat bonomat enabled auto-merge May 21, 2024 05:44
@holzeis
Copy link
Contributor

holzeis commented May 21, 2024

🤔 is this really a bug? Is the label now jumping with the amount? Or is it already doing that?

@bonomat
Copy link
Contributor Author

bonomat commented May 21, 2024

Don't you see the big space between Fee: and 17,880? Or are you saying this was by design, because I find it quite weird 😅

image

@holzeis
Copy link
Contributor

holzeis commented May 21, 2024

Don't you see the big space between Fee: and 17,880? Or are you saying this was by design, because I find it quite weird 😅

image

I think it's by design so that the label is not jumping when the fee amount is changing.

@bonomat
Copy link
Contributor Author

bonomat commented May 21, 2024

I think it's by design so that the label is not jumping when the fee amount is changing.

Lol, and here I sit, thinking this is ugly as a hat full of bums.

@bonomat bonomat closed this May 21, 2024
auto-merge was automatically disabled May 21, 2024 10:11

Pull request was closed

@holzeis
Copy link
Contributor

holzeis commented May 21, 2024

I think it's by design so that the label is not jumping when the fee amount is changing.

Lol, and here I sit, thinking this is ugly as a hat full of bums.

we can still merge it if you want.. 😅

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 this pull request may close these issues.

2 participants