Only add event handlers once

We were previsouly adding multiple event handlers, which led to over-use of CPU and un-responsiveness and other weirdness.

I believe this fixes #207

Author: nrc

static/menus.js

@@ -20,6 +20,7 @@ export class Menu extends React.Component {
     }
 
     didRender() {
+        console.log("menu didRender")
         if (this.items().length == 0) {
             this.props.onClose();
             return;
@@ -82,6 +83,7 @@ export class MenuHost extends React.Component {
         let contextMenu = (ev) => {
             this.setState({ menuOpen: { "top": ev.pageY, "left": ev.pageX, target: ev.target }});
             ev.preventDefault();
+            ev.stopPropagation();
         };
 
         let onClick = null;

static/rustw.out.js

@@ -6819,6 +6819,7 @@ var MenuHost = exports.MenuHost = function (_React$Component2) {
             var contextMenu = function contextMenu(ev) {
                 _this4.setState({ menuOpen: { "top": ev.pageY, "left": ev.pageX, target: ev.target } });
                 ev.preventDefault();
+                ev.stopPropagation();
             };
 
             var onClick = null;
@@ -6951,6 +6952,7 @@ function FileResult(props) {
             };
             props.app.loadSource(file_name, highlight);
             e.preventDefault();
+            e.stopPropagation();
         };
         var snippetClick = function snippetClick(e) {
             var highlight = {
@@ -6961,6 +6963,7 @@ function FileResult(props) {
             };
             props.app.loadSource(file_name, highlight);
             e.preventDefault();
+            e.stopPropagation();
         };
 
         return _react2.default.createElement(
@@ -6986,6 +6989,7 @@ function FileResult(props) {
     var onClick = function onClick(e) {
         props.app.loadSource(file_name);
         e.preventDefault();
+        e.stopPropagation();
     };
     return _react2.default.createElement(
         'div',
@@ -7422,7 +7426,9 @@ function add_ref_functionality(self) {
         var showRefMenu = function showRefMenu(ev) {
             self.setState({ refMenu: { "top": ev.pageY, "left": ev.pageX, target: ev.target, id: id } });
             ev.preventDefault();
+            ev.stopPropagation();
         };
+        element.off("contextmenu");
         element.on("contextmenu", showRefMenu);
         element.addClass("hand_cursor");
     };
@@ -7539,6 +7545,7 @@ var SourceView = exports.SourceView = function (_React$Component) {
                 // The data for what to do on-click is encoded in the data-link attribute.
                 // We need to process it here.
                 e.preventDefault();
+                e.stopPropagation();
 
                 var docUrl = e.target.dataset.docLink;
                 if (docUrl) {

static/rustw.out.js.map

@@ -60,6 +60,7 @@ function FileResult(props) {
             };
             props.app.loadSource(file_name, highlight);
             e.preventDefault();
+            e.stopPropagation();
         };
         const snippetClick = (e) => {
             const highlight = {
@@ -70,6 +71,7 @@ function FileResult(props) {
             };
             props.app.loadSource(file_name, highlight);
             e.preventDefault();
+            e.stopPropagation();
         };
 
         return <div key={`${kind}-${count}-${l.line_start}`}>
@@ -85,6 +87,7 @@ function FileResult(props) {
     const onClick = (e) => {
         props.app.loadSource(file_name);
         e.preventDefault();
+        e.stopPropagation();
     };
     return <div>
         <div className="div_search_file_link" onClick={onClick}>{file_name}</div>

static/search.js

@@ -34,7 +34,9 @@ function add_ref_functionality(self) {
         const showRefMenu = (ev) => {
             self.setState({ refMenu: { "top": ev.pageY, "left": ev.pageX, target: ev.target, id }});
             ev.preventDefault();
+            ev.stopPropagation();
         };
+        element.off("contextmenu");
         element.on("contextmenu", showRefMenu);
         element.addClass("hand_cursor");
     }
@@ -108,6 +110,7 @@ export class SourceView extends React.Component {
             // The data for what to do on-click is encoded in the data-link attribute.
             // We need to process it here.
             e.preventDefault();
+            e.stopPropagation();
 
             var docUrl = e.target.dataset.docLink;
             if (docUrl) {
@@ -146,7 +149,9 @@ export class SourceView extends React.Component {
 
         let refMenu = null;
         if (!!this.state.refMenu) {
-            const onClose = () => this.setState({ refMenu: null });
+            const onClose = () => {
+                return this.setState({ refMenu: null });
+            };
 
             refMenu = <RefMenu app={this.props.app} location={this.state.refMenu} onClose={onClose} target={this.state.refMenu.target} id={this.state.refMenu.id} />;
         }