Skip to content

fix: app unresponsive#8189

Merged
CurryYangxx merged 1 commit intodevelopfrom
fix/app-unresponsive
Nov 21, 2024
Merged

fix: app unresponsive#8189
CurryYangxx merged 1 commit intodevelopfrom
fix/app-unresponsive

Conversation

@CurryYangxx
Copy link
Member

@CurryYangxx CurryYangxx commented Nov 21, 2024

Fix #7445

Based on the user's feedback, the app's unresponsiveness is caused by this interval logic.

Changes:

  • listen focus and blur event, stop interval when the app is not focus
  • trigger sync when user come back to app

@CurryYangxx CurryYangxx merged commit 73b03d0 into develop Nov 21, 2024
@CurryYangxx CurryYangxx deleted the fix/app-unresponsive branch November 21, 2024 09:38
@sentry
Copy link

sentry bot commented Nov 27, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: Expected fetch controller: :r30: app:///index.html View Issue
  • ‼️ ApplicationNotResponding: Application Not Responding for at least 5000 ms triggerSync(src/ui/components/dropdowns/sync-dr... View Issue
  • ‼️ ApplicationNotResponding: Application Not Responding for at least 5000 ms <anonymous>(src/main/window-utils) View Issue
  • ‼️ ApplicationNotResponding: Application Not Responding for at least 5000 ms <anonymous>(src/main/window-utils) View Issue
  • ‼️ ApplicationNotResponding: Application Not Responding for at least 5000 ms S3(src/common/constants) View Issue

Did you find this useful? React with a 👍 or 👎

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.

Application Freezes / Unresponsive

3 participants