Skip to content

Conversation

@ivanjx
Copy link

@ivanjx ivanjx commented Aug 20, 2025

apparently the web gui's refresh rate is already configurable but there is no input in the menu that configures it. this pr adds just that.

fixes #2781

@Rukario
Copy link
Contributor

Rukario commented Aug 20, 2025

Refresh rate and time in second can be hard to get along in the UI.

    const changerr = (e) => {
      const rrs = this.prefs.refresh_rate_sec === 1 ? 5 : 1;
      this.prefs.refresh_rate_sec = rrs;
      e.target.textContent = `${rrs}s`;
    };

    const button = document.createElement('BUTTON');
    button.addEventListener('click', changerr.bind(this));
    button.textContent = `${this.prefs.refresh_rate_sec}s`;

    div.append(button);

I know my UI isn't good either but it's something I settled with using in my custom client, there's "Refresh interval" with choice between 1s and 5s (literally a button toggling between them).

I'm not sure what UI can be so I'm just brainstorming.

@ivanjx
Copy link
Author

ivanjx commented Aug 20, 2025

im just following what's already available in the preferences. since the preferences expects a number (not a boolean) i determined it is more appropriate to use number input with min max and steps rather than a toggle button (or even a select box). im currently using 2 seconds now and a toggle between just one and five is already unusable for me.

@killemov
Copy link

I agree with @ivanjx that a select box (1/2/5/10/20/50?) is more appropriate.

@Rukario Rukario mentioned this pull request Nov 3, 2025
4 tasks
@ckerr ckerr added needs update The PR has needs to be updated by the submitter type:ui labels Nov 8, 2025
@ckerr ckerr added this to the decide-after-4.1.0 milestone Nov 12, 2025
@ivanjx ivanjx force-pushed the refresh-interval-setting branch from 3951923 to 8f482f7 Compare December 12, 2025 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs update The PR has needs to be updated by the submitter scope:web type:ui

Development

Successfully merging this pull request may close these issues.

Make refreshTorrents interval configurable

5 participants