nwangtw commented on a change in pull request #3147: [UI] Move reset view
button to left and change plan graph listener fr…
URL: https://github.com/apache/incubator-heron/pull/3147#discussion_r246462287
##########
File path: heron/tools/ui/resources/static/js/plan-controller.js
##########
@@ -184,8 +184,8 @@ function PlanController(baseUrl, cluster, environ,
toponame, physicalPlan, logic
highlightFocusedElements();
};
- // treat any click that bubbles up unhandled as intent to clear the focus
- $(document).on('click', ".plans .graphics", function () {
+ // treat double click as intent to clear the focus
Review comment:
We dont have an instruction page for heorn UI I think currently. Could be
useful to have one though.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
