Bitshares Graphene-ui 2.0.151125 released

Started by wmbutler, November 26, 2015, 02:10:54 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

wmbutler

Quote from: Thom on December 07, 2015, 01:25:02 AM
Quote from: svk on December 06, 2015, 07:20:01 AM
Quote from: Thom on December 06, 2015, 06:14:40 AM
Quote from: svk on November 29, 2015, 12:22:35 PM
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.

I'm a bit fan of tool tips. I think almost everything should have one attached. A tool tip on that value is all that's required to explain what that number represents, and, it keeps the UI clean. I had to ask what it was, b/c I'm not sure it's right, or how it calculates the value. It doesn't seem to add up in my case and the wallet contains only 2 accounts with very few assets.

It doesn't currently total assets held as collateral or open orders....but it will for v1209 !!

https://github.com/cryptonomex/graphene-ui/issues/584
PTS: PnBVP1iLTsV6U8z4BeJYhF8jMpkLhtTi9r
BTS2.0: billbutler
There are 10 kinds of people. Those who understand binary and those who do not.

Thom

Quote from: svk on December 06, 2015, 07:20:01 AM
Quote from: Thom on December 06, 2015, 06:14:40 AM
Quote from: svk on November 29, 2015, 12:22:35 PM
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.

I'm a bit fan of tool tips. I think almost everything should have one attached. A tool tip on that value is all that's required to explain what that number represents, and, it keeps the UI clean. I had to ask what it was, b/c I'm not sure it's right, or how it calculates the value. It doesn't seem to add up in my case and the wallet contains only 2 accounts with very few assets.
Injustice anywhere is a threat to justice everywhere - MLK |  Verbaltech2 Witness Reports: [url="https://bitsharestalk.org/index.php/topic,23902.0.html"]https://bitsharestalk.org/index.php/topic,23902.0.html[/url]

svk

Quote from: Thom on December 06, 2015, 06:14:40 AM
Quote from: svk on November 29, 2015, 12:22:35 PM
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Yes. No label to keep it clean, it really shouldn't be necessary imo, there will be a tooltip though  breaking down the totals by asset.
Worker: dev.bitsharesblocks

Thom

Quote from: svk on November 29, 2015, 12:22:35 PM
I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.

Is this the value displayed on the top line to the left of the settings gear / icon? Any reason you didn't use a label with it, like "Wallet Value:"
Injustice anywhere is a threat to justice everywhere - MLK |  Verbaltech2 Witness Reports: [url="https://bitsharestalk.org/index.php/topic,23902.0.html"]https://bitsharestalk.org/index.php/topic,23902.0.html[/url]

wmbutler

Thanks for the tips. Best to retire this thread since we have more recent upgrade announcement threads.
PTS: PnBVP1iLTsV6U8z4BeJYhF8jMpkLhtTi9r
BTS2.0: billbutler
There are 10 kinds of people. Those who understand binary and those who do not.

btstip

Hey thera, here are the results of your tips...
  • svk: has been credited 100 SMILE
  • wmbutler: has been credited 100 SMILE
Curious about BtsTip? Visit us at http://sharebits.io and start tipping BTS on https://bitsharestalk.org/ today!
Created by hybridd

thera

Quote from: wmbutler on November 27, 2015, 02:49:54 AM
Actually, @svk is the machine. If you guys want to give props, please do so for him. As far as I can tell he's doing this without any formal agreements (yet) from CNX. If he were to submit a worker proposal for his efforts, I'd fully support it.

Thank you @svk and @wmbutler!

#sharebits "svk" 100 SMILE
#sharebits "wmbutler" 100 SMILE

Feel free to share the SMILEs!  :)
gentleness is the greatest strength ~ iroquois proverb


svk

Quote from: karnal on November 28, 2015, 03:27:10 PM
I find that very unexpected behaviour. Perhaps others disagree?

There should be an easy "My assets worth in $this" overview.

Done the way it is now, it definitely shouldn't keep the last account value when the dashboard is in focus.. then we have a number that refers to a subset of the total in the context of the total, it's confusing. Indeed, for some brief seconds my blood pressure shot up as it seemed BTS had gone even lower, as I (and the 3 other enthusiasts who happened to be around at that time) looked at the equivalent amount in EUR as shown by this version.. and this was before selecting ANY account (guess it picked the oldest or something).

The idea is good, but I think it needs to be contextualized better. Perhaps a smaller font and two lines: "My assets", and "Accounts I follow" .. something like that.

Thoughts?

I've pushed a change that makes it so the total displayed in the header is the total for all accounts controlled by the currently open wallet. It will no longer be affected by accounts that you're just browsing. If you control no accounts, no total will be displayed.
Worker: dev.bitsharesblocks

Akado

This should be posted on the BitShares Upgrade Annoucements thread
[url="https://metaexchange.info"]https://metaexchange.info[/url] | Bitcoin<->Altcoin exchange | Instant | Safe | Low spreads

karnal

I find that very unexpected behaviour. Perhaps others disagree?

There should be an easy "My assets worth in $this" overview.

Done the way it is now, it definitely shouldn't keep the last account value when the dashboard is in focus.. then we have a number that refers to a subset of the total in the context of the total, it's confusing. Indeed, for some brief seconds my blood pressure shot up as it seemed BTS had gone even lower, as I (and the 3 other enthusiasts who happened to be around at that time) looked at the equivalent amount in EUR as shown by this version.. and this was before selecting ANY account (guess it picked the oldest or something).

The idea is good, but I think it needs to be contextualized better. Perhaps a smaller font and two lines: "My assets", and "Accounts I follow" .. something like that.

Thoughts?

svk

Quote from: karnal on November 27, 2015, 09:04:22 AM
ATTENTION The "display total assets in chosen currency" is buggy. It only shows the totals for ONE account.

Furthermore, if an(other) account is selected, it updates, then back to the dashboard, and the "total" remains as the single total of the previously chosen account.

This is not a bug per se. It could of course be expanded to include all accounts that you control (and only those you control then), but right now it only includes the "current account" which is either the account you choose in the trade section, or the last account that you've viewed.
Worker: dev.bitsharesblocks

karnal

ATTENTION The "display total assets in chosen currency" is buggy. It only shows the totals for ONE account.

Furthermore, if an(other) account is selected, it updates, then back to the dashboard, and the "total" remains as the single total of the previously chosen account.

karnal

Don't forget to update the download link at bitshares.org!

Also, can we have GPG signatures, or at the very least SHA256 checksums on the binaries ? (mention them when posting a new release announcement here)

wmbutler

Actually, @svk is the machine. If you guys want to give props, please do so for him. As far as I can tell he's doing this without any formal agreements (yet) from CNX. If he were to submit a worker proposal for his efforts, I'd fully support it.
PTS: PnBVP1iLTsV6U8z4BeJYhF8jMpkLhtTi9r
BTS2.0: billbutler
There are 10 kinds of people. Those who understand binary and those who do not.

speedy

Btw I had to checkout the BitShares branch, which is 1 commit ahead of this new release, yet is titled:

Quotea couple of small fixes before 2.0.151125 release

38PTSWarrior

Enables this version oneself :D to claim all balances in time?

Akado

Quote from: svk on November 26, 2015, 08:31:43 PM
Quote from: Akado on November 26, 2015, 08:29:59 PM
Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.

It does, although I had a report the cancel button doesn't work.

Well, at least the user can be aware of his open orders, that's the most important.
[url="https://metaexchange.info"]https://metaexchange.info[/url] | Bitcoin<->Altcoin exchange | Instant | Safe | Low spreads

svk

Quote from: Akado on November 26, 2015, 08:29:59 PM
Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.

It does, although I had a report the cancel button doesn't work.
Worker: dev.bitsharesblocks

Akado

Does the "Orders" tab work now and show all our orders? I don't see it on the the fix list of this week and the next. Seems like it could be added as well.
[url="https://metaexchange.info"]https://metaexchange.info[/url] | Bitcoin<->Altcoin exchange | Instant | Safe | Low spreads

jakub

 +5%
Amazing progress.
Really well done, guys.

I've updated the roadmap to include the major things that you've done in this milestone
http://neura-sx.github.io/roadmap.xml

bitacer


Thom

@wmbutler you are knocking it out of the park with your efforts to report progress and keep the team focused.

+5% +5% +5% +5% +5% Keep it up, you're doing great!

THANK YOU!
Injustice anywhere is a threat to justice everywhere - MLK |  Verbaltech2 Witness Reports: [url="https://bitsharestalk.org/index.php/topic,23902.0.html"]https://bitsharestalk.org/index.php/topic,23902.0.html[/url]

testz

Quote from: Pheonike on November 26, 2015, 04:33:22 PM

Don't see the back/forward buttons in web wallet.

In web wallet you can use browser back/forward buttons.  :)
Back/forward buttons was added into light wallet.

Pheonike


Don't see the back/forward buttons in web wallet.

wmbutler

@sudo language setting saving is definitely broken on the light wallets. It's fine on the web wallet. Tracking at:

https://github.com/cryptonomex/graphene-ui/issues/544
PTS: PnBVP1iLTsV6U8z4BeJYhF8jMpkLhtTi9r
BTS2.0: billbutler
There are 10 kinds of people. Those who understand binary and those who do not.


luckybit

[url="https://metaexchange.info"]https://metaexchange.info[/url] | Bitcoin<->Altcoin exchange | Instant | Safe | Low spreads

sudo

still  can't save language settings
default  english
change to chinese   works
closed &reopen bts   english again!!!!!!!!!!!!!!!!!!!!!!!

wmbutler

PTS: PnBVP1iLTsV6U8z4BeJYhF8jMpkLhtTi9r
BTS2.0: billbutler
There are 10 kinds of people. Those who understand binary and those who do not.