ember-nf-graph - npm Package Compare versions

Comparing version 1.0.0-beta.25 to 1.0.0-beta.26

addon/mixins/graph-graphic-with-tracking-dot.js

@@ -133,3 +133,4 @@ import Ember from 'ember';
 
-    var handler = e => {
+    var mousemoveHandler = e => {
+      this._hovered = true;
       var evt = this._getEventObject(e);
@@ -139,23 +140,27 @@ this.set('trackedData', evt);
 
-    content.on('mousemove', handler);
+    content.on('mousemove', mousemoveHandler);
 
     this._onHoverCleanup = () => {
-      content.off('mousemove', handler);
+      content.off('mousemove', mousemoveHandler);
     };
   },
 
+  _hovered: false,
+
   _onEndUntrack() {
     var content = this._content;
 
-    var handler = () => {
+    var mouseoutHandler = () => {
       this.set('trackedData', null);
     };
 
-    content.on('mouseout', handler);
+    content.on('mouseout', mouseoutHandler);
 
     this._onEndCleanup = () => {
-      content.off('mouseout', handler);
+      content.off('mouseout', mouseoutHandler);
     };
 
-    handler();
+    if(!this._hovered) {
+      this.set('trackedData', null);
+    }
   },
@@ -166,13 +171,22 @@
 
-    var handler = () => {
+    var mouseoutHandler = () => {
+      this._hovered = false;
       this.set('trackedData', this.get('lastVisibleData'));
     };
 
-    content.on('mouseout', handler);
+    var changeHandler = () => {
+      if(!this._hovered) {
+        this.set('trackedData', this.get('lastVisibleData'));
+      }
+    };
 
+    content.on('mouseout', mouseoutHandler);
+    this.addObserver('lastVisibleData', this, changeHandler);
+
     this._onEndCleanup = () => {
-      content.off('mouseout', handler);
+      content.off('mouseout', mouseoutHandler);
+      this.removeObserver('lastVisibleData', this, changeHandler);
     };
 
-    handler();
+    changeHandler();
   },
@@ -183,13 +197,22 @@
 
-    var handler = () => {
+    var mouseoutHandler = () => {
+      this._hovered = false;
       this.set('trackedData', this.get('firstVisibleData'));
     };
 
-    content.on('mouseout', handler);
+    var changeHandler = () => {
+      if(!this._hovered) {
+        this.set('trackedData', this.get('firstVisibleData'));
+      }
+    };
 
+    content.on('mouseout', mouseoutHandler);
+    this.addObserver('firstVisibleData', this, changeHandler);
+
     this._onEndCleanup = () => {
-      content.off('mouseout', handler);
+      content.off('mouseout', mouseoutHandler);
+      this.removeObserver('firstVisibleData', this, changeHandler);
     };
 
-    handler();
+    changeHandler();
   },
@@ -196,0 +219,0 @@

CHANGELOG.md

 # Changelog
 
-### v1.0.0-beta.25
+### 1.0.0-beta.26
 
+- FIX Ensure tracked data updates when firstVisibleData or lastVisibleData has updated in certain tracking modes
+
+### 1.0.0-beta.25
+
 - FIX issue where firstVisibleData and lastVisibleData would error if no data is present
 
-### v1.0.0-beta.24
+### 1.0.0-beta.24
 
 - FIX trackingData no longer throws errors when underlying data is not present
 
-### v1.0.0-beta.23
+### 1.0.0-beta.23
 
 - FIX `snap-last` and `snap-first` type trackingModes will now initialize properly and render
 
-### v1.0.0-beta.22
+### 1.0.0-beta.22
 
@@ -21,3 +25,3 @@ - FIX fix regression in `nf-line` tracking behavior
 
-### v1.0.0-beta.21
+### 1.0.0-beta.21
 
@@ -28,3 +32,3 @@ - UPDATE components with tracking capabilities (`nf-line`, `nf-area` and `nf-bars`) updated to support additional data passed with trackingData, rendered values are `renderX` and `renderY`, where actual values are still at `x` and `y`. This was to account for new stacked area functionality.
 
-### v1.0.0-beta.20
+### 1.0.0-beta.20
 
@@ -35,11 +39,11 @@ - UPDATE `nf-graph` component now has `didAutoUpdateMinX`, `didAutoUpdateMinY`, `didAutoUpdateMaxX`, `didAutoUpdateMaxX` that will trigger when the corresponding values are updated automatically by a mode of `"push-tick"`, `"auto"` or `"tick"`.
 
-### v1.0.0-beta.19
+### 1.0.0-beta.19
 
 - FIX issue where `nf-area` having null data would throw an unhandled error.
 
-### v1.0.0-beta.18
+### 1.0.0-beta.18
 
 - FIX axes components will once again use the passed templates for pre-Glimmer Ember.
 
-### v1.0.0-beta.17
+### 1.0.0-beta.17
 
@@ -55,3 +59,3 @@ - ADD `nf-tracker` component. This is a component for templatable tracking dots
 
-### v1.0.0-beta.16
+### 1.0.0-beta.16
 
@@ -61,11 +65,11 @@ - FIX [#64](https://github.com/Netflix/ember-nf-graph/issues/64) put `tickFactory` back where it belongs on the axes components
 
-### v1.0.0-beta.15
+### 1.0.0-beta.15
 
 - FIX Moved ember-new-computed to deps not devDeps
 
-### v1.0.0-beta.14
+### 1.0.0-beta.14
 
 - FIX [#63](https://github.com/Netflix/ember-nf-graph/pull/63) Uses new computed property syntax w/ polyfill for host apps running older ember
 
-### v1.0.0-beta.13
+### 1.0.0-beta.13
 
@@ -76,3 +80,3 @@ - FIX [#56](https://github.com/Netflix/ember-nf-graph/pull/56) Added rxjs to host app bower blueprint
 
-### v1.0.0-beta.12
+### 1.0.0-beta.12
 
@@ -82,3 +86,3 @@ - FIX performance issue with axes caused by graphics registering after render
 
-### v1.0.0-beta.11
+### 1.0.0-beta.11
 
@@ -88,3 +92,3 @@ - Update Ember-CLI to `0.2.3` and Ember to `~1.11.3`
 
-### v1.0.0-beta.10
+### 1.0.0-beta.10
 
@@ -96,3 +100,3 @@ - FIX [#44](//github.com/Netflix/ember-nf-graph/issues/44) remove nf-scroll-area, which was leftover from splitting out nf-table
 
-### v1.0.0-beta.9
+### 1.0.0-beta.9
 
@@ -104,11 +108,11 @@ - FIX [#16](//github.com/Netflix/ember-nf-graph/issues/16) SVG classes not swapping when selection changed
 
-### v1.0.0-beta.8
+### 1.0.0-beta.8
 
 - [CRITICAL BUGFIX] Moved babel to `dependencies` so our addon is actually transpiled in host apps #34
 
-### v1.0.0-beta.7
+### 1.0.0-beta.7
 
 - Do not use.
 
-### v1.0.0-beta.6
+### 1.0.0-beta.6
 
@@ -118,3 +122,3 @@ - UPDATE [#4](//github.com/Netflix/ember-nf-graph/issues/4) add block params to axis components.
 
-### v1.0.0-beta.5
+### 1.0.0-beta.5
 
@@ -125,3 +129,3 @@ - UPDATE name to ember-nf-graph
 
-### v1.0.0-beta.4
+### 1.0.0-beta.4
 
@@ -138,7 +142,7 @@ - DEPRECATE [#11](//github.com/netflix/ember-nf-graph/issues/11) Add warning message that name will be changing from ember-nf-graph
 
-### v1.0.0-beta.3
+### 1.0.0-beta.3
 
 - FIX [#9](//github.com/netflix/ember-nf-graph/issues/9): Blueprints now properly named
 
-### v1.0.0-beta.2
+### 1.0.0-beta.2
 - FIX Update license
@@ -149,4 +153,4 @@ - FIX updated documentation and removed document cruft
 
-### v1.0.0-beta.1
+### 1.0.0-beta.1
 
 - UPDATE: just releasing to public

package.json

 {
   "name": "ember-nf-graph",
-  "version": "1.0.0-beta.25",
+  "version": "1.0.0-beta.26",
   "description": "A graphing/charting DSL for Ember.js",
@@ -5,0 +5,0 @@ "files": [

No alert changes

Improved metrics

Total package byte prevSize

192933

increased by0.48%

Lines of code

5639

increased by0.36%

No dependency changes