menu.js error (missing: .js-context-menu) about photon HOT 2 OPEN

I believe it's supposed to be referring to something similar to this:
https://github.com/connors/photon/blob/master/docs/demo-app.html#L51-L53

... but that doesn't appear to have been included in the template-app example.

from photon.

I think it's an incomplete adaptation of https://github.com/atom/electron/blob/master/docs/api/menu.md.

window.addEventListener('contextmenu', function (e) {
  e.preventDefault();
  menu.popup(remote.getCurrentWindow());
}, false);

One way around the error produced by the current implementation in this repo would be to modify it like this:

document.addEventListener('DOMContentLoaded', function () {
  var nodeList = document.querySelectorAll('.js-context-menu')

  Array.prototype.forEach.call(nodeList, function (item) {
    item.addEventListener('contextmenu', function (event) {
      menu.popup(remote.getCurrentWindow());
    })
  })
})

That way there will be no error if the node list is empty. I'm using this workaround in https://github.com/ngoldman/proton without any issues so far.

from photon.