Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Add on-chain balance to wallet fragment #424

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlbertoBoldrini
Copy link

Add on-chain balance to wallet fragment

Description

In the wallet home page, it shows the lightning balance and the on-chain balance.
The code is simply a suggestion, probably there are better ways to do it.
A problem is the position of switchButtonImage which now is a little bit ambiguous.

Motivation and Context

You can easily see how much you can pay with lightning or with an on-chain transaction.

How Has This Been Tested?

Tested on my phone Xiaomi Redmi Note 9 Pro

Screenshots (if appropriate):

screenshot

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the Contribution document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@michaelWuensch
Copy link
Contributor

@AlbertoBoldrini
Thanks a lot for the pull request!
I can understand that it is sometimes frustrating to only see one balance, especially if you are a power user and you know how bitcoin & lightning works.
Did you know that you can make a long tap on the balance and a popup will occur that shows you some details about the ballance? So basically it is already possible to see on chain ballance.
Showing only one balance was done on purpose to remove complexity and I would like to keep it that way, at least as default.
I would agree to it if we make this optional and expose a setting in the advanced settings tab.

And for some reason the Scan button is missing in your screenshot.

@AlbertoBoldrini
Copy link
Author

AlbertoBoldrini commented Oct 21, 2022

Hi michael, thank you for the answer. Ahah no, I didn't know the hidden feature.
However, I agree with you. The total balance is cleaner and simpler.
I put an option in the advanced settings tab
image

I'm doing it mainly for myself, if you like it when it's finished you can merge it, but it's not a problem if you prefer not to.
Are there icons of the Zap project that can be used for lightning and on-chain balances? Currently I used unicode emoji, but I don't like them so much.

Copy link

@azimanuar5874 azimanuar5874 left a comment

Choose a reason for hiding this comment

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

HelLO there. Im newbie here. To be honest, im dont know how to deal with this all code. Fork, spoon, PR, COMMIT. So depression when heard to verb of faxing KOD

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.

3 participants