Skip to content

Fix refresh button not resetting countdown timer (#153)#154

Merged
ParhamTehrani merged 1 commit intodevelopmentfrom
fix/153-refresh-button-timer
Mar 9, 2026
Merged

Fix refresh button not resetting countdown timer (#153)#154
ParhamTehrani merged 1 commit intodevelopmentfrom
fix/153-refresh-button-timer

Conversation

@ParhamTehrani
Copy link
Copy Markdown
Contributor

  • Fix clearTimeout to clearInterval for interval handle
  • Add _lastManualRefreshTime to track manual refresh events
  • Show 0:00 for 2 seconds after manual refresh to provide visual feedback
  • Replace unused _refresh_timer with functional _lastManualRefreshTime

Close #153

Describe your changes

...

Submission Review Guidelines:

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Will this be part of a product update? If yes, please write one phrase about this update.
  • I have reviewed my code for security best practices.
  • Following the above guidelines will result in quick merges and clear and detailed feedback when appropriate.
  • My code follows the style guidelines of this project
  • I have updated the change-log in CHANGELOG.md.

Type of change

  • Fix - Fixes an existing bug
  • Add - Adds functionality
  • Update - Update existing functionality
  • Dev - Development related task
  • Tweak - A minor adjustment to the codebase
  • Performance - Address performance issues
  • Enhancement - Improvement to existing functionality

- Fix clearTimeout to clearInterval for interval handle
- Add _lastManualRefreshTime to track manual refresh events
- Show 0:00 for 2 seconds after manual refresh to provide visual feedback
- Replace unused _refresh_timer with functional _lastManualRefreshTime

Fixes #153

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ParhamTehrani ParhamTehrani merged commit fe5fdb0 into development Mar 9, 2026
@parhumm parhumm deleted the fix/153-refresh-button-timer branch March 9, 2026 17:42
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.

1 participant