hush
/
SilentDragonLite
9
1
Fork 4
Code Issues 38 Pull Requests Projects Releases 8 Wiki Activity
Labels Milestones
New Issue

"Back" button does not work on "New Wallet Wizard" page, that shows the seedphrase #67

Closed
opened 2 years ago by duke · 2 comments
duke commented 2 years ago
Owner

The "Cancel" button also does nothing. But the user is able to click the X at the top of the window, which shows the main GUI window and the wallet starts to sync. If the user again clicks the X at the top of the window to close that GUI window, the wallet still continues to sync in the background, as shown from STDOUT output. If I press CTRL-C to stop it, SDL coredumps. Not ideal.

@onryo @jahway603 if someone has generated a new seed, should we even have a "Back" button?

The "Cancel" button also does nothing. But the user is able to click the X at the top of the window, which shows the main GUI window and the wallet starts to sync. If the user again clicks the X at the top of the window to close that GUI window, the wallet still continues to sync in the background, as shown from STDOUT output. If I press CTRL-C to stop it, SDL coredumps. Not ideal. @onryo @jahway603 if someone has generated a new seed, should we even have a "Back" button?
fekt commented 2 years ago
Collaborator

Should also be fixed here: ff8692fa39

There was code disabling the cancel button and cancelEvent slot also not setup correctly. With cancel button working, I ended up disabling the "X" at top of the window to prevent funky stuff from happening. User either has to complete wizard setup or close the app entirely. I also removed back button.

Should also be fixed here: https://git.hush.is/hush/SilentDragonLite/commit/ff8692fa391ccb6ea889002a6fff14419a5e68c6 There was code disabling the cancel button and cancelEvent slot also not setup correctly. With cancel button working, I ended up disabling the "X" at top of the window to prevent funky stuff from happening. User either has to complete wizard setup or close the app entirely. I also removed back button.
👍 1
duke commented 2 years ago
Poster
Owner

@fekt nice work on fixing this!

@fekt nice work on fixing this!
duke closed this issue 2 years ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
blake3
danger
denio
dev
duke
hushchat
import_zecw
jahway603
language
master
mempool
onryo
qtjson
recurring_on
rescanheight
russian
show_server
Labels
Apply labels
Clear labels
0-500 bounty
bounty up to 500 HUSH 2001-5000 bounty
bounty between 2001 and 5000 HUSH 501-2000 bounty
bounty between 501 and 2000 HUSH arm
something doesn't work on arm beginners
for new developers bug
may or may not be a bug build
problems building documentation
not enough information feature
new feature high priority
high priority i2p
related to i2p low priority
low priority medium priority
medium priority question
something is not clear release
release label or issue related to it tor
related to tor translation
translation update windows
related to windows wontfix
this won't be fixed
No Label 0-500 bounty 2001-5000 bounty 501-2000 bounty arm beginners bug build documentation feature high priority i2p low priority medium priority question release tor translation windows wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes