search

mkaply / cck2wizard

CCK2
Mozilla Public License 2.0
46 stars 9 forks source link

Add new code for hiding devtools #55

Closed mkaply closed 7 years ago

kaihendry commented 7 years ago

I attempted to grab the relevant bits and integrate it but F12 still seems to work: https://github.com/Webconverger/webconverger-addon/commit/f390dc402c74d6019141da936f0fc280361357dd

Tbh I have no idea how to debug if the code is even running.

mkaply commented 7 years ago

That's odd. in my testing, It removed F12 as well. Best way to debug is to throw in some "Components.utils.reportError('foo')" and use the Javascript console. I'll see what I can figure out.

kaihendry commented 7 years ago

Wonder if the old let console = (Cu.import("resource://gre/modules/devtools/Console.jsm", {})).console trick works still? Still my development iteration is maddening ... https://www.youtube.com/watch?v=aBIqpEFmfUI

kaihendry commented 7 years ago

Oh btw, here is your work in https://github.com/Webconverger/webconverger-addon/commit/cbad324af0ddeee25ae23b725ea3f71f817b6c71

mkaply commented 7 years ago

No, your console trick won't work. The devtools JS console is completely different on Firefox 52. And yes, I agree it's maddening.