Make network graph slider easier to use #483

1. 
   rebroad commented at 10:56 am on November 25, 2021: contributor

   Currently the network graph slider has 288 possible positions, and most of these are likely never used, due to obscure durations.

   This pull request simplifies the slider to have only 12 positions, making it much easier to use, and setting the durations to more likely-wanted settings: 1m, 2m, 5m, 10m, 20m, 30m, 1h, 2h, 3h, 6h, 12h, 24h.

   This change also allows easier rearranging of the window, for example, adding an extra button next to the reset button, shrinking the slider, and therefore making it still easy to use when smaller in length.

2. Make network graph slider easier to use ceb5881a2f
3. hebasto added the label UX on Nov 25, 2021
4. shaavan approved
5. 
   shaavan commented at 10:21 am on November 26, 2021: contributor

   I like the idea of reducing the number of options and removing the redundant ones. I also think these 12 options are sufficient for any general user’s need.

   I was able to test this PR successfully on Ubuntu 20.04. Here’s the screenshot of the Network Traffic window taken of the PR branch (slider at 20 minutes).

   

6. 
   katesalazar commented at 11:13 pm on November 27, 2021: contributor

   Concept NAK, 12 sliders are too few positions for such a large slider.

   I booted it and tested the 12 sliders. It feels like few positions. Please iterate.

7. 
   jarolrod commented at 6:39 am on November 28, 2021: member

   I think this is fine to do, but I’m concept 0 on this.

   Currently the network graph slider has 288 possible positions, and most of these are likely never used, due to obscure durations.

   I think this is true.

   This pull request simplifies the slider to have only 12 positions, making it much easier to use

   How exactly does this make the slider “easier to use”? The workflow of using the slider as a slider remains untouched

8. 
   rebroad commented at 11:49 am on December 7, 2021: contributor

   Concept NAK, 12 sliders are too few positions for such a large slider.

   I booted it and tested the 12 sliders. It feels like few positions. Please iterate. @katesalazar This is intentional as there is talk about adding an extra button, and so the slider will be reduced in size in the future, but I do agree with you as it is.

   What extra durations do you propose being added? I propose 1-10 in steps of 1, 10-60 in steps of 5, 1hr to 2hr in steps of 0.5hrs, 2hr to 6hr in steps of 1hr, 6hrs to 24hrs in steps of 3hrs. That’s a total of 10+10+2+4+3 = 29 steps.

   or 1,2,3,5,10,15,20,30,1h,2h,3h,6h,12h,18h,24h ? I really question whether people want 6,7,8,9,25,35,45,50,55,1.5h,4h,5h,9h,15h,21h… these seem unlikely to be used.

9. 
   rebroad commented at 11:52 am on December 7, 2021: contributor

   How exactly does this make the slider “easier to use”? The workflow of using the slider as a slider remains untouched @jarolrod because it is faster to use, i.e. less moving left and right to get the exact duration desired (usually a round number).

10. 
    katesalazar commented at 8:05 pm on December 10, 2021: contributor

    nice, looking forward to the button

    On Tue, Dec 7, 2021 at 12:49 PM Rebroad @.***> wrote:

    Concept NAK, 12 sliders are too few positions for such a large slider.

    I booted it and tested the 12 sliders. It feels like few positions. Please iterate.

    This is intentional as there is talk about adding an extra button, and so the slider will be reduced in size in the future, but I do agree with you as it is.

    — You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/bitcoin-core/gui/pull/483#issuecomment-987852952, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMRS4W674BMU6YHNH2MBLSDUPXYGDANCNFSM5IYHJOAQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

11. 
    DrahtBot commented at 5:42 am on February 25, 2022: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    No conflicts as of last run.

12. 
    DrahtBot commented at 1:21 am on May 6, 2023: contributor

    There hasn’t been much activity lately. What is the status here?

    Finding reviewers may take time. However, if the patch is no longer relevant, please close this pull request. If the author lost interest or time to work on this, please close it and mark it ‘Up for grabs’ with the label, so that it can be picked up in the future.

13. 
    hebasto commented at 7:05 pm on May 18, 2023: member

    There hasn’t been much activity lately. What is the status here?

    Closing due to lack of interest.

14. hebasto closed this on May 18, 2023

    ---

15. bitcoin-core locked this on May 17, 2024

Contributors
rebroad shaavan katesalazar jarolrod DrahtBot hebasto

Labels
UX