Skip to content

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Dec 17, 2020

From bitcoin/bitcoin#20555 (comment):

0.21.0rc3 compiled from source, Ubuntu 20.04 + i3: tested mainnet, testnet, and signet. created a descriptor wallet in qt, tested torv3 with addnode, and checked if anchors.dat is created on shutdown and removed on startup.

I've noticed the "Advanced options" label being cut-off in the wallet creation dialog.

This PR should fix this issue.

@maflcko
Copy link
Contributor

maflcko commented Dec 18, 2020

cc @0xB10C

@0xB10C
Copy link
Contributor

0xB10C commented Dec 18, 2020

This seems to fix it! Thanks.

rc3:
image

pr155:
image

@maflcko
Copy link
Contributor

maflcko commented Dec 18, 2020

review ACK e71b656

didn't test

@maflcko maflcko merged commit 3dd3f7c into bitcoin-core:master Dec 18, 2020
@jonasschnelli
Copy link
Contributor

Shouldn't this be a flexible (stretch layout instead of fixed pixles?

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Dec 18, 2020
@maflcko
Copy link
Contributor

maflcko commented Dec 18, 2020

This can be reworked for 0.22, but I think for rc4 we should aim for a minimal fix

@hebasto
Copy link
Member Author

hebasto commented Dec 18, 2020

Shouldn't this be a flexible (stretch layout instead of fixed pixles?

The goal of this PR was the minimal fix as @MarcoFalke said. Besides, the issue has spotted on i3 window manager, and the real problem could be much deeper.

@hebasto hebasto deleted the 201218-i3 branch December 18, 2020 08:02
@jonasschnelli
Copy link
Contributor

post merge utACK the fix. I missed the rc4 issue or 0.21 tag...

sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 18, 2020
e71b656 qt: Align layout of checkboxes (Hennadii Stepanov)

Pull request description:

  From bitcoin#20555 (comment):
  > 0.21.0rc3 compiled from source, Ubuntu 20.04 + i3: tested mainnet, testnet, and signet. created a descriptor wallet in qt, tested torv3 with addnode, and checked if anchors.dat is created on shutdown and removed on startup.
  >
  > I've noticed the "Advanced options" label being cut-off in the wallet creation dialog.

  This PR should fix this issue.

ACKs for top commit:
  MarcoFalke:
    review ACK e71b656

Tree-SHA512: dcc997a440a96fc936296323afaf9f4040ac06cf5ad32724a88c2e2e16c726c35b1b2bacbb3e081839d8a69b5a6086b2570d6b2e98976f29b5427db2d434eee9
ComputerCraftr pushed a commit to ElectraProtocol/XEP-Core that referenced this pull request Jan 3, 2021
Github-Pull: bitcoin-core/gui#155
Rebased-From: e71b656f317f38ef0ba0874736f116dae39efc67
ComputerCraftr pushed a commit to ComputerCraftr/XEP-Core that referenced this pull request Jan 5, 2021
Github-Pull: bitcoin-core/gui#155
Rebased-From: e71b656f317f38ef0ba0874736f116dae39efc67
apoelstra pushed a commit to apoelstra/elements that referenced this pull request Sep 1, 2021
Github-Pull: bitcoin-core/gui#155
Rebased-From: e71b656
(cherry picked from commit ef7a155)
apoelstra pushed a commit to apoelstra/elements that referenced this pull request Sep 18, 2021
Github-Pull: bitcoin-core/gui#155
Rebased-From: e71b656
(cherry picked from commit ef7a155)
apoelstra pushed a commit to apoelstra/elements that referenced this pull request Sep 22, 2021
Github-Pull: bitcoin-core/gui#155
Rebased-From: e71b656
(cherry picked from commit ef7a155)
@bitcoin-core bitcoin-core locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants