diff options
| author | Robert Speicher <rspeicher@gmail.com> | 2016-06-13 16:23:38 -0400 |
|---|---|---|
| committer | Robert Speicher <rspeicher@gmail.com> | 2016-06-13 16:23:38 -0400 |
| commit | e20aa4581b0d17eae36d9722ee8789af47f57727 (patch) | |
| tree | 36c4cbf3ce74b229dd5eacf06e2cb99f79870996 | |
| parent | 6aefd3c3213277d80e0680809b45643ef19df4d0 (diff) | |
| download | gitlab-ce-rs-fix-note-polling.tar.gz | |
Fix note polling when a window has been hiddenrs-fix-note-polling
`refresh` was called, `refreshing` was set to true, but then because
`document.hidden` was true, `getContent` was never called, and
`refreshing` never got reset to `false`, which stopped polling entirely
until refresh.
| -rw-r--r-- | CHANGELOG | 2 | ||||
| -rw-r--r-- | app/assets/javascripts/notes.js.coffee | 6 |
2 files changed, 6 insertions, 2 deletions
diff --git a/CHANGELOG b/CHANGELOG index 7a6a14919da..a5e5c5c5c8f 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -8,6 +8,8 @@ v 8.9.0 (unreleased) - Allow enabling wiki page events from Webhook management UI - Bump rouge to 1.11.0 - Fix issue with arrow keys not working in search autocomplete dropdown + - Fix an issue where note polling stopped working if a window was in the + background during a refresh. - Make EmailsOnPushWorker use Sidekiq mailers queue - Fix wiki page events' webhook to point to the wiki repository - Don't show tags for revert and cherry-pick operations diff --git a/app/assets/javascripts/notes.js.coffee b/app/assets/javascripts/notes.js.coffee index ad216910c8d..e2d3241437b 100644 --- a/app/assets/javascripts/notes.js.coffee +++ b/app/assets/javascripts/notes.js.coffee @@ -115,12 +115,14 @@ class @Notes , @pollingInterval refresh: => - return if @refreshing is true - @refreshing = true if not document.hidden and document.URL.indexOf(@noteable_url) is 0 @getContent() getContent: -> + return if @refreshing + + @refreshing = true + $.ajax url: @notes_url data: "last_fetched_at=" + @last_fetched_at |