controller.Application: should not catch contextMenu without explicit debug choice

Hi George,

obviously, what you can change inside the dev tools (console) also works inside your code :)

Meaning: you could just write Neo.util.Logger.enableComponentLogger = false; into your app / top level view controller.

@Dinkh implemented it in a way, that only CTRL & right clicks will trigger it (which is otherwise not used inside the browser context).

regarding the listener: by default. contextmenu is treated as a global event in neo. meaning: there is just one DOM based listener and it will send this event including the full dom nodes path to the app worker anyway.

so what the logger does is just a subscription inside the app worker realm (manager.DomEvent). of course other components can subscribe to it as well and trigger their own logic.

I see, thanks for explaining. (So why does logger subscribe in Application (and when should we), instead of in Logger itself?)

How about amending the proposal to make enableComponentLogger a DefaultConfig property, to conveniently choose whether to enable it in neo-config?

As Neo will not work without a neo-config file it is equally convenient to replicate an enabling entry, an a disabling entry, in the file. The default more or less documents framework opinion.

I encountered use of contextMenu with Plotly, so it seems legit, if perhaps ill-advised, in some circles.

For others who may search on the Ctrl Right-Click behavior: putting the Neo.util.Logger.enableComponentLogger = false; statement into the app.mjs file conveniently kills the thing.

Inasmuch as Ctrl Right-Click is used in the browser context ... charts e.g. ... a neo-config.json setting would be appropriate. That would document this secret behavior and disable it by default.

Developers would no longer be ambushed by the behavior. Instead, if the option was in the neo-config.json generated by the npx Neo script, developers would be aware this enabling feature from the get-go.

A related option would allow developers to specify an alternative key combination.

hi george,

let's move enableComponentLogger and enableLogsInProduction into src/DefaultConfig so that we can change it inside neo-config.json files.

@Dinkh