Fix other listeners

8d43fe4e · Luke Bennett · 98da0a8b · 8d43fe4e · 8d43fe4e
Commit 8d43fe4e authored Apr 13, 2018 by Luke Bennett
Showing with 4 additions and 4 deletions

app/assets/javascripts/feature_highlight/feature_highlight.js ...assets/javascripts/feature_highlight/feature_highlight.js +2 -2

app/assets/javascripts/milestone.js app/assets/javascripts/milestone.js +2 -2

No files found.
--- a/app/assets/javascripts/feature_highlight/feature_highlight.js
+++ b/app/assets/javascripts/feature_highlight/feature_highlight.js
@@ -31,8 +31,8 @@ export function setupFeatureHighlightPopover(id, debounceTimeout = 300) {
    .on('mouseenter', mouseenter)
    .on('mouseleave', debouncedMouseleave(debounceTimeout))
    .on('inserted.bs.popover', inserted)
-    .one('show.bs.popover', () => {
+    .on('show.bs.popover', () => {
-      window.addEventListener('scroll', hideOnScroll);
+      window.addEventListener('scroll', hideOnScroll, { once: true });
    })
    // Display feature highlight
    .removeAttr('disabled');

--- a/app/assets/javascripts/milestone.js
+++ b/app/assets/javascripts/milestone.js
@@ -61,8 +61,8 @@ export default class Milestone {
    })
    .on('mouseenter', mouseenter)
    .on('mouseleave', debouncedMouseleave())
-    .one('show.bs.popover', () => {
+    .on('show.bs.popover', () => {
-      window.addEventListener('scroll', hideOnScroll);
+      window.addEventListener('scroll', hideOnScroll, { once: true });
    });
  }
 }