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

Total balance #70

Merged
merged 4 commits into from Apr 25, 2017
Merged

Total balance #70

merged 4 commits into from Apr 25, 2017

Conversation

zillionn
Copy link
Contributor

@zillionn zillionn commented Apr 8, 2017

Show total fiat/bitcoin balance for all accounts on the homepage.

@@ -149,7 +149,7 @@
<md-content id="home-box-accounts" style="min-width:50%; opacity:0.9; border-radius:10px;" md-whiteframe="3">
<md-toolbar layout="row">
<div class="md-toolbar-tools">
<h2><translate>My Accounts</translate> {{ul.network.symbol}}{{ul.myAccountsBalance()}}</h2>
<h2><translate>My Accounts</translate> - {{ul.network.symbol}}{{ul.myAccountsBalance()}} / {{ul.currency.symbol}}{{(ul.myAccountsBalance()*ul.connectedPeer.market.price[ul.currency.name]).toFixed(2)}}</h2>
Copy link

@fix fix Apr 14, 2017

Choose a reason for hiding this comment

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

why an extra "-"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, no reason, just separates the title from the numbers. I can remove it if you want.

@fix
Copy link

fix commented Apr 14, 2017

yes because it could look like negative numbers

I get a vertical scrollbar when transaction ID is in the title because of the max-width.
@fix fix merged commit 35ba6f5 into ArkEcosystem:master Apr 25, 2017
@fix
Copy link

fix commented Apr 25, 2017

+5

@boldninja
Copy link
Member

@zillionn can you provide me with permanent Ark address where you want +5 rewards and any other github rewards you might get for helping out ? Thanks.

@zillionn
Copy link
Contributor Author

zillionn commented May 3, 2017

Don't worry about it, I don't help for the rewards.

@boldninja
Copy link
Member

Thank you, but rules are rules :)

@zillionn
Copy link
Contributor Author

zillionn commented May 3, 2017

OK, I'd like to donate them :).

@boldninja
Copy link
Member

Haha ok I won't bug you with this 👍 I'll donate to Ark Community Fund once address is known sounds OK?

@zillionn
Copy link
Contributor Author

zillionn commented May 3, 2017

OK

@boldninja
Copy link
Member

Thanks for your work here - much appreciated.

krau612 pushed a commit to krau612/ark-desktop that referenced this pull request Sep 16, 2017
@zillionn zillionn deleted the patch-1 branch December 10, 2018 10:37
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.

None yet

3 participants