Skip to content

Activity page fixes #476

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

Merged
merged 2 commits into from
Jul 22, 2025
Merged

Activity page fixes #476

merged 2 commits into from
Jul 22, 2025

Conversation

johnny9
Copy link
Collaborator

@johnny9 johnny9 commented Jul 22, 2025

There were two errors that prevented the Activity page from working correctly

The first was a missing anchor for the listView, The patch correctly has the listView fill its parent so it appears now.

Before:
Screenshot from 2025-07-21 22-57-11
After:
Screenshot from 2025-07-21 22-57-18

The second fix fixes a bad reference to the stackView and gets rid of the following error in debug.log
2025-07-22T02:47:12Z GUI: qrc:/qml/pages/wallet/Activity.qml:19: TypeError: Cannot call method 'pop' of null

johnny9 added 2 commits July 21, 2025 22:52
listView needs an explict anchor.fill: parent because its parent
is an Item.
Copy link
Contributor

@pinheadmz pinheadmz left a comment

Choose a reason for hiding this comment

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

ACK ffd0c09

built and tested macos/arm64, reviewed the minimal diff, works as expected. Thanks!

Show Signature
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

ACK ffd0c09e77c3fec14f5105f125281d2f3442adcb
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEE5hdzzW4BBA4vG9eM5+KYS2KJyToFAmh/aasACgkQ5+KYS2KJ
yTpdKA/+O9Ed/nGLJoECNja0AB/DOnRh3XYLt2CfMJ+PyNy7OMAPIp9YAOBGB+nb
mQz4vNCLbUp8LEr4BuDG4YAcuJfwcPyAsE7m16lSUlpIGpQVZGNbBx3MCrJPgy0f
YTCWT/NGA+bylqvQkGAemJGuApVwk3iN1bhGDODCg8LViZOne8GIaRxDNQi30UtT
y5fkxI1hsrw6EC1n2jREXer023N4QyA0uqBZpjkIkJOVRDpRMq6QGgyI9cEJT1D1
qiejEqd6OFYV2qrvPlN9mAv8YkAskBL/PMgGks91K6EaxFZ7/WwlVJOUa1zlfGI8
KQuHciE72v/4HqF7kzUGIo3LGdvS5mxUCgPMiCzU0IqocKQj3vb7tY3VYimGsQ09
HzUcVMhVH97dRhYx6F0pN4pJi7B0x0TKc37ua1i+Re4znjrPmQe0JPYG0CnOPhmZ
No9BWJ4y8kRlq6Xbk3cr6bKvQ+iVEat1qn0ynO2M6yE0TBpUGBOHDWj4WFjsAwac
0j3UIADEP1C8krH2cZTn6aA1IPXaRT5PB5bYfkNOL3ejgkEf0CIAMvLdM7stcW17
Fjhx3xUKfB6EZFoVaNCb/e9dcId8IfZXUO5m52bxsrDINSu8tv2fEa3NflSPbB0D
rmxhLMkuzOMlti2ZGS1oeiRwUy87ZfhgG2eUcLi2min56CeSBKk=
=9Okt
-----END PGP SIGNATURE-----

pinheadmz's public key is on openpgp.org

@hebasto hebasto merged commit 09b02a2 into bitcoin-core:qt6 Jul 22, 2025
2 checks passed
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.

3 participants