Skip to content
This repository has been archived by the owner on Jun 4, 2018. It is now read-only.

Published transaction should display transaction id #74

Closed
marcopeereboom opened this issue Jul 5, 2016 · 4 comments
Closed

Published transaction should display transaction id #74

marcopeereboom opened this issue Jul 5, 2016 · 4 comments
Assignees
@jrick
Labels
UX
Milestone
0.3.0

Comments

@marcopeereboom
Copy link
Member

This message box should show a selectable label with the transaction id.

published
@marcopeereboom marcopeereboom added this to the 0.1.0-alpha milestone Jul 5, 2016
@jrick
Copy link
Member

jrick commented Jul 5, 2016

See #18. MessageBox is a Winforms feature, we shouldn't be depending on it in a WPF app. I don't even know if it's possible to make text in MessageBoxes selectable.

We can display the transaction hash somewhere else in the create transaction view after it is published.

@marcopeereboom
Copy link
Member Author

Honestly I hate the message box altogether but we need some sort of indicator to the user.

Not sure if this is the right spot to complain about it but the sign button should be disabled upon click.

@marcopeereboom marcopeereboom modified the milestones: 0.2.0-alpha, 0.1.0-alpha Jul 15, 2016
@marcopeereboom marcopeereboom changed the title Published transaction message box should display transaction id Published transaction should display transaction id Jul 25, 2016
@marcopeereboom
Copy link
Member Author

Per discussion don't use a message box; use a copy/pastable label instead.

@jrick
Copy link
Member

jrick commented Jul 28, 2016

image

@jrick jrick self-assigned this Jul 28, 2016
@jrick jrick mentioned this issue Jul 28, 2016
Merged
jrick added a commit that referenced this issue Jul 28, 2016
@jrick
Show "published" message directly in create tx view.
4eebccf 
Show the transaction hash in this message as well, and make it
copyable.

The message and transaction hash are hidden whenever switching to the
create tx view.

Fixes #74.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jrick jrick

Labels
UX
Projects
None yet
Milestone
0.3.0
Development

No branches or pull requests
2 participants
@jrick @marcopeereboom