Enhanced Darkcoin Wallet UI

Status
Not open for further replies.
moli said:
I'm not sure I understand what you mean?
Click to expand...
Sometimes when you click repair, wallet crash, but somehow repair function can start wallet again (which is the goal),
but crashed wallet is still in the memory, thats why you get that 'already running' error.
 
AjM said:
Sometimes when you click repair, wallet crash, but somehow repair function can start wallet again (which is the goal),
but crashed wallet is still in the memory, thats why you get that 'already running' error.
Click to expand...
Yes, got it. Thanks. :)
 
  • Like
Reactions: AjM
crowning UdjinM6

I put this in here because the list is all about the gui.
Relevant ver: v0.12.0.37 (5.8.2015) -->>

My GUI & translation error/bug/fix/memo/wish list:

1. Source english is still with this: -Can vary +/- %1 satoshi(s) per input.

2. Wallet repair->rescan button->text is too narrow, text truncated (Finnish).

3. All help menu objects (cmd opt, both about) does not use latest translations, its old data there.

4. Tooltip text are in some cases too wide, half a meter :) (options->wallet->Spend unconfirmed change)

5. Transactions-> if selected row, selected amount is calculated ok, it should reset if you deselect or go to other tab.

6. Send->coin control->coin selection -> address and date columns are truncated and too narrow.

7. Splash screen on startup does not use latest translations.

8. Tools->Peers list -> right side data should reset if user deselects row or go to other tab or selected row disappear.

9. Found from the log:
2015-07-06 19:02:51 GUI: QLabel(0xb0049b8, name = "labelCoinControlChangeLabel") does not have a property named "padding"

10. Transactions-> Min amount box -> cant input dot(.), only comma (,), so you cant enter amounts below 1 or any decimal amount.

11. Tools window -> Information -> Number of Masternodes -> row is not translated (included to source english).

12. Litemode does not hide Darksend options (rounds/amount) in options -> main tab.

13. Command-line options -> -litemode=<n> should also say this also is going to disable instantx sending option.

Edit: 1, 2, 5, 6, 9, 10 done by crowning .
Edit: 3, 4, 7, 11, 13 done by UdjinM6 .
 
Last edited by a moderator:
AjM said:
It is amount setting info, why we need 6 or 8 decimal for that?
Its looks so silly 10.12345678 tDASH, i like old 2 decimal better 10.12 tDASH or without decimals 10 tDASH.
Click to expand...

Just FYI, I did a little proof of concept, maybe Evan accepts it:
https://github.com/dashpay/dash/pull/403

Note the number of decimal digits on the left side of the Overview-page:
Wallet.jpg
 
Configure choice of decimal places? Nice! I like that!

Edit: How could he not accept that?
 
Last edited by a moderator:
AjM said:
Ah, you made it two decimals to ALL left amounts, nice, i like it.
I was talking about mixing amount & rounds setting, but i take it, thanks.
Click to expand...

It's a start...my first version changed it everywhere, but this is not possible at places where users cut & paste the amount (e.g. when sending funds via coin-control I normally copy the amount after fee from there...not good when it's rounded).
 
crowning said:
It's a start...my first version changed it everywhere, but this is not possible at places where users cut & paste the amount (e.g. when sending funds via coin-control I normally copy the amount after fee from there...not good when it's rounded).
Click to expand...
Yep, overview tab is... well overview page, i think decimals in 'overview' is not so critical.:smile:
 
Could it be an idea to instead of typing commands in debug console, we could access "masternode" and "mnbudget" commands via GUI?
Maybe another pull-down menu on the main menu bar (as like Tools), or another choice on display window (as like Overview, Send, Receive)...
call it "MN tools", or "MN control" or some such?
 
AjM said:
crowning UdjinM6

I put this in here because the list is all about the gui.
Relevant ver: v0.12.0.7-d39815d (2.7.2015) -->>

My GUI & translation error/bug/fix/memo/wish list:

1. Source english is still with this: -Can vary +/- %1 satoshi(s) per input.

2. Wallet repair->rescan button->text is too narrow, text truncated (Finnish).

3. All help menu objects (cmd opt, both about) does not use latest translations, its old data there.

4. Tooltip text are in some cases too wide, half a meter :) (options->wallet->Spend unconfirmed change)

