Skip to content

fix: improve hover text 'unknown' for better UX.#47

Merged
MantisClone merged 4 commits intoRequestNetwork:masterfrom
PijiDev:master
Jan 16, 2023
Merged

fix: improve hover text 'unknown' for better UX.#47
MantisClone merged 4 commits intoRequestNetwork:masterfrom
PijiDev:master

Conversation

@PijiDev
Copy link
Contributor

@PijiDev PijiDev commented Jan 2, 2023

Fixes #38

@MantisClone MantisClone changed the title Fix: improve hover text 'unknown' for better UX. fix: improve hover text 'unknown' for better UX. Jan 2, 2023
@MantisClone MantisClone self-requested a review January 2, 2023 15:45
@PijiDev
Copy link
Contributor Author

PijiDev commented Jan 4, 2023

The build now works, for now I kept the ? 'The balance computation might have failed', because the original code expects a conditional string.

@PijiDev PijiDev requested a review from MantisClone January 6, 2023 06:14
Copy link
Member

@MantisClone MantisClone left a comment

Choose a reason for hiding this comment

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

Thanks for taking a second look at this. I like that you changed the state name from unknown to waiting. However, I think the hover text can still be improved.

@MantisClone MantisClone merged commit 72f0a65 into RequestNetwork:master Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[create] "balance computation failed" improve hover text

2 participants