Resolve comments

elastic · Nov 24, 2020 · b263727 · b263727
1 parent 24e66c0
commit b263727
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/src/plugins/vis_type_timeseries/public/application/visualizations/lib/active_cursor.ts b/src/plugins/vis_type_timeseries/public/application/visualizations/lib/active_cursor.ts
@@ -18,11 +18,6 @@
  */
 
 import { Subject } from 'rxjs';
+import { PointerEvent } from '@elastic/charts';
 
-export const ACTIVE_CURSOR = 'ACTIVE_CURSOR';
-
-interface ActiveCursor {
-  cursor: any;
-}
-
-export const activeCursor$ = new Subject<ActiveCursor>();
+export const activeCursor$ = new Subject<PointerEvent>();
diff --git a/src/plugins/vis_type_timeseries/public/application/visualizations/views/timeseries/index.js b/src/plugins/vis_type_timeseries/public/application/visualizations/views/timeseries/index.js
@@ -54,7 +54,7 @@ const generateAnnotationData = (values, formatter) =>
 const decorateFormatter = (formatter) => ({ value }) => formatter(value);
 
 const handleCursorUpdate = (cursor) => {
-  activeCursor$.next({ cursor });
+  activeCursor$.next(cursor);
 };
 
 export const TimeSeries = ({
@@ -73,8 +73,8 @@ export const TimeSeries = ({
   const chartRef = useRef();
 
   useEffect(() => {
-    const updateCursor = ({ cursor }) => {
-      if (chartRef.current && cursor) {
+    const updateCursor = (cursor) => {
+      if (chartRef.current) {
         chartRef.current.dispatchExternalPointerEvent(cursor);
       }
     };