Skip to content

Refresh popup data when opening module popups#2988

Open
eyupcanakman wants to merge 1 commit intoexelban:masterfrom
eyupcanakman:fix/popup-refresh-on-open-2140
Open

Refresh popup data when opening module popups#2988
eyupcanakman wants to merge 1 commit intoexelban:masterfrom
eyupcanakman:fix/popup-refresh-on-open-2140

Conversation

@eyupcanakman
Copy link

Summary

  • prime popup readers when a popup is reopened
  • replay latest cached values from non-popup readers when a popup opens
  • avoid ad-hoc concurrent non-popup reads from popup visibility handling
  • add regression tests for popup reader reopen and replay behavior

Why

Fixes #2140.

With higher update intervals, popup values could stay stale until the next scheduled update.

Test plan

  • xcodebuild -project Stats.xcodeproj -scheme Stats -destination 'platform=macOS' -derivedDataPath /tmp/stats-dd test -only-testing:Tests CODE_SIGNING_ALLOWED=NO
  • xcodebuild -project Stats.xcodeproj -scheme Stats -destination 'platform=macOS' -configuration Release -derivedDataPath /tmp/stats-dd archive CODE_SIGNING_ALLOWED=NO

- Prime popup readers when reopened\n- Replay cached values for non-popup readers when popup opens\n- Add reader lifecycle regression tests for reopen and replay behavior
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.

[Feature Request] Refresh data when clicking menu bar item.

2 participants