Fixed a bug where 'Inspect element' would always inspect the same element. Fixed some typos.

[bbqjohan]

When you inspected your first element, it and its position was cached when creating the context menu. So the same element and position would always be passed into inspectMenuTemplate() whenever you tried to inspect another element.

I stored the element and the coordinates in variables outside the functions to fix this. You could also just remove the if statement on line 42, so this:

function onContextMenu(e) {
  if (menu === null) {
    menu = inpectElementMenu({x: e.x, y: e.y}, ifInspectable(e.target));
  }

becomes:

function onContextMenu(e) {
    menu = inpectElementMenu({x: e.x, y: e.y}, ifInspectable(e.target));

without needing to changing anything else. But that would create a new menu every time you right-click, and I thought that was something you didn't want.

Also fixed some typos I found.

[parro-it]

@bbqjohan, thank you for the patch!

But that would create a new menu every time you right-click, and I thought that was something you didn't want.

Unfortunately, this does not work when using debug-menu as middleware,
because here the inpectElementMenu has to still support arguments.

Could you change that part also? You'll have to manually set posX, posY and elm variable before calling
inspectMenuTemplate

[bbqjohan]

@parro-it

There, fixed! ^_^

[parro-it]

Thank you @bbqjohan