5. Transactions-> if selected row, selected amount is calculated ok, it should reset if you deselect or go to other tab.

6. Send->coin control->coin selection -> address and date columns are truncated and too narrow.

7. Splash screen on startup does not use latest translations.

8. Tools->Peers list -> right side data should reset if user deselects row or go to other tab or selected row disappear.

9. Found from the log:
2015-07-06 19:02:51 GUI: QLabel(0xb0049b8, name = "labelCoinControlChangeLabel") does not have a property named "padding"
2015-07-06 19:02:51 GUI: QLabel(0xb0049b8, name = "labelCoinControlChangeLabel") does not have a property named "padding"

Edit: added no.7.
Edit2: added no.8.
Edit3: typo fix.
Edit4: added no.9.
Click to expand...


Updated 6.7.2015.
https://dashtalk.org/threads/enhanced-darkcoin-wallet-ui.1705/page-25#post-58570
 
moli said:
...
Click to expand...

Those should be fixed now.

Remarks:
  1. the spaces in "Available" amounts (and everywhere else) are the way Bitcoin does display numbers to avoid confusion with separators from other locales, e.g. in German "." and "," have the exactly opposite meaning. Makes sense to leave it like that.
  2. same place: you can't make the field wide enough to properly display every(!) possible amount of DASH in duffs. Remember that your screenshot shows are more than 100000 DASH.
  3. when you use duffs the "Amound and Rounds" field doesn't show rounds. I guess we can live with that.
I'll surely fix 2+3 when the value of 1 DASH is so high that 99.9% of all users use duffs as default unit.
In other words, when I already live on my own island...
joint.gif
 
Last edited by a moderator:
crowning said:
Those should be fixed now.

Remarks:
  1. the spaces in "Available" amounts (and everywhere else) are the way Bitcoin does display numbers to avoid confusion with separators from other locales, e.g. in German "." and "," have the exactly opposite meaning. Makes sense to leave it like that.
  2. same place: you can't make the field wide enough to properly display every(!) possible amount of DASH in duffs. Remember what your screenshot shows are more than 100000 DASH.
  3. when you use duffs the "Amound and Rounds" field doesn't show rounds. I guess we can live with that.
I'll surely fix 2+3 when the value of 1 DASH is so high that 99.9% of all users use duffs as default unit.
In other words, when I already live on my own island...
joint.gif
Click to expand...
Thanks so much, crowning. I don't believe dash will ever go that high but I can hope and I'd fly to visit you on your island from my own island..... LOL
 
AjM said:
1: Overview-> available and pending backround colors missing, theme is messed some how.
2: Transactions-> if selected row, selected amount is calculated ok, it should reset if you deselect or go to other tab.
3: Send->coin control->coin selection -> address and date columns are truncated and too narrow.
4: Tools->Wallet repair -> recover transactions 1 -> wallet crashed immediately, tested only this.
Click to expand...

Status update:
  1. Should be fixed now
  2. that's a tough one. Reset on de-select could be possible (needs some investigations), but de-selecting on tab-change is a can of worms I surely will NOT open...sorry, but it's WAY too much work with little gain.
  3. since it's impossible to fit everything in there (the Coin Selection screen is already VERY wide) I've added tooltips to the address and date columns. Not perfect, but does the job. See https://github.com/dashpay/dash/pull/334
  4. Fixed, see https://github.com/dashpay/dash/pull/392
Edit: just improved 2: a bit, see https://github.com/dashpay/dash/pull/423
 
Last edited by a moderator:
crowning said:
Status update:
  1. Should be fixed now
  2. that's a tough one. Reset on de-select could be possible (needs some investigations), but de-selecting on tab-change is a can of worms I surely will NOT open...sorry, but it's WAY too much work with little gain.
  3. since it's impossible to fit everything in there (the Coin Selection screen is already VERY wide) I've added tooltips to the address and date columns. Not perfect, but does the job. See https://github.com/dashpay/dash/pull/334
  4. Fixed, see https://github.com/dashpay/dash/pull/392
Edit: just improved 2: a bit, see https://github.com/dashpay/dash/pull/423
Click to expand...
Ok, thanks, but its not over yet:smile:
Do not forget this: https://dashtalk.org/threads/enhanced-darkcoin-wallet-ui.1705/page-25#post-58570
 
Status
Not open for further replies.
Back
Top