huijunw 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_r246290990
##########
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:
besides comments, shall we add the instruction somewhere, so that users can
easily find it and know how to operate the UI webpage?
----------------------------------------------------------------
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
