You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by bb...@apache.org on 2022/03/02 00:47:06 UTC

[airflow] 05/08: fix hover and extra prop

This is an automated email from the ASF dual-hosted git repository.

bbovenzi pushed a commit to branch mapped-task-drawer
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit ce953b379925f3fba3ab0f0053b434f943631451
Author: Brent Bovenzi <br...@gmail.com>
AuthorDate: Mon Feb 28 21:01:53 2022 -0500

    fix hover and extra prop
---
 airflow/www/static/js/tree/StatusBox.jsx      | 5 +++--
 airflow/www/static/js/tree/renderTaskRows.jsx | 1 -
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/airflow/www/static/js/tree/StatusBox.jsx b/airflow/www/static/js/tree/StatusBox.jsx
index 815e6a1..0953a79 100644
--- a/airflow/www/static/js/tree/StatusBox.jsx
+++ b/airflow/www/static/js/tree/StatusBox.jsx
@@ -37,7 +37,7 @@ const StatusBox = ({
   const hoverBlue = `${colors.blue[100]}50`;
 
   // Fetch the corresponding column element and set its background color when hovering
-  const onMouseOver = () => {
+  const onMouseEnter = () => {
     if (selectedInstance.runId !== runId) {
       [...containerRef.current.getElementsByClassName(`js-${runId}`)]
         .forEach((e) => { e.style.backgroundColor = hoverBlue; });
@@ -50,6 +50,7 @@ const StatusBox = ({
 
   const onClick = (e) => {
     e.stopPropagation();
+    onMouseLeave();
     onSelectInstance(instance);
   };
 
@@ -73,7 +74,7 @@ const StatusBox = ({
           cursor={!group.children && 'pointer'}
           data-testid="task-instance"
           zIndex={1}
-          onMouseEnter={onMouseOver}
+          onMouseEnter={onMouseEnter}
           onMouseLeave={onMouseLeave}
           {...rest}
         >
diff --git a/airflow/www/static/js/tree/renderTaskRows.jsx b/airflow/www/static/js/tree/renderTaskRows.jsx
index 9b454f0..7440774 100644
--- a/airflow/www/static/js/tree/renderTaskRows.jsx
+++ b/airflow/www/static/js/tree/renderTaskRows.jsx
@@ -98,7 +98,6 @@ const TaskInstances = ({
             key={key}
             instance={instance}
             containerRef={containerRef}
-            extraLinks={task.extraLinks}
             group={task}
             onSelectInstance={onSelectInstance}
             selectedInstance={selectedInstance}