-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[VDG] Display fee rate & estimated time constantly instead of tooltip #7638
Conversation
Co-authored-by: ichthus1604 <ichthus1604@gmail.com>
It was downvoted. That is why @ichthus1604 reverted it. |
Why was it downvoted? IMO given that the fee graph window is for advanced users it makes more sense to display those value constantly rather than requiring the user to hover over the slider to see them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Hey, I found this one and it looks there were a consensus to not accept these changes. |
What was the reason to not accept these changes? As I stated above the fee graph is for advanced users anyway so it makes sense to display the fee rate values and the estimation time constantly. |
@yahiheb I'm closing this PR due to reasons previously stated. There is consensus to not introduce these changes. |
@ichthus1604 I still don't know what are the reasons to not accept these changes, and the comment you linked simply says that Dan asked to revert those changes but without any explanation. |
@yahiheb the idea was not to visually highlight or put the fee rate into a position where it can grab too much user attention. |
Is the fee graph in the normal send work flow? I guess the answer to that is no. And as I said repeatedly before the fee graph is for advanced users and it makes sense to display the fee rate values constantly instead of having to hover over the slider to get that. |
This PR addresses one of the issues mentioned in #10488.
|
Why is that?
For a better UX. To allow users to see that information easily without hovering over the slider to get that. |
They are just put there and centralized. Normally the values should be aligned in a column like:
And put them somewhere inside the chart to avoid the crowded bottom part and have the chart on almost full dialog size. |
Agree they should be put inside chart to save vertical space and will look much better. |
@soosr @wieslawsoltes I agree, but I do not know how to do it, so please feel free to take this and do that. |
@wieslawsoltes Should we add it to the chart directly, or just put an overlay on the chart? |
Overlay, adding to chart adds unnecessary complexity. |
This has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
The chart dialog is not displayed in the default workflow, and some argue it is for advanced users so I think this is good enough for now, and later on the UX can be improved more. |
2 years until merge, lol, glad it's over the finish line 🖤 |
Fixes #6980
Credit goes to @ichthus1604 for writing this code which was reverted by 07e1755 in #7212