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

[Qt][Feature Request] Add search option to My Addresses list #2250

Closed
wants to merge 3 commits into from
Closed

[Qt][Feature Request] Add search option to My Addresses list #2250

wants to merge 3 commits into from

Conversation

PicklesRcool
Copy link

@PicklesRcool PicklesRcool commented Mar 15, 2021

This PR adds a filter box to the My Addresses list in the Receive section.

Issue: #1635

How it works:
https://user-images.githubusercontent.com/36901185/111183252-603cb500-85b8-11eb-8549-e31a9f18ba70.mp4

The edit box filters the address rows for both the name or the address itself. It also has a maximum input length of 40 characters.

@JSKitty
Copy link
Member

JSKitty commented Mar 15, 2021

This is cool! 💜

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

This is indeed pretty cool, and it works great. Thanks @PicklesRcool!
The only nit would be to squash the three commits, that are all editing the same (newly introduced) code.

@random-zebra random-zebra added this to the 6.0.0 milestone Mar 25, 2021
@random-zebra random-zebra requested review from furszy and Fuzzbawls April 9, 2021 13:45
@random-zebra
Copy link

I think this would be good to have. Maybe we can squash-merge it, addressing possible nits in a follow-up PR.
@furszy , @Fuzzbawls thoughts?

@furszy
Copy link

furszy commented Apr 28, 2021

If @PicklesRcool is not around to make the changes, the best path would be to cherry-pick the squashed commit and fixup any change from the review under his authorship in another PR. The ability for maintainers to push to his branch seems to not be active.

I still have this one pending to test. Code wise, the increment of the receivewidget.ui width isn't needed and can be cleaned.

random-zebra added a commit that referenced this pull request Apr 30, 2021
…widget (PicklesRcool)

274fb26 Add search option to My Addresses list in receive widget (Volodia)

Pull request description:

  This is an updated version of #2250 :
  - remove un-needed width increment in receivewidget.ui
  - squashed the commits down to one
  - push under @PicklesRcool's authorship

ACKs for top commit:
  furszy:
    Tested ACK 274fb26 .

Tree-SHA512: fe6aa6def3525c8c60b9f425e5e60f223d8cbbca7ee109fab98ba572a0306a2dd9df85d157232668f2549e90045f2996ecbfcf40ab87e31b8736912316727685
@random-zebra
Copy link

Squash-merged in #2353

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

Successfully merging this pull request may close these issues.

4 participants