Skip to content

chore(send): align unit in collaborators fee breakdown#708

Merged
theborakompanioni merged 3 commits intomasterfrom
chore/707-align-sats
Dec 14, 2023
Merged

chore(send): align unit in collaborators fee breakdown#708
theborakompanioni merged 3 commits intomasterfrom
chore/707-align-sats

Conversation

@theborakompanioni
Copy link
Collaborator

Resolves #707.

Shows the expected collaborator fees in the unit of the users choice.
Jam can still display values in sats (instead of BTC) - so the toggle will influence the displayed unit (as opposed to always show in BTC). Getting rid of the conversion to sats has been thought about, but should be done in a separate issues if needed.

📸

@theborakompanioni theborakompanioni added bug Something isn't working UI/UX Issue related to cosmetics, design, or user experience labels Dec 12, 2023
@theborakompanioni theborakompanioni self-assigned this Dec 12, 2023
Copy link
Contributor

@MarnixCroes MarnixCroes left a comment

Choose a reason for hiding this comment

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

ack fbc852a

Jam can still display values in sats (instead of BTC) - so the toggle will influence the displayed unit (as opposed to always show in BTC). Getting rid of the conversion to sats has been thought about, but should be done in a separate issues if needed.

do you mean getting rid of the BTC/sats option in settings in general?
for me that would be a nack, it's nice.

the thing that could be improved about this is having also the input field in BTC when unit is BTC. now those are still sats which can be annoying/confusing.

@editwentyone
Copy link

what about these?
290032138-abf46f0e-faa0-49f3-a6cc-fbc7f3c3f4f5

@editwentyone
Copy link

the thing that could be improved about this is having also the input field in BTC when unit is BTC. now those are still sats which can be annoying/confusing.

already on it : #692

@editwentyone
Copy link

editwentyone commented Dec 13, 2023

tiny stuff in details view: because of the bond?

Bildschirmfoto 2023-12-13 um 11 24 14

@theborakompanioni
Copy link
Collaborator Author

what about these? 290032138-abf46f0e-faa0-49f3-a6cc-fbc7f3c3f4f5

  • First circle: Can be fixed - just overlooked - thank you!
  • Second circle: Will try.. this is a i18n string.. let's see how to use the balance component here..

@theborakompanioni
Copy link
Collaborator Author

do you mean getting rid of the BTC/sats option in settings in general? for me that would be a nack, it's nice.

Yes. The current behaviour of the BTC unit should (theoretically) combine best of both worlds.. which would make the differentiation obsolete.. would be less code while having nearly same functionality. However, I can live with keeping it.

the thing that could be improved about this is having also the input field in BTC when unit is BTC. now those are still sats which can be annoying/confusing.

Yep, that is planned to be worked on soon and hopefully lands in v0.2.0. 🙌

@theborakompanioni
Copy link
Collaborator Author

what about these? [...]

@editwentyone

@theborakompanioni theborakompanioni merged commit 643db85 into master Dec 14, 2023
@theborakompanioni theborakompanioni deleted the chore/707-align-sats branch December 14, 2023 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UI/UX Issue related to cosmetics, design, or user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show Sats in the same way everywhere

3 participants